<html><head></head><body>Dear Костя,<br>
<br>
Is there a way to include a link to the diff in the bug instead of simply to you entire repository. I'd lime to review and comment on your fix (as I'm using my phone to check my mail) and have spent 5 minutes or so trying to get the website to just switch to the branch you mention and it doesn't respond, at least on Chrome for Android. I've seen other people post links to diffs, in the past. Is their a way to create a link which shows the work done in your branch?<br>
<br>
Regarding the bug, there are 2 separate issues in my mind.<br>
<br>
First, the use of distinct sorting algorithms between module creation and module consumption could cause this problem. I don't know if this is the problem, but I've always worried about it. We have 3 build scenarios for SWORD which worry me for this bug, i11n supported by: ICU, Qt, neither. If the module build tools configuration differs from the client configuration, I am worried our binary sort will loop, though it's only a suspicion we use i11n code in the sort algo (toupper).<br>
<br>
The second issue is user-displayed order of keys. The binary sort is an index for fast lookup. We've talked about keeping another index which preserves the 'book order' of the module. The LD driver is meant only for fast lookups, not for showing a module sequentially.<br>
<br>
Troy<br><br><div class="gmail_quote">On March 17, 2014 5:23:18 AM MST, "Костя Маслюк" <kostyamaslyuk@gmail.com> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">Please, consider this fix:<br /><br /><a href="http://www.crosswire.org/tracker/browse/MOD-61">http://www.crosswire.org/tracker/browse/MOD-61</a> and<br /><a href="http://www.crosswire.org/tracker/browse/API-150">http://www.crosswire.org/tracker/browse/API-150</a> is similar.<br /><br /><br />Actually i got weird things as things are not threaded in time. Blessings.<br /><br />2014-01-14 14:02 GMT+04:00 Chris Little <chrislit@crosswire.org>:<br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> On 1/14/2014 12:51 AM, Костя Маслюк wrote:<br /><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"><br /> What about the "inconsequent lexicon index" fix i have posted here a<br /> month ago? I realize that it would not be threated as Critical, but it<br /> is still bad behaivior that affects every Sword frontend.<br /><br /> I also realize that fix would not be complete, but no one even asked me<br /> for deeper investigation, if people have no time.<br /></blockquote><br /><br /> The patch you submitted just covers up the underlying problem without<br /> solving anything (or finding the root of the problem), so I would say it is<br /> not a candidate for inclusion in Sword.<br /><br /> Some brief discussion of the bug is here:<br /> <a href="http://www.crosswire.org/tracker/browse/MOD-61">http://www.crosswire.org/tracker/browse/MOD-61</a><br /><br /> If you can find the cause of these modules being built incorrectly (or being<br /> interpreted incorrectly), I'd love to see a fix be committed.<br /><br /> --Chris<br /><br /><br /><br /><hr /><br /> sword-devel mailing list: sword-devel@crosswire.org<br /> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br /> Instructions to unsubscribe/change your settings at above page<br /></blockquote><br /><hr /><br />sword-devel mailing list: sword-devel@crosswire.org<br /><a href="http://www.crosswire.org/mailman/listinfo/sword-devel">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br />Instructions to unsubscribe/change your settings at above page</pre></blockquote></div><br>
-- <br>
Sent from my Android device with K-9 Mail. Please excuse my brevity.</body></html>