NetarchiveSuite
Review (NS-63): Bug 1650
Author |
Jonas |
Moderator |
Jonas |
State |
Closed |
Objectives
Summary
Only cosmetic issues.
Total Time Used (Coding,Documentation,Review):
Time use (Coding,Documentation,Review) SVC: 0.5 JOLF: 0.25 HBK: 0.25
General comments:
Description |
Classification |
Status |
Comments on file 'trunk/src/dk/netarkivet/harvester/harvesting/HeritrixFiles.java', revision 888
Lines |
Description |
Classification |
Status |
62-63 |
There should no be two empty lines next to each other. |
Cosmetic |
OK |
101 |
. |
Cosmetic |
OK |
104 |
. |
Cosmetic |
OK |
105 |
. |
Cosmetic |
OK |
110 |
. |
Cosmetic |
OK |
113, 117-118, 123-124 |
Bug 1650: Check that the jmx-files exists. Describe that it is not checked here, and why. |
Cosmetic |
OK |
363-366 |
Improper method description. Does not respect the codestyle. |
Cosmetic |
OK |
370-373 |
Improper method description. Does not respect the codestyle. |
Cosmetic |
OK |
Comments on file 'trunk/src/dk/netarkivet/harvester/harvesting/JMXHeritrixController.java', revision 888
Lines |
Description |
Classification |
Status |
283 |
TODO: |
Cosmetic |
OK |
287 |
"The password file '" |
Cosmetic |
OK |
287-291 |
The string is only used once and a different message is thrown in IOFailure. |
Cosmetic |
OK |
290 |
"Failed to read password file '" |
Cosmetic |
OK |
296-300 |
The string is only used once and a different message is thrown in IOFailure. |
Cosmetic |
OK |
296 |
"The access file '" |
Cosmetic |
OK |
299 |
"Failled to read access file '" |
Cosmetic |
OK |