Re: Suboptimal and suspicious patterns using COM macros and functions

On 11.08.2009 11:13, Dmitry wrote:
> steveking wrote:
>
>>> 2. In src\TortoiseShell\​ItemIDList.cpp near the end of>>ItemIDList::toString() method there's a condition:
>>> if (szDisplayName == NULL)
>>> {
>>> CoTaskMemFree(szDisplayName);
>>> return ret; //to avoid a crash!
>>> }
>>> The call to CoTaskMemFree() will have no effect if szDisplayName>>is NULL - this call only pollutes the code.
>
>> here, the if() clause is not because of CoTaskMemFree() but because the
>> next line
>> ret = szDisplayName;
>> would crash if szDisplayName is NULL (you can't assign NULL to an
>> std::string).
>
> Yeap, I understand that. This part is necessary. I wanted to say that the call to CoTaskMemFree() is redundant.

Right, that call really is redundant.
Removed in r16903.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2382792
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].