<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 12, 2021 at 9:03 PM Serge Gautherie <<a href="mailto:winehq-git_serge_180711@gautherie.fr">winehq-git_serge_180711@gautherie.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello Alexandre,<br>
<br>
I am looking for a reviewer for my patches...<br>
<br>
[PATCH] wininet: Add spaces to 3 ERR() and fix 1 'escape' copypasta.<br>
<a href="https://source.winehq.org/patches/data/199448" rel="noreferrer" target="_blank">https://source.winehq.org/patches/data/199448</a><br>
<br>
[PATCH] comctl32/tests: Add more cases for ImageList_Remove().<br>
<a href="https://source.winehq.org/patches/data/199500" rel="noreferrer" target="_blank">https://source.winehq.org/patches/data/199500</a></blockquote><div><br></div><div>For the second patch, I'm not sure why you had to remove some existing tests. Also there is no need to make failure messages that explicit:</div><div><pre style="color:rgb(0,0,0);white-space:pre-wrap">"Removed nonexistent item 1 from NULL handle\n"</pre><pre style="color:rgb(0,0,0);white-space:pre-wrap">Obviously nothing was removed, because there is no list to remove from.</pre><pre style="color:rgb(0,0,0);white-space:pre-wrap"><pre style="white-space:pre-wrap">+    ok(!pImageList_Remove(himl, 3), "Removed nonexistent item 3\n");
+    ok(!pImageList_Remove(himl, -2), "Removed nonexistent item -2\n");</pre><pre style="white-space:pre-wrap">Same here, differences in return value does not mean item that does not exist was removed. Item -2 can't exist in a first place.</pre><pre style="white-space:pre-wrap"><br></pre></pre></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
Thanks.<br>
<br>
-- <br>
<br>
</blockquote></div></div>