⇤ ← Revision 1 as of 2009-06-11 11:37:44
1071
Comment:
|
1065
|
Deletions are marked like this. | Additions are marked like this. |
Line 14: | Line 14: |
Line 21: | Line 22: |
|| File does not end with a new line: Add new line after EOF || Cosmetic || NOTOK || || add a unittest for the get() method || Cosmetic || NOTOK || |
|| File does not end with a new line: Add new line after EOF || Cosmetic || OK || || add a unittest for the get() method || Cosmetic || OK || |
Line 25: | Line 28: |
|| 148 || remove tab from line || Cosmetic || NOTOK || || 148 || remove unused line || Cosmetic || NOTOK || |
|| 148 || remove tab from line || Cosmetic || OK || || 148 || remove unused line || Cosmetic || OK || |
Review (NS-46): Review of Bug 1694: LocalArcRepositoryClient is broken Fixes the ARCReader constructor call.
Author |
Søren |
Moderator |
Søren |
State |
Closed |
Objectives
/trunk/src/dk/netarkivet/common/distribute/arcrepository/LocalArcRepositoryClient.java, revision 838, lines 147-148
Summary
Followup by Nicolas Giraud
Total Time Used (Coding,Documentation,Review):
Time use (Coding,Documentation,Review) NGiraud: 0.25 SVC: 0
General comments:
Description |
Classification |
Status |
File does not end with a new line: Add new line after EOF |
Cosmetic |
OK |
add a unittest for the get() method |
Cosmetic |
OK |
Comments on file 'trunk/src/dk/netarkivet/common/distribute/arcrepository/LocalArcRepositoryClient.java', revision 838
Lines |
Description |
Classification |
Status |
148 |
remove tab from line |
Cosmetic |
OK |
148 |
remove unused line |
Cosmetic |
OK |