Re: [Archive-access-cvs] archive-access/projects/wayback/src/java/org/archive/wayback/cdx/indexer BDBResourceIndexWriter.java,1.1,1.2

[stack <st...@ar...>]

Brad wrote:

>Update of /cvsroot/archive-access/archive-access/projects/wayback/src/java/org/archive/wayback/cdx/indexer
>In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv11989/src/java/org/archive/wayback/cdx/indexer
>
>Modified Files:
>	BDBResourceIndexWriter.java 
>Log Message:
>FEATURE: changed String.contains to indexOf for backvards compat
>
>Index: BDBResourceIndexWriter.java
>===================================================================
>RCS file: /cvsroot/archive-access/archive-access/projects/wayback/src/java/org/archive/wayback/cdx/indexer/BDBResourceIndexWriter.java,v
>retrieving revision 1.1
>retrieving revision 1.2
>diff -C2 -d -r1.1 -r1.2
>*** BDBResourceIndexWriter.java	16 Nov 2005 03:11:29 -0000	1.1
>--- BDBResourceIndexWriter.java	17 Nov 2005 02:49:59 -0000	1.2
>***************
>*** 89,93 ****
>  			}
>  			lineNumber++;
>! 			if ((lineNumber == 1) && (line.contains(CDX_HEADER_MAGIC))) {
>  				continue;
>  			}
>--- 89,93 ----
>  			}
>  			lineNumber++;
>! 			if ((lineNumber == 1) && (-1 != line.indexOf(CDX_HEADER_MAGIC))) {
>  				continue;
>  
>
To be consistent -- e.g. consistent with the just-previous clause -- I'd 
suggest flipping putting the '-1' to RHS of the '==' rather than LHS.

St.Ack

>  			}
>
>
>
>-------------------------------------------------------
>This SF.Net email is sponsored by the JBoss Inc.  Get Certified Today
>Register for a JBoss Training Course.  Free Certification Exam
>for All Training Attendees Through End of 2005. For more info visit:
>http://ads.osdn.com/?ad_id=7628&alloc_id=16845&op=click
>_______________________________________________
>Archive-access-cvs mailing list
>Arc...@li...
>https://lists.sourceforge.net/lists/listinfo/archive-access-cvs
>  
>