HI Ismael,

Thanks for your comments.  I have revised my patch.

On Mar 18, 2007, at 5:43 AM, Ismael wrote:

Hi Michael,
I have read your patch, and i will mainly comment your merged version of drawAlphaRect.

Is CGContextSave really unnecessary so that you removed it?

Should RefreshRect be placed before or after EndGraphics?

I re-read BeginGraphics and I see you are right - it is better to save the context.

I moved the RefreshRect after EndGraphics and it seems to work.

Try it out and tell me what you think.

http://www.mikesicotte.com/patch/salgdi_invert.cxx.diff

Mike

Reply via email to