On Thu, Mar 22, 2018 at 4:01 AM, David Haslam <dfh...@protonmail.com> wrote:
> Hi Greg, > > With *Plain* output format, there's a minor side effect in the way the > Psalm title was output in the wrong place. > Verse 2 of the same Psalm was output with a *leading space* before the > Reference. > > https://www.dropbox.com/s/uz1amnrp7ez0b4y/Screenshot% > 202018-03-22%2008.37.42.png?dl=0 > > This might be less obvious in a terminal, but it's quite evident when the > output is piped to a file. > > Please ensure that the patch also *removes* this spurious space, if > perhaps it ddidn't do so already. > As someone who isn't the maintainer of either Diatheke or the filters, I'm not about to dig through the source to find one errant space in the output. Doubly so in that I do not have nor can I easily locate the SpaRV1865 module that you have screenshot there. I don't see the same artifact in KJV so I am going to conclude, for now, that this is either a bug that was already fixed between the older version of Diatheke that you are using and now or that it is an artifact of the module itself. Compare the KJV and the Spa1909 that's on CrossWire's server: $ diatheke -b KJV -o h -f Plain -k Ps.3 Psalms 3:1: LORD, how are they increased that trouble me! many are they that rise up against me. A Psalm of David, when he fled from Absalom his son. Psalms 3:2: Many there be which say of my soul, There is no help for him in God. Selah. $ diatheke -b SpaRV1909 -o h -f Plain -k Ps.3 Psalms 3:1: Salmo de David, cuando huía de delante de Absalom su hijo. ¡OH Jehová, cuánto se han multiplicado mis enemigos! muchos se levantan contra mí. Psalms 3:2: Muchos dicen de mi vida: No hay para él salud en Dios. (Selah.) The SpaRV1909 module definitely has spurious spaces around many different places. I suspect that your SpaRV1865 module is in a similar place. --Greg > While I'm on a roll, I noticed also that there's a subtle change in how > diatheke treats the start and end of a *hi* element. > There's now a *space* after the leading asterisk and *another space* > before the trailing asterisk. > https://www.dropbox.com/s/2h2u1768rgx9p95/Screenshot% > 202018-03-22%2008.38.21.png?dl=0 > > If this change was unintended, it ought to be reverted for consistency > with earlier versions. > Sounds like a filter thing. --Greg > > Best regards, > > David > > Sent with ProtonMail <https://protonmail.com> Secure Email. > > ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ > On 21 March 2018 3:33 AM, Greg Hellings <greg.helli...@gmail.com> wrote: > > > > On Tue, Mar 20, 2018 at 10:21 PM, Troy A. Griffitts <scr...@crosswire.org> > wrote: > >> If I had to guess, I suspect diatheke is not calling renderText before >> asking for the header. The renderText method triggers all entryAttributes >> to be filled. The header is an entryAttribute. >> > > Spot on. If I were offering commentary, I would call that a bug in the UX > of the API. But I do get why it is that way. Attaching a patch that solves > the problem. > > --Greg > > >> >> >> On March 20, 2018 8:13:41 PM MST, Greg Hellings <greg.helli...@gmail.com> >> wrote: >> >>> To be quite specific: diatheke does not encounter that header in the >>> preverse content until it reaches Psalm 3:2 for some reason that is beyond >>> my ken. Therefore, it is properly rendering that content as preverse, but >>> it has attached it to the wrong verse. Output from my slightly modified >>> diatheke confirms this: >>> >>> $ ./inst/bin/diatheke -b KJV -o h -f Plain -k Ps 3 >>> key: Psalms 3:1 >>> header: >>> Psalms 3:1: LORD, how are they increased that trouble me! many are they >>> that rise up against me. >>> key: Psalms 3:2 >>> header: <title canonical="true" type="section"><w >>> savlm="strong:H04210">A Psalm</w> <w savlm="strong:H01732">of David</w>, <w >>> savlm="strong:H01272">when he fled</w> <w savlm="strong:H06440">from</w> <w >>> savlm="strong:H053">Absalom</w> <w savlm="strong:H01121">his >>> son</w>.</title> >>> A Psalm of David, when he fled from Absalom his son. >>> Psalms 3:2: Many there be which say of my soul, There is no help for him >>> in God. Selah. >>> >>> --Greg >>> >>> On Tue, Mar 20, 2018 at 9:53 PM, Greg Hellings <greg.helli...@gmail.com> >>> wrote: >>> >>>> It's easier to see the problem when using plain formatting: >>>> >>>> $ diatheke -b KJV -o h -f Plain -k Ps 3 >>>> Psalms 3:1: LORD, how are they increased that trouble me! many are they >>>> that rise up against me. >>>> A Psalm of David, when he fled from Absalom his son. >>>> Psalms 3:2: Many there be which say of my soul, There is no help for >>>> him in God. Selah. >>>> Psalms 3:3: But thou, O LORD, art a shield for me; my glory, and the >>>> lifter up of mine head. >>>> Psalms 3:4: >>>> I cried unto the LORD with my voice, and he heard me out of his holy >>>> hill. Selah. >>>> Psalms 3:5: I laid me down and slept; I awaked; for the LORD sustained >>>> me. >>>> Psalms 3:6: I will not be afraid of ten thousands of people, that have >>>> set themselves against me round about. >>>> Psalms 3:7: Arise, O LORD; save me, O my God: for thou hast smitten all >>>> mine enemies upon the cheek bone; thou hast broken the teeth of the >>>> ungodly. >>>> Psalms 3:8: Salvation belongeth unto the LORD: thy blessing is upon thy >>>> people. Selah. >>>> (KJV) >>>> >>>> --Greg >>>> >>>> >>>> On Tue, Mar 20, 2018 at 9:50 PM, Greg Hellings <greg.helli...@gmail.com >>>> > wrote: >>>> >>>>> I'm not so sure your initial assertion is correct. >>>>> >>>>> $ diatheke -b KJV -o h -k Ps 3 >>>>> Psalms 3:1: <w savlm="strong:H03068"><seg><di >>>>> vineName>Lord</divineName></seg></w>, <w savlm="strong:H07231">how >>>>> are they increased</w> <w savlm="strong:H06862">that trouble</w> <w >>>>> savlm="strong:H07227">me! many</w> <transChange >>>>> type="added">are</transChange> <w savlm="strong:H06965">they that rise >>>>> up</w> against me. >>>>> <title canonical="true" type="section"><w savlm="strong:H04210">A >>>>> Psalm</w> <w savlm="strong:H01732">of David</w>, <w >>>>> savlm="strong:H01272">when he fled</w> <w savlm="strong:H06440">from</w> >>>>> <w >>>>> savlm="strong:H053">Absalom</w> <w savlm="strong:H01121">his >>>>> son</w>.</title>Psalms 3:2: <w savlm="strong:H07227">Many</w> <transChange >>>>> type="added">there be</transChange> <w savlm="strong:H0559">which say</w> >>>>> <w savlm="strong:H05315">of my soul</w>, <transChange type="added">There >>>>> is</transChange> <w savlm="strong:H03444">no help</w> <w >>>>> savlm="strong:H0430">for him in God</w>. <w savlm="strong:H05542">Selah</w >>>>> >. >>>>> >>>>> If you notice carefully, the canonical title is there. It is just >>>>> misplaced. It comes after the text of Psalm 3:1 and not before. >>>>> >>>>> --Greg >>>>> >>>>> On Tue, Mar 20, 2018 at 4:40 PM, David Haslam <dfh...@protonmail.com> >>>>> wrote: >>>>> >>>>>> Maybe someone could patch diatheke to fix this? >>>>>> >>>>>> Best regards, David >>>>>> >>>>>> >>>>>> Sent from ProtonMail Mobile >>>>>> >>>>>> >>>>>> On Wed, Mar 14, 2018 at 12:14, David Haslam <dfh...@protonmail.com> >>>>>> wrote: >>>>>> >>>>>> Diatheke does not output the canonical Psalm titles when >>>>>> output option h is used. >>>>>> >>>>>> Try this and see what I mean. >>>>>> >>>>>> diatheke -b KJV -o h -k Ps 3 >>>>>> >>>>>> This is a significant lack of feature. >>>>>> >>>>>> Best regards, >>>>>> >>>>>> David >>>>>> >>>>>> Sent from ProtonMail Mobile >>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> 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 >>>>>> >>>>> -- >> Sent from my Android device with K-9 Mail. Please excuse my brevity. >> > >
_______________________________________________ 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