Differences between revisions 1 and 2
⇤ ← Revision 1 as of 2009-02-20 12:55:59 →
Size: 1153
Editor: ColinRosenthal
Comment:
← Revision 2 as of 2009-03-12 16:22:07 → ⇥
Size: 1202
Editor: SoerenCarlsen
Comment:
Deletions are marked like this. Additions are marked like this.
Line 1: Line 1:
## page was renamed from IssueFoundInReviewNs16

Review (NS-16): Bugfix 1152

Author

Colin

Moderator

Colin

State

Closed

Objectives

Total Time Used (Coding,Documentation,Review): 
Time use (Coding,Documentation,Review) 
CSR: 2 MD
SVC: 0.5 MD
General comments: 
  
 Description 

  
 Classification 

  
 Status 






Comments on file 'trunk/src/dk/netarkivet/common/distribute/arcrepository/ARCLookup.java', revision 584


  
 Lines 

  
 Description 

  
 Classification 

  
 Status 




  
 115 

  
 missing spaces between arguments 

  
 Cosmetic 

  
 NOTOK 






Comments on file 'trunk/src/dk/netarkivet/viewerproxy/WebProxy.java', revision 584


  
 Lines 

  
 Description 

  
 Classification 

  
 Status 




  
 289 

  
 Spelling: charcters 

  
 Cosmetic 

  
 NOTOK 




  
 295 

  
 Make consistent or explain why "{" is escaped and not "}"? 

  
 Minor 

  
 NOTOK 




  
 288 

  
 You probably meant: java.net.URI(String s) Adapt javadoc to liittle bit more precise 

  
 Cosmetic 

  
 NOTOK 




  
 272 

  
 Improperly named variable: Should be renamed to uriString 

  
 Cosmetic 

  
 NOTOK 




IssuesFoundInReviewNs16  (last edited 2010-08-16 10:25:15 by localhost)