I’ve had family visting this weekend. I’d like to look at it more closely. The code tries to be very careful in how it handles titles before the first verse of a chapter. The patches certainly solve a bug where it appends NT intros to the OT.
DM > On Jun 25, 2016, at 9:16 AM, Troy A. Griffitts <scr...@crosswire.org> wrote: > > Dear John, > > I've applied your osisheadings patch mentioned below. Thank you! > > DM, do you have any comments on the osis2mod patches? You have been the > pumpkin holder for that tool for a while now. I am happy to apply the > patches but, of course, want to defer to you first. > > Troy > > > On 06/13/2016 07:35 PM, John Austin wrote: >> I'm updating old modules in IBT's repository to use CrossWire best practice >> OSIS, using usfm2osis.py. So these OSIS files are CrossWire standard, but >> the corner cases are revealing some bugs to fix. I've reported these on >> Jira, along with patches, but mention them here in case there is discussion >> to be had. >> >> osisheadings.cpp: >> - Canonical pre-verse Psalm titles are being filtered out by the >> osis-headings filter (like Psalm 10 in SynodalProt which has a canonical >> title before verse 1). >> >> osis2mod.cpp: >> - Currently New Testament introductory material is appended to the end of >> Malachi rather than being prepended to Matthew's introduction. >> >> - A majorSection osis2mod.cpp patch from Dec 2014 causes new problems. >> Introduction text is now lost when majorSection titles appear in the Bible >> or Testament introduction. I believe both the original issue and the new >> issues can be fixed by treating majorSection titles (and subSection titles >> for good measure) as regular section divs in terms of pre-verse content. >> Testing has shown this to work perfectly on IBT's new UZV module which uses >> majorTitles in many places (Bible & Testament introductions, Proverbs, and >> Daniel). >> >> -john >> >> _______________________________________________ >> sword-devel mailing list: sword-devel@crosswire.org >> <mailto:sword-devel@crosswire.org> >> http://www.crosswire.org/mailman/listinfo/sword-devel >> <http://www.crosswire.org/mailman/listinfo/sword-devel> >> Instructions to unsubscribe/change your settings at above page > > _______________________________________________ > sword-devel mailing list: sword-devel@crosswire.org > http://www.crosswire.org/mailman/listinfo/sword-devel > Instructions to unsubscribe/change your settings at above page
_______________________________________________ sword-devel mailing list: sword-devel@crosswire.org http://www.crosswire.org/mailman/listinfo/sword-devel Instructions to unsubscribe/change your settings at above page