[PATCH] uxtheme: fixed the todo blocks in the SetWindowTheme tests. [try 2]
Reece Dunn
msclrhd at googlemail.com
Tue Oct 28 11:47:26 CDT 2008
2008/10/28 James Hawkins <truiken at gmail.com>:
> On Tue, Oct 28, 2008 at 10:17 AM, Rob Shearman <robertshearman at gmail.com> wrote:
>> 2008/10/28 James Hawkins <truiken at gmail.com>:
>>> This code is screaming for a goto.
>>
>> I disagree.
>
> Yes, it was late and my mind stopped after the next thought. There's
> nothing to clean up, so simply returning the failed result is optimal.
Ok, so I will use this version of the code but will hunt down where
the hwnd is being used, as per Alexandre's comments. I'll send a
revised patch later on today or tomorrow.
Thanks,
- Reece
More information about the wine-devel
mailing list