Step #1 of some code refactoring and cleanups.  The nav radio code was
written very early in the project and has grown and evolved and been added
onto many times.  It is long overdue for a code cleanup/reorg pass.

Remove a debugging cout.

Detect the hostname as early as possible to prevent a segmentation fault (and an unknown exception in the main loop situation.

no more "Aborting ...\nPossible cause: Success"

Revert to the previous version, this is just a glimps of the future.

Only test for --fg-root and --aircraft in ~/.fgfsrc et all if it wasn't specified at the command line.

Add an option to enable or disable the save-on-exit option. Add a season commandline option.

Put the user configuration file sequence into it's own function. This will slightly tidy up the code but more importantly will make it consistant across the program.

Fix a typo and an unitialized variable

Get the environment variables HOME and HOSTNAME only once.

Vassilii Khachaturov:

clean up some build warnings caught with gcc-4.0.

Olaf Flebbe:

The prototype of update_metar_properties does not match overridden func.
trafficmgr: iterators below begin() and after end().
tower.cxx : iterator incrementing beyond end().

Vassilii Khachaturov:

clean up some build warnings caught with gcc-4.0.

MSVC fix.

Stefan Seifert: implement persistent dialog options, so changes in rendering, static LOD and sound settings dialogs are made permanent.

add _setlistener() function, so that slow property polling loops can be
replaced with efficient listener callbacks. One use is the new FPS display.
This is reviewed and OK'ed by Andy, relatively trivial and separated from
the rest of Nasal, so problems are quite unlikely and confined to users of
this function.

The callback is executed whenever the property is written to -- even if
the value didn't change. The triggering Node is available via cmdarg().

Examples:  _setlistener("/sim/crashed", func {print("haha!")});
           _setlistener("/foo/bar", func { print(cmdarg().getPath() ~ " changed")})

Remove the old hardcode fps display.  This is now replaced with a nasal/gui
display that can be dragged around the screen to an unobtrusive location.

     remove a few unused vars
menubar: cosmetics of the day

don't check for subsequent indices (n), but for child positions; This way
it's also possible to enable/disable menu/item entries with higher numbers.
This can be useful for adding entries from other config files (aircraft
specific or local). I'd say aircraft files can use indices starting with
[100] and local files starting with [1000]. Such high number will never
collide with an entry in menubar.xml, even if entries are added/removed
there.

Mathias Fröhlich's code, Curt Olson's comments:

Display the ground intersection point on mouse click (if click not consumed
by the gui or the panel.)  This should eventually get stuffed into the
property tree.

Mathias Fröhlich:

Computes the pick direction and starting point.  This is code that translates
a screen coordinate into a vector in the FlightGear world (between the eye and
the on-screen coordinate.)  Armed with this vector in FG world coordinates,
you could call a scenery intersection routine and lookup the lon/lat/elev of
the point in the world that was clicked on.

Vassilii Khachaturov:

* in some cases more specific sg exception types were used in place
  of the more generic one, e.g., sg_io_exception instead of sg_exception
  when the context of the error was an IO error
* in some cases, the error message was made more specific
* minor style fix for exception rethrowing --- using throw; whenever
  a re-throw is made; sometimes optimizing away the exception symbol name
  in the catch handler at all
* more specific catch handlers added in some places -- e.g.,
  an sg_io_exception caught ahead of sg_exception

The most important part is that it fixes possible
memory corruption with that iterator decrement past the erase call.

Additional checks to io arguments. This prevents fg from crashing on invalid
arguments.

MSVC8 fixes.

final cosmetics (s/_entries/_objects/ and comments)

- fix typos
- fix indentation (there were 2, 3, 4, 7(!) space indents as well as tab
  indents, all mixed together)
- no code changes, except one "if (foo) {}" changed to "if (!foo) return; ..."

Make UFO honor elevator trim so it will respond to autopilot.  This allows
you to do altitude hold mode and fly automated ILS approaches.

yet more cleanup; clearer warning messages

removal of yet more stuff, left over from development; further simplification;
requires to remove some constness, though (we are adding listeners!);
If I continue like that, nothing will be left.  :-)

Vivian: downgrade log levels from ALERT to INFO, tidy up the code.

remove two checks that were only useful during development; fix typo

Add files required by autotools.

ignore some autogenerated files.

