hlink/hlink_main.c : Remove an unneeded assignment.

[Amine Khaldi]

> Hi Amine, 
Hi Jacek,

>
>>       r = register_clsid(&CLSID_StdHlink);
>>       if (SUCCEEDED(r))
>>  -        r = register_clsid(&CLSID_StdHlinkBrowseContext);
>>  +        register_clsid(&CLSID_StdHlinkBrowseContext);
>>
>>       return S_OK;
>
>
> In this case the correct fix is to return r. Please be more careful 
> then doing janitorial work. There are more places in your patches that 
> seem questionable. You should make sure that we want to ignore an 
> error instead of handling it (and usually we don't) before deleting 
> such assignments. Otherwise it's just hiding bugs.

Hmm.. I missed this point, sorry.

Thanks,
Amine.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20091216/870f32a1/attachment-0001.htm>