<div><span style="line-height: 1.5;">Dear All, </span></div><div><includetail><div>I'm sorry for my mistakes. I modified my patch, please review it again if you have time.</div><div>Thanks a lot!</div><div> </div><div style="font:Verdana normal 14px;color:#000;"><div style="FONT-SIZE: 12px;FONT-FAMILY: Arial Narrow;padding:2px 0 2px 0;">------------------ Original ------------------</div><div style="FONT-SIZE: 12px;background:#efefef;padding:8px;"><div id="menu_sender"><b>From: </b> "Sebastian Lackner"<sebastian@fds-team.de>;</div><div><b>Date: </b> Sat, Nov 22, 2014 00:19 AM</div><div><b>To: </b> "wine-devel"<wine-devel@winehq.org>; "liuchanghui"<liuchanghui@linuxdeepin.com>; <wbr></div><div></div><div><b>Subject: </b> Re: gdiplus:GdipDrawImagePointsRect has a nosie vertical line outputafter resampled</div></div><div> </div>Hi,<br><br>while looking quickly over your patch I noticed a couple of mistakes / things you should probably change:<br><br>On 21.11.2014 15:32, 刘昌辉 wrote:<br>>+    int src_img_height = src_rect.Height;<br>>+    int dest_img_width = dest_rect.X + dest_rect.Width;<br>>+    int dest_img_height = dest_rect.Height;<br>>+    BYTE* src_img_data = malloc (src_img_width*src_img_height*4);<br>>+    BYTE* dest_img_data = malloc (dest_img_width*dest_img_height*4);<br><br>You shouldn't use malloc(...) and free(...) in wine source, use HeapAlloc(...) and HeapFree(...) instead.<br><br>>+<br>>+    /*first line*/<br>>+    value = *( (DWORD*)dest_img_data + dest_rect.X) ;<br>>+    /*printf ("pixel(%d,%d) = %08x\n", dest_rect.X, 0, value );*/<br><br>I assume the printf was for debugging, remove it, or if it is really very important then use trace(...) instead.<br>Other tests use GdipBitmapGetPixel(...) for reading the color of specific pixels. If its possible to use that here too, it would look a bit better than directly accessing the memory.<br><br>>+    ok ( value > 0,<br>>+            "dest image color should same as src image color\n");<br><br>You are switching between different styles in your patch several times. Try to make the style match a bit more the sourrounding code, especially whitespace before and after functions. In this case a linebreak is also not really necessary for example, other lines are much longer.<br><br>Regards,<br>Sebastian<br></div><!--<![endif]--></includetail></div>