Mathias: silence some valgrind warnings so that you can concentrate better on the real problems.

Give Vivian some more credit.

allow disabling/enabling of menu entries via "enabled" property;
Unfortunately, we don't have an easy way to access the puObjects
only by knowing the respective XML property node, because the
menu structure was built by plib from string lists. That's why
we walk the puMenuBar tree and store {property node}->{puObject*}
pairs in a map. With this infrastructure in place we can now
easily enable/disable entries, but we can also make other changes
to menu buttons as we see need. The structure of puMenuBar is
described in the pui documentation, so it's less of a hack than
it looks.  :-)

Mathias FROEHLICH:

remove a hack and do it properly: if the requested elevation is for some
reason below the surface and the intersection test fails (as it appears
to be the case in EGLL), try again from 10000m ASL

mf: typos

globals.cxx -- FGGlobals::set_fg_scenery():
    Insert empty string as marker between FG_SCENERY path elements.
    FG_SCENERY=A:B expands to [A/Terrain, A/Objects, "", B/Terrain, B/Objects, ""]
    (assuming that both A/ and B/ have Terrain/ and Objects/ subdirs).

tileentry.cxx -- FGTileEntry::load():
    Check all tile dirs in FG_SCENERY from left to right: add all objects
    to the scenery until a terrain tile was found: In this case read the
    rest of that group (i.e. the Objects/ twin dir) and then stop scanning.

    Better structuring of log messages & fix warnings.

Alex Romosan:  Pass strings by reference not value

Alex Romosan: Make GpsPage::GetId() pure virtual, and pass strings by reference not value

FGTileEntry::load(): store objects in a vector, then generate terrain or
sea tile, and finally process the objects. This guarantees that all objects
are placed relative to a valid tile center, rather than to the origin (0/0/0).
This is important for objects in sea tiles, and allows to display objects
of *.stg files that came sooner in FG_SCENERY.

Mathias FROEHLICH: fix strange gcc 4.0.* strict aliasing problem

returning addresses of auto vars is *dangerous* (ask Vasilii! :-)
This was the reason why some people (especially SuSE10.0/gcc 4.0.2
users) couldn't see others in MP. I don't even know why I'm committing
that. It's less important than taxi lights ...

Swap signs of the twist and incidence values to match the
documentation and author expectations.  Positive just wants to mean
"up", no matter what the axis conventions have to say.

Ignore generated files

Add support for a special-instrument xml attribute, which triggers loading of a know c++ hard-coded instrument.  This has some ugliness that will need removing at some point.  In particular, panel reload is broken for panels that include a special-instrument, since this triggers subsystem reload which is a fatal error.  Note that this *only* affects panels that use a special-instrument - all other panels are unaffected

Build the KLN89 stuff...

Track added files

Code to extend FGs current GPS functionality to allow the simulation of a specific approach capable GPS.  Eventually this file should be broken up and the code removed to other files, such as FGs current gps files

2D polygon drawing routines for a GPS instrument

kln89 GPS unit simulation

kln89 user interface logical pages

kln89 symbol table

Allow the nav radios to be slaved to a gps unit

Add a method to return a const pointer to the raw airport data, plus a couple of whitespace cleanups

General tidy up - add an extra ctor method, remove surplus defined constants, remove cout and iostream

Vivian MEAZZA:

"Remove redundant code, tidy up and improve output properties"
make elevator work

Correct conditional so that FlightGear will compile without
special-purpose FDMs.

Add a lower-bound type navaid lookup, and the ability to specify navaid type in the find nearest lookup, for the GPS code

ban not so informative message

Remove nav.hxx, which has been superceded by navrecord.hxx and is no longer used except by some old non-working test code

Add a lower-bound search function for fixes for GPS units with next-match database search capabilities

Vivian MEAZZA: use coords of tuned in carrier, not of the first one

Use the prefered GLUT  game_mode_string format and add the posibility to match the screen refresh rate and the framerate.

Mathias Fröhlich:

While digging in flightgears lost memory, I found that endless loop.

Fix property typo

cosmetics -- NO CODE CHANGE!

remove trailing spaces
fix very liberal indentation (logical block not indented at all, 5 spaces, ...)
removed commented out code

Vivian MEAZZA:

bugfix: don't always refer to carrier #0

