Melchior Franz writes: > Update of /var/cvs/FlightGear-0.9/FlightGear/src/Instrumentation > In directory baron:/tmp/cvs-serv8740 > > Modified Files: > wxradar.cxx wxradar.hxx > Log Message: > make headers include headers they depend on, don't rely on the c(xx) > file to do that. (This is a requirement for header precompiling.)
i don't understand what you mean by precompiling but making the header file include unnecessary headers is wrong. i looked at this particular case, and it's okay to forward declare a class and include the header file in the c++ source file. you actually want to do this as it minimizes the amount of recompilation in case the base header files changes. i suspect all your changes are unnecessary (this one really looks like it) so i suggest you revert them. --alex-- -- | I believe the moment is at hand when, by a paranoiac and active | | advance of the mind, it will be possible (simultaneously with | | automatism and other passive states) to systematize confusion | | and thus to help to discredit completely the world of reality. | ------------------------------------------------------- This SF.Net email is sponsored by xPML, a groundbreaking scripting language that extends applications into web and mobile media. Attend the live webcast and join the prime developer group breaking into this new coding territory! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642 _______________________________________________ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel
