<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
<title></title>
I have a patch that does the following: ( I am finishing testing now )<br>
Verse Accuracy.<br>
The code dealing with verse accuracy was fragile (huge switch, inside
of switch, inside of switch statements.) So I have re-written it. It is
significantly smaller, tighter, and a bit faster. And it fixes a few
bugs.<br>
The significant differences are:<br>
Accuracy has been changed to an enumerated type with behavior (like
ModuleType). I.e. it is a total rewrite.<br>
ACCURACY_NONE is gone. This kind of thing was needed as a catch-all for
switch statements. It is not needed for enumerated type.<br>
ACCURACY_NUMBER_ONLY has been split into CHAPTER_ONLY and VERSE_ONLY.
This improved the accuracy of determining what a verse is.<br>
I added VerseFactory and VerseRangeFactory as the only means to create
a Verse or VerseRange from a string.<br>
I eliminated #3 as a means of specifying a book and v5 as a means of
specifying a verse. The basic reason was that these were not available
along all paths of parsing verses. I could have fixed it but did not.
It would not be too hard to add it back in along all paths.<br>
<br>
Sentence Case<br>
I made Sentence Case an enumerated type with behavior. Called CaseType.<br>
fixed the spelling of sentence (it was sentance), but left SentanceUtil
class name alone.<br>
<br>
Blur restriction<br>
I made Blur restriction an enumerated type with behavior<br>
<br>
Passage Constants<br>
I moved the remaining constants out of PassageConstants into the class
that should be their home.<br>
<br>
HTTPSwordInstaller<br>
Added //$NON-NLS... to the strings. Added some whitespace.<br>
<br>
BibleInfo<br>
removed _ from variable names<br>
made getBookNumber return -1 if no book is found. Rather than throwing
an exception. Turns out the code was using this method as a test and
get. Exceptions are expensive.<br>
<br>
</body>
</html>