<div dir="ltr"><div>It looks like your way indeed solves the problem as I am no longer able to observe the issue I saw earlier and I simply have not fully understood the way you rewrote the code.</div><div>Thanks for cleaning it up then.</div><div>Michał.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 4 July 2018 at 09:41, Alexandre Julliard <span dir="ltr"><<a href="mailto:julliard@winehq.org" target="_blank">julliard@winehq.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Michał Janiszewski <<a href="mailto:janisozaur@gmail.com">janisozaur@gmail.com</a>> writes:<br>
<br>
> Hi,<br>
> I would like to complain the commit mentioned by the bot is not the commit I sent out.<br>
> My patch guarded from access to unallocated memory, while the commit mentioned and attributed to me tests if there are any bits to set/clear and proceeds to use the memory it does not own anyway.<br>
> Please revert a1ed500836806ef46fc3eafa7ce5f8<wbr>621938cbc1, I shall come up with a test for the scenario I'm talking about.<br>
<br>
</span>I believe it does the same thing in a simpler way, but if you have a<br>
test demonstrating that it's still broken, please submit it and I'll fix<br>
the code.<br>
<span class="HOEnZb"><font color="#888888"><br>
-- <br>
Alexandre Julliard<br>
<a href="mailto:julliard@winehq.org">julliard@winehq.org</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Michal Janiszewski</div>
</div>