<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=ISO-8859-1"
 http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
<div class="moz-text-flowed"
 style="font-family: -moz-fixed; font-size: 13px;" lang="x-western">
<blockquote type="cite" style="color: rgb(0, 0, 0);">Hi Amine, </blockquote>
Hi Jacek,
<br>
<br>
<blockquote type="cite" style="color: rgb(0, 0, 0);"><br>
  <blockquote type="cite" style="color: rgb(0, 0, 0);">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; r =
register_clsid(&amp;CLSID_StdHlink);
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (SUCCEEDED(r))
    <br>
&nbsp;-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; r = register_clsid(&amp;CLSID_StdHlinkBrowseContext);
    <br>
&nbsp;+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; register_clsid(&amp;CLSID_StdHlinkBrowseContext);
    <br>
    <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return S_OK;
    <br>
  </blockquote>
  <br>
  <br>
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.
  <br>
</blockquote>
<br>
Hmm.. I missed this point, sorry.
<br>
<br>
Thanks,
<br>
Amine.
<br>
</div>
</body>
</html>