<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-12-11 16:14 GMT+03:00 Marco van de Voort 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-">On Sun, Dec 10, 2017 at 01:25:59PM +0300, Sergey Bodrov via Lazarus wrote:<br>
><br>
> I change code formatting, many private names, add many comments and embrace<br>
> whole blocks of code into try..finally. And now can't tell, which parts is<br>
> most essential.<br>
<br>
</span>Yes. Somebody will have to figure that mess out, preferably by somebody who<br>
has some recollection about what he has done.  It is always a bad idea to do<br>
big style related operations mixing it with real fixes.  Now you know why<br>
:-)<br></blockquote><div><br></div><div>Problem is that I first change code formatting just to make it readable, and then do fixes. Most fixes was impossible without reformatting.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
And when I had a quick peak, I scanned two whole files, and didn't find any<br>
fixes except adding "const" to some parameters, and some 2.6.x era unicode<br>
fixes that used Lazarus units, and thus can't be merged into FPC repo<br>
anyway.  (using lazconvencoding or something like that).<br></blockquote><div><br></div><div>Hmm, don't remember, that I doing addiction of new includes to 'chm' units. And encoding-related operations was made in LHelp project, after HTML content readed from CHM file, but before it passed to IPro HTML viewer.</div><div><br></div><div>There was some compiler warnings about WideString -> AnsiString conversion, that I eliminate with UTF8Encode() from RTL</div><div><br></div><div>Maybe you mention file lcid_conv.pas, which contain MS Locale ID to Windows Codepage conversion and common for LHelp and chmmaker projects? Yes, chm package is a wrong place for it. I don't know, where is better place.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
So that would need to be cleaned up using FPC 3.0 string support first.<br></blockquote><div><br></div><div>What is better way to convert WideString to AnsiString without warnings?</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-">> Naming and linking problems.<br>
<br>
</span>The units have no name with chm in them, so I see no naming issues and FPC<br>
in general does not employ static libraries or so for linking purposes.<br></blockquote><div><br></div><div>And if that unit names will be same in different packages?</div><div><br></div><div>Also, is chm is really needed and used in FPC, without Lasarus?</div><div><br></div></div><div><br></div>-- <br><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>