<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/25 Henri Verbeet <span dir="ltr"><<a href="mailto:hverbeet@gmail.com" target="_blank">hverbeet@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
On 25 June 2013 11:05, Christian Costa <<a href="mailto:titan.costa@gmail.com">titan.costa@gmail.com</a>> wrote:<br>
<div class="im">> 2013/6/25 Henri Verbeet <<a href="mailto:hverbeet@gmail.com">hverbeet@gmail.com</a>><br>
>> At least try to pretend you put<br>
>> some effort into making sure the patch is as good as it can be before<br>
>> submitting it.<br>
><br>
><br>
> I could have send the patch to wine-devel first but would I get something<br>
> longer that "several issues" as feedback?<br>
</div>No, you're missing the point. Why do you expect me to even look at a<br>
patch if you clearly didn't even check for obvious typos yourself?<br>
Bottom line, if you want me to spend time reviewing a patch and giving<br>
feedback, make sure it's worth that time.<br></blockquote><div><br></div><div>Are we really discussing about a typo in the subject line? "Implement" just comes naturally from the fixme when I create an empty patch. It turned out after coding that "Fix" is more appropriate. I forgot to updated it. This will be fix.<br>
</div><div>According to the feedback I got, altough the patch is not correct, it doesn't seem it does not worth looking at. And now there is materials to make it better.<br></div></div><br></div></div>