On Sat, Jan 3, 2009 at 12:37 AM, Dmitry Timoshkov <span dir="ltr"><<a href="mailto:dmitry@codeweavers.com">dmitry@codeweavers.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
"Dylan Smith" <<a href="mailto:dylan.ah.smith@gmail.com" target="_blank">dylan.ah.smith@gmail.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
- SendMessageA(GetParent(hWnd), WM_COMMAND, (nCode<<16)|GetWindowLongW(hWnd, GWLP_ID), (LPARAM)hWnd);<br>
+ SendMessageA(GetParent(hWnd), WM_COMMAND, (nCode<<16)|(0xffff & GetWindowLongW(hWnd, GWLP_ID)), (LPARAM)hWnd);<br>
</blockquote>
<br>
MAKEWPARAM and LOWORD are supposed to be used here.</blockquote><div><br>Alright, I'll change that before resending.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Also, have you<br>
tested what happens if real window id is larger than a 16-bit value,<br>
or just a negative 16-bit one?<br>
</blockquote><div><br>I was just running the test application provided in Bug 16349 and I found while debugging that GetWindowLongW(hWnd, GWLP_ID) was returning 0x20034 which is larger than a 16-bit value. That doesn't seem like a negative 16-bit value.<br>
<br>Besides that I know very little about what this value normally would be, and don't know how I would test this in the test suite, since this doesn't always occur (e.g. I got GetWindowLongW(hWnd, GWLP_ID)=0x7d1 when running wine wordpad).<br>
</div></div>