James Turner wrote: > On 31 Dec 2008, at 17:38, Martin Spott wrote: > > Now that the release is out, my I add a little reminder to this patch > > which was meant to add some 'cleanup' to SimGear's use of threading > > libraries: > > > > http://blaniel.free.fr/pub/flightgear/patches/ot_simgear.patch
> That patch is entirely reasonable, but Yon's is better - rather than > making SGAtomic use some OpenThreads primitives, it just delegates the > entire atomic int handling (policy and platform decisions included!) > to OpenThread's own atomic class. I was aware that a part of Daniel's changes has been superseded by Yon's patch. Nevertheless I understand that adding the 'remaining' parts of Daniel's patch make sense for the sake of unification - similar Pthread-to-OpenThreads changes had been committed to FlightGear earlier this year, see: http://mapserver.flightgear.org/git/gitweb.pl?p=flightgear;a=commitdiff;h=655cbcbce39e4420a511749a6b2cad889c088f08;hp=cd7ee03fbf780779ae15d0372381658ac4c29402 Do/merge/leave whatever/however you like, I just wanted to make sure Daniel's changes don't get lost. Cheers, Martin. -- Unix _IS_ user friendly - it's just selective about who its friends are ! -------------------------------------------------------------------------- ------------------------------------------------------------------------------ _______________________________________________ Flightgear-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/flightgear-devel
