<div dir="ltr"><div>Thanks for the quick response. I have had a brief look at the new commits.</div><div><br></div><div>A lot of the attributes aren't being returned now so it is tricky to test and there are various errors but running the current tip '<a href="https://github.com/crosswire/jsword/commit/80020f51c6a762d458ce8ae70007b78eadee1fb3" class="" title="Partial load of conf file.
Fixed a bug from the last commit in the loading of mods.d.tar.gz, adjustConfig was accidentally deleted.
This is part 1. Part 2 will be adding the ability to reload fully." style="font-size:13px;color:rgb(64,120,192);outline:0px;font-family:Helvetica,arial,nimbussansl,liberationsans,freesans,clean,sans-serif,'Apple Color Emoji','Segoe UI Emoji','Segoe UI Symbol';line-height:20px;white-space:nowrap;background-color:rgb(245,245,245)">Partial load of conf file.</a>' the SBMD for eBible is now only a quarter of the original size at 10Mb which is fine but I still don't understand why it is so large for the minimal attribute set now being returned.</div><div><br></div><div>I get a lot of errors like:</div><div><div>SwordBookMetaData(492): Book not supported: malformed conf file for [BBE] no ModDrv found.</div><div>SwordBookMetaData(492): Malformed conf file: missing [BBE]Description=. Using BBE</div></div><div><br></div><div>and peculiarly the eBible repo seems to be the only repo I can use because all the others error.</div><div><br></div><div>I also tried the previous commit <a href="https://github.com/crosswire/jsword/commit/cc32ba8f1bb245932a747390d03874b2be70e9a1" class="" title="Cut the memory requirements of a SwordBookMetaData in half." style="color:rgb(78,87,91);outline:0px;font-family:Helvetica,arial,nimbussansl,liberationsans,freesans,clean,sans-serif,'Apple Color Emoji','Segoe UI Emoji','Segoe UI Symbol';font-size:15px;font-weight:bold;line-height:21px;background-color:rgb(247,251,252)">Cut the memory requirements of a SwordBookMetaData in half.</a> but it did not work because basic attributes like language were not being returned.</div><div><br></div><div>I still don't understand why removing configSword should reduce memory by half because it should just be removing references to data that is also referenced from configAll, so it would reduce memory slightly but not much.</div><div><br></div><div>Martin</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 10 January 2016 at 04:14, DM Smith <span dir="ltr"><<a href="mailto:dmsmith@crosswire.org" target="_blank">dmsmith@crosswire.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">OK. That’s done. Also accidentally introduced a bug with the last commit. It is noticeably fast.<div><br></div><div>Next up, allow for *a* SwordBookMetaData to be reloaded fully. This is needed to bring in all the other elements which are information only, such as About, in order to display info to the end user. Since the user will only look at one modules info at a time, it will load that one. You may need to change your code (hope not) to force that one to reload.</div><div><br></div><div>Give the code a try to see if it solves your out of memory error.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>DM</div><div><br></div></font></span><div><br><div><blockquote type="cite"><div><div class="h5"><div>On Jan 9, 2016, at 9:06 PM, DM Smith <<a href="mailto:dmsmith@crosswire.org" target="_blank">dmsmith@crosswire.org</a>> wrote:</div><br></div></div><div><div><div class="h5"><div style="word-wrap:break-word">I’ll be adding a filter to IniSection. Something like:<div>if (filter.test(key)) {</div><div><span style="white-space:pre-wrap"> </span>use the key</div><div>} else {</div><div><span style="white-space:pre-wrap"> </span>do nothing</div><div>}</div><div><br></div><div>SwordBookMetaData will be responsible for building the filter. At least for a first go around. A single object should do.</div><div><br></div><div>DM</div><div><br><div><blockquote type="cite"><div>On Jan 9, 2016, at 6:29 PM, DM Smith <<a href="mailto:dmsmith@crosswire.org" target="_blank">dmsmith@crosswire.org</a>> wrote:</div><br><div><div style="word-wrap:break-word"><div><div><blockquote type="cite" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><div><div dir="ltr"><div><br>Yes, like you I have thought of streamlining conf loading for repo lists. One idea I had was to enable specification of a filter to SwordBookMetaData to limit the conf values that are stored.</div></div></div></blockquote><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><br></div><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;float:none;display:inline!important">I was thinking of something similar. My ideas aren’t good enough to be put into practice, but some kind of flag indicating empty, partially or fully loaded. Empty would mean that it hasn’t gone to disk to get the conf. Partial means that it read everything, but threw away most as not interesting (since the conf does not have order you have to read and parse it all). Full would mean that nothing was pitched. SwordBookMetaData.getProperty would need to be changed to determine whether the key is in memory or might be on disk and do the right thing. Or we could keep getProperty as it is and if you want one of the fields that is not stored (e.g. About) you have to call reload().</span></div><div><br></div><div>Maybe we could also cache that info into a separate file(s)? When mods.d.tar.gz is updated then the cache would be recomputed. In doing the computation, each conf would be read then pitched. Basically, the storage would be o.c.c.utils.Ini, if one file or IniSection, if many files.</div><div><br></div><div>What do you think?</div></div></div><br></div></blockquote></div><br></div></div></div></div><span class="">_______________________________________________<br>jsword-devel mailing list<br><a href="mailto:jsword-devel@crosswire.org" target="_blank">jsword-devel@crosswire.org</a><br><a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/jsword-devel</a><br></span></div></blockquote></div><br></div></div><br>_______________________________________________<br>
jsword-devel mailing list<br>
<a href="mailto:jsword-devel@crosswire.org">jsword-devel@crosswire.org</a><br>
<a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/mailman/listinfo/jsword-devel</a><br>
<br></blockquote></div><br></div>