Re: MSI: Radio Buttons

2005-03-03 Thread Dmitry Timoshkov
"Ulrich Czekalla" <[EMAIL PROTECTED]> wrote: > +if (control->hwnd) > +{ > +WNDPROC oldproc = (WNDPROC) SetWindowLongPtrW(control->hwnd, > GWLP_WNDPROC, > +(LONG)MSIRadioGroup_WndProc); Then it's better to cast to LONG_PTR instead of LONG. -- Dmitry.

Re: MSI: Radio Buttons

2005-03-03 Thread Dmitry Timoshkov
"Ulrich Czekalla" <[EMAIL PROTECTED]> wrote: > +if (control->hwnd) > +{ > +WNDPROC oldproc = (WNDPROC) SetWindowLongW(control->hwnd, > GWLP_WNDPROC, > +(LONG)MSIRadioGroup_WndProc); I'd suggest to use SetWindowLongPtrW even it's just a macro in Win32 we still need to

Re: MSI: Radio Buttons

2005-03-03 Thread Alexandre Julliard
Ulrich Czekalla <[EMAIL PROTECTED]> writes: > const static WCHAR szStatic[] = { 'S','t','a','t','i','c',0 }; > +const static WCHAR szButton[] = { 'B','U','T','T','O','N', 0 }; > + > +const static WCHAR szButtonData = { 0 }; Did you really intend to use an empty string here for the button propert