On 04/04/2012 08:51 AM, Lubos Lunak wrote:
On Wednesday 04 of April 2012, Daniel Bankston wrote:
Lubos Lunak wrote:
It seems to me you are correct. I've added !IsEmpty() check to the if(),
which should do what the original author wanted.
I'm still a little worried about that aClip object that is being
intersected with aRegion which is now on line 673 but was originally in the
conditional on the line below. The reason I'm worried is because the aClip
object is used again on line 697. I don't know if the original author knew
that he/she was intersecting aClip with aRegion on line 673. So what I'm
asking is: Should we completely remove SetClipRegion( aClip ) on line 673?
I'm not familar with the code, but I think the current version is correct
(and it matches what the previous code did, since 2002). It looks to me that
aRegion is the clipping region for the entire operation and so clipping each
of the iterations by it too should be correct.
Ok, I'll leave it be then. Thanks for pushing and looking at it!
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice
