<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Am 12.12.2017 um 18:24 schrieb Sergey
Bodrov via Lazarus:<br>
</div>
<blockquote type="cite"
cite="mid:CAEnbvRGXsFRj4MjT-E5vjZifYmEywmzhUEGF9FA8GpTMSg1pPA@mail.gmail.com">
<div dir="auto"><br>
<div class="gmail_extra" dir="auto">
<div class="gmail_quote">On Dec 12, 2017 6:52 PM, "Werner
Pamler" <<a href="mailto:werner.pamler@freenet.de"
moz-do-not-send="true">werner.pamler@freenet.de</a>>
wrote:<br type="attribution">
<blockquote class="quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
<div class="quoted-text">
<div class="m_8989880247261128364moz-cite-prefix"><br>
</div>
</div>
If you don't mind I'll write a bug report for this part
of your code. But a question before doing so: The
original function ReadWLCEntries returns a
TChmWLCTopicArray, you put this into a var parameter and
return a boolean value. But later you ignore the
function result. Is there a special idea behind this?</div>
</blockquote>
</div>
</div>
<div dir="auto">Strange way to make changes, but I don't mind.</div>
</div>
</blockquote>
<br>
What is strange? <br>
<br>
Why I disassemble your changes? In this case, you still would not
have understood what this entire discussion is all about. Your
sources will be rejected because the essential code is buried
underneath tons of formatting changes. The developer who will review
such a bug report will not simply take your sources and copy them
over the official ones. He must understand what you changed, he must
be able to study your sources for possible side-effects which the
reporter often does not think of. With your code this is not
possible. I think on the other hand, that it would be a pity if your
work would not be considered. Therefore I joined this discussion.
Now that I understood one of the issues you are fixing and was able
to reproduce it I looked for differences between your code and the
"official" one, and found that it would be sufficient to replace
only a single procedure. With such a patch the reviewer knows what
is fixed and how it is fixed. And my report
(<a class="moz-txt-link-freetext" href="https://bugs.freepascal.org/view.php?id=32814">https://bugs.freepascal.org/view.php?id=32814</a>) gives him an easy
way to verify that a bug exists and that the patch really fixes it.<br>
<br>
Or is it strange that I write the bug report for you? I just wanted
to help you. No problem if you write the next one yourself.<br>
<br>
</body>
</html>