1566
Comment:
|
1560
|
Deletions are marked like this. | Additions are marked like this. |
Line 29: | Line 29: |
|| 54 || Java-doc. Does not say what this function does. || Cosmetic || NOTOK || | || 54 || Java-doc. Does not say what this function does. || Cosmetic || OK || |
Line 36: | Line 36: |
|| 46 || Finish javadoc || Cosmetic || NOTOK || | || 46 || Finish javadoc || Cosmetic || OK || |
Review (NS-146): Miscellaneous refactoring of code
Author |
Søren |
Moderator |
Søren |
State |
Closed |
Objectives
Review miscellaneous refactoring of code
Summary
Review completed; followup by SVC
Total Time Used (Coding,Documentation,Review):
Time use (Coding,Documentation,Review) SVC: 0.5 JOLF: 0.1
General comments:
Description |
Classification |
Status |
Comments on file 'trunk/src/dk/netarkivet/common/tools/JMSBroker.java', revision 1259
Lines |
Description |
Classification |
Status |
27 |
Why has this extra line been added? |
Cosmetic |
OK |
Comments on file 'trunk/src/dk/netarkivet/harvester/tools/HarvestTemplateApplication.java', revision 1260
Lines |
Description |
Classification |
Status |
54 |
Java-doc. Does not say what this function does. |
Cosmetic |
OK |
Comments on file 'trunk/src/dk/netarkivet/common/utils/batch/FileRemover.java', revision 1262
Lines |
Description |
Classification |
Status |
34-49 |
Java-doc |
Cosmetic |
OK |
46 |
Add to javadoc: This will not be allowed by the default Java Security Policy. |
Cosmetic |
OK |
Comments on file 'trunk/src/dk/netarkivet/common/tools/WriteBytesToFile.java', revision 1260
Lines |
Description |
Classification |
Status |
46 |
Finish javadoc |
Cosmetic |
OK |