Differences between revisions 1 and 3 (spanning 2 versions)
⇤ ← Revision 1 as of 2010-01-15 09:33:54 →
Size: 1180
Editor: JonasFrellesen
Comment:
← Revision 3 as of 2010-08-16 10:24:35 → ⇥
Size: 1162
Editor: localhost
Comment: converted to 1.6 markup
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 




IssuesFromNs135  (last edited 2010-08-16 10:24:35 by localhost)