Brian Paul wrote:
> Roland Scheidegger wrote:
>> Rune Petersen wrote:
>>> Hi,
>>>
>>> The clamped version of Point.Size, Point._Size is not updated as
>>> Point.Size changes..
>>>
>>> This has broken glSetPointSize() in R300, most other drivers use the
>>> unclamped Point.Size and are unaffected.
>>>
>>> what is needed to update Point._Size?
>> This is likely due to 8dcfcad7a2598ba835930aac8f3fd6576e464c1c, and
>> somewhat related to https://bugs.freedesktop.org/show_bug.cgi?id=9578.
>> Basically, some calculated state which was calculated instantly is now
>> defered like most other state update calculation. However, some
>> drivers still rely on some of this state in the functions directly
>> called by the functions changing that state, which will obviously no
>> longer work. I'd guess you'd either need to calculate the clamped
>> point size in the driver itself or handle point size updates somewhere
>> in the validatestate functions.
>
> I see that ctx->Point.Size is computed/updated via
> _mesa_update_state_locked(), but ctx->Line._Width is immediately updated
> in _mesa_LineWidth(). That's a bit inconsistant.
>
> It probably would be OK to compute ctx->Point._Size in _mesa_PointSize().
>
> You might check the other drivers to see how they're handling line width
> and point size.
>
looks like people are unsure of what to use.
The preferred choice is the most obvious choice: the width & size
parameter. Any chance we could make it the right choice?
Rune Petersen
The results:
------------
gamma:
LineWidth() - width (no clamping)
PointSize() - size (no clamping)
i810:
LineWidth() - Line._Width
i830+915:
LineWidth() - width (clamping!)
PointSize() - size (clamping!)
radeon:
LineWidth() - width (no clamping)
R200:
LineWidth() - Line._Width
PointSize() - Point.Size (no clamping)
R300:
LineWidth() - Line._Width
PointSize() - Point._Size
nouveau:
LineWidth() - width (no clamping)
PointSize() - size (no clamping)
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Mesa3d-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