Josh wanted a compression-m value exported for gear, so he can animate
them more accurately.

of course I should have known that it would only work in tests *before*
committing, and would instantly break afterwards. Causes endless loops.
Re-uglify. Sorry for the noise.  :-/

remove a few if's to make the ugly fix a little less ugly; they only save
a few CPU cycles but are really redundant

AJ found a typo

MIPSpro doesn't like main() to be redefined.

IRIX fix.

A long, long time ago, a bug was inadvertently introduced into the threaded
metar fetcher.  Effectively this caused the metar thread and the main
thread to both attempt to fetch weather data.  This could lead to long pauses
when the main thread decided to fetch the weather, and introduced a race
condition that could cause a segfault/crash.

Investigating this issue, I discovered that even longer ago, someone confused
#defines and #ifdef symbols with C/C++ variables.  If I #define XYZ 0 it is
defined so #ifdef XYZ is true, not false like a variable.  Our thread
detection made this mistake and there were follow up patches to work around
it.

So I fixed the configure script (ahhh, reading the autoconf manual is highly
recommended excercise for people editing the configure.ac file.)  I also
discovered that we were hardwiring with_threads=yes with no way via configure
options to disable threads from the build so I fixed that.

Then I patched up the #ifdef's scattered through the code to match the
configure script changes, oh and by the way, I stumbled upon a past typo
that led to the race condition in the metar fetching thread and fixed that.

"info -f autoconf" is a beautiful thing.  Fixed the configure script so
we can actually configure the project without threads.  Previously threads
were always forced on no matter what configuration options were passed in.

Final updates for v0.9.9

Clock also able to report local time.

737 depends on T38 instruments.

Don't exclude 'data/Data' directory.

Expose local timezone offset in the property tree.

David Culp:

Here's a fix for a bug I introduced when I updated the AIStorm with
turbulence.  The change I made to the FGAIEntity struct was overriding the
thermal <strength-fps> data.

Updates from Fred Bouvier:

You'll find attached the modified fgadmin that takes care of
'Objects' and 'Terrain' if they exist. I also implemented the
'Check all' / 'Check none' feature you suggested several months
ago.

Update 'magic' for more consistant cross platform padding.

number one reason for lots of stale METAR reports is wrongly set system time

Prevent download of thousands of METAR reports if all are for some reason
too old (wrong system time or broken proxy): stops after 10 stale reports
were fetched in a row. This should simply stop further fetching, but due
to a bug in the threading system(?) it does currently lead to abortion,
just like any other exception in the fetcher.

This section was accidentally committed togetehr with another patch, it turned out it needed a fix since env was already declared.

Use the new clouds3d-enable property instead.

Use the new clouds3d-enable property instead.

Start the HTML documentation in the default browser on Mac OS X

Cygwin fix.

Sync with SimGear.

"nine" (via IRC):

don't crash if gui subsystem isn't available yet

This is an ugly fix for an ugly problem. And no, the two uglies don't cancel
each other out. The problem is this: if we press, for example, "Ctrl-a", but
release the "Ctrl" modifier button *before* the "a" button (which nobody does
intentionally, but which happens all the time), then we don't get the RELEASE
signal on "Ctrl-a" (keycode 1), but on the "a" (79). But "a" hasn't been
pressed, so the signal is dropped. And who releases "Ctrl-a"? Nobody!
So the next PRESSED signal for "Ctrl-a" is ignored, too. It is still
"pressed" after all, isn't it? That's the reason for the occasional
non-functioning of keys.

Due to the nearing 0.9.9 release, I only commit a crude last-minute fix.
It's not as intrusive as it looks, and shouldn't be "dangerous" at all.
It only makes sure that when we get an unexpected RELEASE for one letter
key ("a") that the two twins "A" and "Ctrl-A" are released if they are
still in "pressed" state.

The proper fix will be to let fg_os{,_sdl}.cxx always report presses on the
same key ("a", "Shift-a", "Ctrl-a", "Alt-a", and other combinations of
modifiers) as the *same* key (97), only with modifiers appropriately set.

Make use of the new sound_sample function arrangement.

Downgrade some debugging output to the SG_DEBUG log level.

Oops again -- forgot that the solver needs fuel too. :)

try to fix faces that come after (currently) unfixable faces

Added a Thanks entry.