5 Aug
2010
5 Aug
'10
7:28 a.m.
On Aug 4, 2010, at 3:30 PM, Krzysztof Kosiński wrote:
Defensive patch that might fix this. Also attached to the bug.
Regards, Krzysztof
Thanks Krzysztof.
One note, though. I see that the patch is using a "MAX" macro. Now that Inkscape is using C++, we should try to use std::max() instead. Hmm... and the placement of the "+ 1" outside of the max seems a little odd.
Oh, and now that I think of it, if it is helpful to cap the width, it might also be handy to watch the height. That's one area where GTK programs get dinged for not respecting the current dynamic 'usable' area, etc.