Thanks! <br><br>I was unable to sign up on the bugs page (the signon form submitted, but didn't return or timeout), so I've attached the zipped patch and JUnit here. Also, after grabbing the versification changes, BibleInfo and BibleNames seem to have some problems that cause my tests to fail after patching. I'm new to SVN, so it's possible that I didn't pick up all the changes (Using an Eclipse SVN plug-in, I assume using the "update" function with a project selected means that it will update all the files in the project). <br><br>Here's a repeat of what the patch fixes:<br><br>There's a for loop in AccuracyType.fromText(String, String[], AccuracyType, VerseRange) that doesn't use its index when building the NoSuchVerseException<br>(the array is always referenced at index 1). For parts.length == 1, this causes an ArrayIndexOutOfBoundsException, for parts.length >= 3, this produces an invalid message in the exception. <br>This patch moves the exception-building process
into its own static method. <br><br>The solution is "correct," but it may not fit the JSword coding conventions. If it doesn't, please let me know so I can accommodate future changes.<br><br>Thanks,<br>-Phillip<br><br><br><br><br><b><i>DM Smith <dmsmith555@yahoo.com></i></b> wrote:<blockquote class="replbq" style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"> Phillip,<div><br class="khtml-block-placeholder"><div>Thanks! And welcome to JSword!</div><div><br class="khtml-block-placeholder"></div><div>Yes, zip it and post it here. Or join the bugs database at <a href="http://www.crosswire.org/bugs">www.crosswire.org/bugs</a> and post a bug against JSword and attach the patch and junit there. No need to zip attachments there, but it is OK to do so.</div><div><br class="khtml-block-placeholder"></div><div>In His Service,</div><div><span class="Apple-tab-span" style="white-space: pre;"> </span>DM Smith</div><div><br><div><div>On Oct
2, 2006, at 10:08 PM, P. R. B. wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hello, <br><br>I noticed a for loop in AccuracyType.fromText(String, String[], AccuracyType, VerseRange) that doesn't use its index when building the NoSuchVerseException<br>(the array is always referenced at index 1). For parts.length == 1, this causes an ArrayIndexOutOfBoundsException, for parts.length >= 3, this produces an invalid message in the exception. <br><br>I have a SVN patch for it with a corresponding JUnit class. Assuming this problem isn't a known issue, is this patch something worth contributing? If so, should these be zipped and posted here or elsewhere?<br><br>Thanks,<br>-Phillip<br><br><div> <br class="khtml-block-placeholder"></div><hr size="1">How low will we go? Check out Yahoo! Messenger’s low <a href="http://us.rd.yahoo.com/mail_us/taglines/postman8/*http://us.rd.yahoo.com/evt=39663/*http://voice.yahoo.com"> PC-to-Phone call rates.</a><div
style="margin: 0px;">_______________________________________________</div><div style="margin: 0px;">jsword-devel mailing list</div><div style="margin: 0px;"><a href="mailto:jsword-devel@crosswire.org">jsword-devel@crosswire.org</a></div><div style="margin: 0px;"><a href="http://www.crosswire.org/mailman/listinfo/jsword-devel">http://www.crosswire.org/mailman/listinfo/jsword-devel</a></div> </blockquote></div><br></div></div>_______________________________________________<br>jsword-devel mailing list<br>jsword-devel@crosswire.org<br>http://www.crosswire.org/mailman/listinfo/jsword-devel<br></blockquote><br><p> 
<hr size=1>Get your own <a href=" http://us.rd.yahoo.com/evt=43290/*http://smallbusiness.yahoo.com/domains"
>web address for just $1.99/1st yr</a>. We'll help. <a href="http://us.rd.yahoo.com/evt=41244/*http://smallbusiness.yahoo.com/"
>Yahoo! Small Business</a>.