[PATCH] winemac.drv: Support the public UTF-16 type for Unicode text.

Ken Thomases ken at codeweavers.com
Thu Aug 22 10:15:54 CDT 2013


On Aug 21, 2013, at 11:49 PM, Charles Davis wrote:

> On Aug 21, 2013, at 10:34 PM, Ken Thomases wrote:
> 
>> On Aug 21, 2013, at 9:42 PM, Charles Davis wrote:
>>> +static HANDLE import_utf16_to_unicodetext(CFDataRef data)
>>> +{
>>> +    const WCHAR *src;
>>> +    unsigned long data_len;
>>> +    unsigned long new_lines = 0;
>>> +    LPWSTR dst;
>>> +    unsigned long i, j;
>>> +    HANDLE unicode_handle = NULL;
>> 
>> This is an unnecessary initialization / dead store, which is frowned upon.
> I was only blindly copying the code that you wrote ;).

Yeah, I know where it came from, but you then changed the code enough to make the initialization unnecessary where it had been necessary in the original. :)

-Ken




More information about the wine-devel mailing list