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

[strk]

On Tue, May 06, 2008 at 07:35:49PM +0200, Udo Giacomozzi wrote:

> BTW, zero thickness is not even a special value. It's a mathematical
> thing. Zero doesn't scale (remains always zero) and so
> (0 * some_scale / 20 < 1.0) in any case. Any stroke with width <= 1.0
> in renderer coordinate space (hah!) becomes a hairline.

Yes, that's what I tought when I did look at the AGG renderer code.
Was tempted to only rely on the math, but I saw you did set it to
1 for hairlines so didn't try to scale anyway (or would have get
a zero instead).

> You mean that SWFShape_moveTo() only supports integers so that the
> final value will always a multiply of 20 (assuming default scaling)?

Actually SWFShape_moveTo() takes floats, but SWFShape_setLine takes
an integer for the thickness, that's where you *need* Ming_setScale.

--strk;