⇤ ← Revision 1 as of 2010-01-15 09:33:54
1180
Comment:
|
1162
|
Deletions are marked like this. | Additions are marked like this. |
Line 7: | Line 7: |
Line 10: | Line 9: |
{{{ | {{{ |
Line 15: | Line 15: |
Line 22: | Line 21: |
|| 514 || if sendAndWaitForOneReply is timing out then replyNetMsg is set to null, which means we throw an IOFailure. So we don't return null on timeout. || Cosmetic || NOTOK || || 539-540 || The last comment is incorrect. || Cosmetic || NOTOK || || 574 || if sendAndWaitForOneReply is timing out then replyNetMsg is set to null, which means we throw an IOFailure. So we don't return null on timeout. || Cosmetic || NOTOK || || 600 || The last comment is incorrect. || Cosmetic || NOTOK || || 659 || The last comment is incorrect. || Cosmetic || NOTOK || |
|| 514 || if sendAndWaitForOneReply is timing out then replyNetMsg is set to null, which means we throw an IOFailure. So we don't return null on timeout. || Cosmetic || OK || || 539-540 || The last comment is incorrect. || Cosmetic || OK || || 574 || if sendAndWaitForOneReply is timing out then replyNetMsg is set to null, which means we throw an IOFailure. So we don't return null on timeout. || Cosmetic || OK || || 600 || The last comment is incorrect. || Cosmetic || OK || || 659 || The last comment is incorrect. || Cosmetic || OK || |
Review (NS-135): Bug 1836: The value of the variable 'File from' must not be null.
Author |
Jonas |
Moderator |
Jonas |
State |
Closed |
Objectives
Total Time Used (Coding,Documentation,Review):
Time use (Coding,Documentation,Review) JOLF: 0.3 HBK: 0.1
General comments:
Description |
Classification |
Status |
Comments on file 'trunk/src/dk/netarkivet/archive/arcrepository/distribute/JMSArcRepositoryClient.java', revision 1208
Lines |
Description |
Classification |
Status |
514 |
if sendAndWaitForOneReply is timing out then replyNetMsg is set to null, which means we throw an IOFailure. So we don't return null on timeout. |
Cosmetic |
OK |
539-540 |
The last comment is incorrect. |
Cosmetic |
OK |
574 |
if sendAndWaitForOneReply is timing out then replyNetMsg is set to null, which means we throw an IOFailure. So we don't return null on timeout. |
Cosmetic |
OK |
600 |
The last comment is incorrect. |
Cosmetic |
OK |
659 |
The last comment is incorrect. |
Cosmetic |
OK |