On 31 Dec 2008, at 11:28, Yon Uriarte wrote:

>   I havent "cvs update"d in a few days, but merging both
> the "use OpenThreads atomic" and "use display lists for
> shader trees" would benefit win32 users and general fps.
>
>  I'll see if I can merge soon and repost a patch.

I've tested both locally, and found no problems, but I'm on Mac - and  
since they're optimisations (important ones, for sure), it'd be good  
to get some assent from a Windows user (or several) that both patches  
are okay - it'd be frustrating for everyone to post a 1.9.1 that fixes  
the black boxes but introduces another regression. Most people do run  
with random objects enabled, after all.

Not that I don't think both patches are good, but there's quite  a  
clamour for a more stable version 'soon' - throwing performance  
patches in to a bug-fix release seems like asking for trouble.

(I've said such things in work situations before and been over- 
ruled... but I've also been the person pressing to include a 'safe'  
fix that promptly blew up...)

James


------------------------------------------------------------------------------
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel

Reply via email to