sglog().would_log()

[Philosopher]

I'm working a bit on reworking the logging exposed to Nasal, since it hasn't been updated in years (besides the addition of the never-used logprint() by James, the old printlog() only goes to the $FG_HOME/fgfs.log file if it would also go to the console and is implemented in globals.nas - aka can't be depended on in $FG_ROOT/Nasal/*.nas). This is going to eventually help with the Nasal REPL (aka to capture output), but I discovered something really weird: sglog().would_log() returns true for any priority >= INFO! This almost seems like a bug to me, but OTOH it is required by the SG_LOG() macro to ensure that the logging file is not ignored in the if statement (since SG_LOG() uses would_log()).

First of all, I don't see the reason for it; I think it would be better to remove the would_log() from SG_LOG() and make the former return if it will actually get printed to the console. Second of all, it makes me have to check the priority manually, and it would be nice to just have the would_log() API for that. Third, I'm not sure if we're hard-coding the minimum log level for $FG_HOME/fgfs.log? IMO that would not be good, although it is an acceptable default if we were to choose anything. The problem is that it's pretty complicated code, and I don't want to have to chase any more sources right now. Any comments from others? I don't want to go running around and changing this without input .

See this comment:
https://gitorious.org/fg/philosophers-f ... s.cxx#L463
And this particular line of would_log():
https://gitorious.org/fg/philosophers-s ... m.cxx#L268