<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-12-11 21:57 GMT+03:00 Werner Pamler via Lazarus <span dir="ltr"><<a href="mailto:lazarus@lists.lazarus-ide.org" target="_blank">lazarus@lists.lazarus-ide.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">Am 11.12.2017 um 18:32 schrieb Werner Pamler via Lazarus:<br>
</span><span class="gmail-"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I cloned your repo on my HD, and went through it step by step while always trying to build lhelp, I could remove everything from the folder chm except for these units: chmfifimain, chmreader, chmsitemap, chmtypes (and chmobjinstance.inc). I think it is possible to extract the essential changes from these units to get at least the new reading functionality. I can try this - but: before beginning I'd like to get positive feedback from fpc devs that this will be merged (I have some old patches hanging around there, and it is not always motivating to submit fpc patches).<br></blockquote></span></blockquote><div><br></div><div>There is no special new reading functionality, only bugfixes and code improvements. I spend many hours just to understand how it works and how it must work properly. It's not trivial without proper documentation.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
As for the unit lcid_conv: This fits into Lazarus' LazUtils, and should go there independently of how this story ends here. I have commit rights to Lazarus and can put it there. Any objections?<br></blockquote></span></blockquote><div><br></div><div>I don't know, where is better place for such file. Locale ID is MS Windows specific, so maybe it better place to some windows - related unit or LazUtils?</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Sergey, I loaded a chm having codepage iso-8859-1 into your modified lhelp. It crashes in TIpChmDataProvider.DetectHtmlC<wbr>odepage when trying to convert the string 88591 to a Word. Also, this codepage (as well as the other iso's) are not considered in lclid_conv.</blockquote><div><br></div><div> iso-8859-1 encoding have not relation to MS Locale ID. It HTML-specific, and must handled by HTML viewer. I do dirty hack with HTML codepage coversion just because don't ready to make fixes in TIpHtmlPanel.<br></div><div><br></div><div>-- <br></div></div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div><strong>Bodrov Sergey</strong><br></div><div>software development, IT consulting</div><div><a href="http://www.serbod.com" target="_blank">http://www.serbod.com</a></div><div><div><u>Phone (Belarus):</u> +375(25)794-21-58</div><div><u style="font-size:12.8px">Skype:</u><span style="font-size:12.8px"> sergey.bodrov1</span><br></div></div><div><u>e-mail:</u> <a href="mailto:serbod@gmail.com" target="_blank">serbod@gmail.com</a>, <a href="mailto:oxotnuk@yandex.ru" target="_blank">oxotnuk@yandex.ru</a></div><div><br></div></div></div></div></div></div>
</div></div>