<div dir="ltr">Hi Alexander<div><br></div><div><span style="color:rgb(80,0,80)">On Tue, May 20, 2014 at 1:39 PM, Alexander Almaleh <</span><a href="mailto:sashoalm@gmail.com" target="_blank">sashoalm@gmail.com</a><span style="color:rgb(80,0,80)">> wrote:</span><br style="color:rgb(80,0,80)">
<span style="color:rgb(80,0,80)">> I had tried to submit a patch for that 5 years ago, but couldn't figure out</span><br style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">> how to make a patch/test it, and eventually gave up and forgot about it. But</span><br style="color:rgb(80,0,80)">
<span style="color:rgb(80,0,80)">> recently I remembered about it, and now I want to try again.</span><br></div><div><br></div><div>The easiest way to get your patches accepted, in my experience, is to write an automated test. That way it is easier for everyone to verify that your patch is doing the right thing. We will be able to run it on the Wine Testbot and see it working.</div>
<div><br></div><div>It might be difficult in your case though. Wine does usually not include big test files so you would probably have to write some code that draws a test image. The test image needs to be easy to verify in code if it has been transformed correctly. You might be able to draw a filled rectangle in one hdc and a filled parallelogram in another. Transform the filled rectangle using PlgBlt and then compare it to the parallelogram. I have not thought it all the way through so it might not work :-)</div>
<div><br></div><div>It seems like there are no automated bitblt tests in the wine source tree. The first step would be to add a file bitblt.c in /dlls/gdi32/tests/ and then add it to the Makefile.in file in that directory. My first commit to Wine [1] added a new test file brsfolder.c to shell32.</div>
<div><br></div><div>You might be able to get your patch in without the automated test, but I think you need to explain clearly what was wrong with the old code and what you are doing instead. A few more simplified examples that test other cases in your bug report might also help.</div>
<div><br></div><div>Best of luck,</div><div>Michael Mc Donnell</div><div><br></div><div>[1] <a href="http://source.winehq.org/git/wine.git/?a=commit;h=5c715b46f4312a2e76aa14499e09edad2af42e20">http://source.winehq.org/git/wine.git/?a=commit;h=5c715b46f4312a2e76aa14499e09edad2af42e20</a></div>
</div>