* Tim Moore -- Sunday 28 September 2008: > You can call it whatever you like :) The consensus is not > universally negative.
Fact is: there was no consensus at all. IIRC two people on IRC agreed with you and later Curt on the list. None of them knew that the patch was based on wrong assumptions. > I introduced this feature when I found gross performance > problems in some of our models; at the time it was radio masts. Yes. Or more precisely, the wrong assumption that the AC3D format could only store lines as separate block entries and that OSG's AC3D importer could for that reason not create efficient structures. But even later that day (IIRC) we found out that this was wrong. Jon committed fixed pylon*.ac models, and the whole reason for the patch disappeared. Now we have that abomination in CVS, and all it does is annoy people and make FlightGear unpredictable and surprising. > The model names are embedded in the scenery, so it's not > practical to change them without rebuilding the scenery, AFIAK. The change wasn't/isn't even necessary (see above). > The only problem I see is picking up files from unexpected > places, which really means the OSG sample data directory. ... and any other directory where we store models, right? > Would picking another file extension for the substitution, > such as ".fgm", satisfy you? The main goal must be: if a developer *asks* for foo.ac, give him foo.ac. Not foo.osg, not foo.fgm, not anything else. If whatever was asked for couldn't be found, warn him/her and, optionally, pick something else. m. ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Flightgear-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/flightgear-devel
