Am Mittwoch 13 Juni 2007 00:45 schrieb alexis bory: > Hi, > > It's always a good idea looking for means to improve code, but changing > AImanager and AIAircraft had several side effects (collateral damages > ?), at least I noticed: > > - no fuel anymore in the A-10 (fixed here in my working tree).
Huh? How can a change in the AI system affect the fuel state of an aircraft model (Broken by design?) Anyway thanks for fixing, please commit. > - no AAR TACAN anymore (need to be fixed in tacan.cxx) Vivian also found this one and fixed it already. I forgot to initialize the respective property in the property tree. > Sorry Thomas but you were a bit too fast in these changes. Please, next > time, would you mind asking around if anybody sees potential problems > with such changes ? I sent my changes to a commiter (which knows the AI system) for review. I never expected so many side effects, which is a strong hint that the modules definitely needs a cleanup. For the future I wish a more open access to the version control (svn) so changes like that one can be safely developed in a separate branch and thoroughly tested before merging to HEAD. Until then we probably have to live with occasional broken HEAD revisions. Thats what it is for - development. The alternative is that I stop commiting at all. Thomas -- PhD Student, Dept. Animal Physiology, HU Berlin Tel +49 30 2093 6173, Fax +49 30 2093 6375 ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ Flightgear-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/flightgear-devel
