1871
Comment:
|
← Revision 5 as of 2010-08-16 10:24:28 ⇥
1850
converted to 1.6 markup
|
Deletions are marked like this. | Additions are marked like this. |
Line 36: | Line 36: |
|| 283 || TODO: || Cosmetic || NOTOK || || 287 || "The password file '" || Cosmetic || NOTOK || || 287-291 || The string is only used once and a different message is thrown in IOFailure. || Cosmetic || NOTOK || || 290 || "Failed to read password file '" || Cosmetic || NOTOK || || 296-300 || The string is only used once and a different message is thrown in IOFailure. || Cosmetic || NOTOK || || 296 || "The access file '" || Cosmetic || NOTOK || || 299 || "Failled to read access file '" || Cosmetic || NOTOK || |
|| 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 || |
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 |