Re: [Gnash-commit] gnash ChangeLog backend/render_handler_agg.cpp ...

[Udo Giacomozzi]

Hello Bastiaan,

Tuesday, March 6, 2007, 6:40:05 PM, you wrote:
BJ> -           unsigned int width = region.width()+1;

I patched this some while ago because region.width() is *not* enough!
region.width() is implemented like:

  region.width() := getMaxX() - getMinX()

In integer domain this is off by one since width==1 when
getMaxX()==getMinX() while it should be 1, otherwise it would be a
NULL region.

So, region.width()+1 *is* correct.

There must be another reason for the segfault. I think I know the
reason and will investigate this further tomorrow.

Udo