<HTML><BODY style="word-wrap: break-word; -khtml-nbsp-mode: space; -khtml-line-break: after-white-space; ">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-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">_______________________________________________</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">jsword-devel mailing list</DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><A href="mailto:jsword-devel@crosswire.org">jsword-devel@crosswire.org</A></DIV><DIV style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 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></BODY></HTML>