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.

-Brian

-------------------------------------------------------------------------
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

Reply via email to