<div dir="ltr">On Tue, May 7, 2013 at 9:10 AM, Piotr Caban <span dir="ltr"><<a href="mailto:piotr.caban@gmail.com" target="_blank">piotr.caban@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 05/07/13 17:46, Juan Lang wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In general, I think you want to send this to wine-patches, not here.<br>
</blockquote></div>
This patch was also sent to wine-patches.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
On Mon, May 6, 2013 at 12:26 PM, Max Kellermann <<a href="mailto:max@duempel.org" target="_blank">max@duempel.org</a><br></div><div class="im">
<mailto:<a href="mailto:max@duempel.org" target="_blank">max@duempel.org</a>>> wrote:<br>
<br>
� � ---<br>
� � � dlls/msvcrt/printf.h | � �2 +-<br>
� � � 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
� � diff --git a/dlls/msvcrt/printf.h b/dlls/msvcrt/printf.h<br>
� � index cfba4b7..8b749bc 100644<br>
� � --- a/dlls/msvcrt/printf.h<br>
� � +++ b/dlls/msvcrt/printf.h<br>
� � @@ -48,7 +48,7 @@ static int FUNC_NAME(puts_clbk_str)(void *ctx, int<br>
� � len, const APICHAR *str)<br>
� � � � � � � return len;<br>
<br>
� � � � � if(out->len < len) {<br>
� � - � � � �memcpy(out->buf, str, out->len);<br>
� � + � � � �memcpy(out->buf, str, out->len*sizeof(APICHAR));<br>
� � � � � � � out->buf += out->len;<br>
<br>
<br>
If the memcpy was incorrect, the += is also incorrect. I'm not sure<br>
which is the case, but either way, your patch can't be correct as is.<br>
</div></blockquote>
out->buf is of APICHAR* type, so it's updated correctly. The patch looks good for me.<br></blockquote><div><br></div><div style>Thanks, Piotr. Max, my apologies for the sloppy review.�</div></div>--Juan</div></div>