<div><div style="line-height: 21px;">Hi <span style="line-height: 1.5;">Dmitry,</span></div><div style="line-height: 21px;">Thank you for your advice.</div><div style="line-height: 21px;"><br></div><div style="line-height: 21px;">> I wouldn't be surprised if the control just does SendMessage with an<br>> appropriate message to itself. Is it possible to add a message test?</div><div style="line-height: 21px;">I found that the messages sent to DateTimePick32 control is different in different windows and wine,</div><div style="line-height: 21px;">for example,</div><div style="line-height: 21px;">1, In wine, the messages are:</div><div style="line-height: 21px;"><div style="line-height: 21px;">
<p class="p1">trace:datetime:DATETIME_WindowProc 104, 28, 0<span style="line-height: 1.5;"><br></span><span style="line-height: 1.5;">trace:datetime:DATETIME_WindowProc 86, 0, 1e005a<br></span><span style="line-height: 1.5;">trace:datetime:DATETIME_WindowProc 6, 0, 1e005a<br></span><span style="line-height: 1.5;">trace:datetime:DATETIME_WindowProc 8, 1e005a, 0</span></p></div></div><div style="line-height: 21px;"><br></div><div style="line-height: 21px;"><div>2, In winxp pro 32bit:</div><div><br></div><div>datetime.c:839: 0: expected: nothing - actual: 0104 wp 00000028 lp 00000000</div><div>datetime.c:839: 1: expected: nothing - actual: 0055 wp 00030196 lp 00000003</div><div>datetime.c:839: 2: expected: nothing - actual: 0129 wp 00000000 lp 00000000</div><div>datetime.c:839: 3: expected: nothing - actual: 0031 wp 00000000 lp 00000000</div><div>datetime.c:839: 4: expected: nothing - actual: 0014 wp 01010050 lp 00000000</div><div>datetime.c:839: 5: expected: nothing - actual: 007f wp 00000002 lp 00000000</div><div>datetime.c:839: 6: expected: nothing - actual: 007f wp 00000000 lp 00000000</div><div>datetime.c:839: 7: expected: nothing - actual: 007f wp 00000001 lp 00000000</div><div>datetime.c:839: 8: expected: nothing - actual: 000f wp 00000000 lp 00000000</div><div>datetime.c:839: 9: expected: nothing - actual: 0085 wp 5d040652 lp 00000000</div><div>datetime.c:839: 10: expected: nothing - actual: 000d wp 00000208 lp 0022e884</div><div>datetime.c:839: 11: expected: nothing - actual: 000d wp 00000208 lp 0022e920</div><div>datetime.c:839: 12: expected: nothing - actual: 0014 wp 01010053 lp 00000000</div><div><br></div><div><br></div><div>3, In win8 32bit:</div><div><br></div><div>datetime.c:839: 0: expected: nothing - actual: 0104 wp 00000028 lp 00000000</div><div>datetime.c:839: 1: expected: nothing - actual: 0055 wp 000300f6 lp 00000003</div><div>datetime.c:839: 2: expected: nothing - actual: 0129 wp 00000000 lp 00000000</div><div>datetime.c:839: 3: expected: nothing - actual: 0031 wp 00000000 lp 00000000</div><div>datetime.c:839: 4: expected: nothing - actual: 0014 wp 140105d7 lp 00000000</div><div>datetime.c:839: 5: expected: nothing - actual: 007f wp 00000001 lp 00000000</div><div>datetime.c:839: 6: expected: nothing - actual: 007f wp 00000002 lp 00000000</div><div>datetime.c:839: 7: expected: nothing - actual: 007f wp 00000000 lp 00000000</div><div>datetime.c:839: 8: expected: nothing - actual: 031f wp 00000001 lp 00000000</div><div>datetime.c:839: 9: expected: nothing - actual: 031f wp 00000001 lp 00000000</div><div>datetime.c:839: 10: expected: nothing - actual: 000f wp 00000000 lp 00000000</div><div><br></div><div>so I think it is not suitable write a message test.</div><div><br></div><div>> Did you miss a 'break;' or a fall through is intentiional? In the latter<br>> case please add a proper comment.</div></div><div style="line-height: 21px;"><span style="line-height: 1.5;">It is the latter case, I attached a new path, please help me review it again when you are free.</span></div><div style="line-height: 21px;"><span style="line-height: 1.5;">Thanks a lot.</span></div></div><div><br></div><div><sign signid="0"><div style="color:#909090;font-family:Arial Narrow;font-size:12px"><br><br><br><br>------------------</div><div style="font-size:14px;font-family:Verdana;color:#000;"><div>Regards,</div><div>Changhui.</div>
</div></sign></div><div> </div><div><includetail><div> </div><div> </div><div style="font:Verdana normal 14px;color:#000;"><div style="FONT-SIZE: 12px;FONT-FAMILY: Arial Narrow;padding:2px 0 2px 0;">------------------ Original ------------------</div><div style="FONT-SIZE: 12px;background:#efefef;padding:8px;"><div id="menu_sender"><b>From: </b> "Dmitry Timoshkov"<dmitry@baikal.ru>;</div><div><b>Date: </b> Fri, Feb 13, 2015 07:07 PM</div><div><b>To: </b> "Changhui Liu"<liuchanghui@linuxdeepin.com>; <wbr></div><div><b>Cc: </b> "wine-devel"<wine-devel@winehq.org>; <wbr></div><div><b>Subject: </b> Re: comctl32: SysDateTimePick32 control add Alt+down hot key likewindows</div></div><div> </div>"Changhui Liu" <liuchanghui@linuxdeepin.com> wrote:<br><br>> + case WM_SYSKEYDOWN:<br>> + if (wParam == VK_DOWN)<br>> + {<br>> + POINT pt;<br>> + RECT rect;<br>> + GetClientRect(hwnd, &rect);<br>> + pt.x = rect.right - GetSystemMetrics(SM_CXVSCROLL)/2;<br>> + pt.y = rect.top + GetSystemMetrics(SM_CYHSCROLL)/2;<br>> +<br>> + return DATETIME_LButtonDown (infoPtr, (SHORT)pt.x, (SHORT)pt.y);<br><br>I wouldn't be surprised if the control just does SendMessage with an<br>appropriate message to itself. Is it possible to add a message test?<br><br>> + }<br>> +<br><br>Did you miss a 'break;' or a fall through is intentiional? In the latter<br>case please add a proper comment.<br><br>-- <br>Dmitry.<br></div><!--<![endif]--></includetail></div>