* D Luff -- Wednesday 03 April 2002 01:27:
> [...] especially since they fix only valgrind/compiler warnings rather
> than user-visible bugs.
Err ... I'm actually also trying to fix other problems than those reported
by valgrind. :->
* D Luff -- Wednesday 03 April 2002 02:21:
> However, default.nav and default.ils both start their comments with
> // (where I copied it from !!!!!) so I don't know how they get away
> with it and default.atis doesn't.
>
> Cheers (and apologies Melchior) - Dave
Never mind.
I wasn't at my machine for almost two weeks, otherwise I would have jumped
in and tried to help finding the cause. The problem is, indeed, that
the stream handler only knows about shell style comments, while the navigation
files use C++ style comments in the first line. These were simply discarded
by ignoring the first line altogether. In the ATIS case this lead to the
loss of the first entry, so I removed that first line suppression. The best
fix would obviously have been to change the // comments to # comments. But as
the navigation databases aren't maintained by the fgfs project and are gzipped
(so that any change would have required all these huge databases to be fully
downloaded again), I chose the second best solution for atislist.cxx.
By the way, the constructors that I introduced in {ils,fix,nav,atis}list.cxx
seem cleaner, but they don't make me perfectly happy either. They are executed
in cases where they aren't needed. I'll have to think about an alternate
approach ...
m.
_______________________________________________
Flightgear-devel mailing list
[EMAIL PROTECTED]
http://mail.flightgear.org/mailman/listinfo/flightgear-devel
