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 |