[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