Jon Griffiths wrote:
Hi,
There is no point in sending WM_QUERYUISTATE. I believe it is only really used for the toolbar control, but the native comctl32 has a common framework that means it is sent for all controls.
Sure, it helps when diffing the traces of native and builtin though.
By sending this and creating the scroll window later I get the exact
same traces and window handles as native. That makes it a lot easier
to find differences later on. Its also concievable (though unlikely)
that the parent may use the message to determine the control is being
created.
Ok, there is no harm in having it in. Just for the record it is used to emulate the keyboard cues in Windows 2000+ menus.
I haven't looked at the rest of the changes yet and I won't have a chance until Sunday, but the changelog looks sane enough.
Is it possible to split the patch up to make it more easily
reviewable?
Possible, but a lot of work. the change to pass infoPtr around hits
every function definition and function call, and the change to use an
accessor hits every access to the infoPtr->items array. I'm happy if
Alexandre doesn't commit until you've had a chance to review it
though.
Ok. Deal.
Rob
