* src/ATC/atis.cxx
* src/ATC/atislist.cxx

Patch from Melchior Franz:

- merged the _MWERKS_ & generic #ifdef branch, only the loop head needs
  different treatment
- commented out a "n >> skipeol" that is used in navlist.cxx to gobble
  the starting C++-style comment from default.nav.gz; atis inherited
  that, but there is no such comment in default.atis.gz, so the first
  data entry got thrown away
- commented out min/max frequency exploration (-> only for debugging)
- enhanced the constructor; the operator>> wouldn't have initialized all
  variables in case of a broken default.atis.gz entry, so we would have
  got a mixture of the broken one and the previous one;
- move the automatic FGATIS variable into the loop, so that it gets
  cleanly constructed for every database entry.
- don't access a.type before it is initialized
- updated the commented out debug output statements; they were still
  designed for the FGNav class ...

Patch from Melchior Franz:

- merged the _MWERKS_ and the generic branch, only the loop head needs
  different treatment
- created a constructor; the operator>> wouldn't have initialized all
  variables in case of a broken default.fix.gz entry, so we would have
  got a mixture of the broken one and the previous one; (valgrind
  complained ...)
- move the automatic FGFix variable into the loop, so that the gets
  cleanly constructed for every database entry.
- don't access fix.type before it is initialized
- updated the commented out debug output statements (they were copied
  over from navlist.cxx but never adapted)

Patch from Melchior Franz:

radiostack.cxx:703 says
      bool light_on = ( outer_blink || middle_blink || inner_blink );
but none of the flags has ever been initialized when this line is first
executed.

Fixed buffer size again.

Patches from Jim Wilson:

This patch creates a seperate scene graph for the cockpit.  The near plane is
only moved up when in the interior (pilot) view.  This is because with
rounding (I presume) it the visible ground is a bit up higher than it is with
the older nearplane setting.  Not much, but it is enough to bury the wheels.
I suspected this might be true but spliting to two sg's confirms it.  If
necessary we can adjust the model up a bit when in interior view.  This might
be good so we can set the near plane even closer when in the cockpit (its
still at 0.2m).

In general this looks a lot better on my Voodoo with this patch.  No
perceptibel change in frame rate on my system.  In terms of future plans I'd
see the sense in making the model plug into either scene.  This will be
necessary when we have multiple model instances in the frame.

Make sure that saved mouse position is reset when mouse is centred on
mode change.

Mouse changes suggested by Norm Vine.  Cursor always centres on mode
changes, and wrap-around is smoother.

Created a new /devices property subtree to hold input device status
(currently just the mouse, but later the joystick and keyboard as
well).  Publish mouse button status to the property tree.

Fixed bug reported by Norman Vine:

Mouse was not working properly with PUI dialogs that use sliders.  It
turned out that the up/down sense was reversed.

Patch from Norm Vine:

patch to enpower HUD to  display elevator trimval

Changes corresponding to recent property manager changes
Remove bogus warning messages
Documentation changes
Minor speedups

Patch from Melchior Franz:

The matrix doesn't define some cells, which are actually used
in the multiplication. That makes the result unpredictable.
I have no idea if 0.0 is the correct value for these, but
garbage is hardly the correct value either. Should some of them
be set to 1.0?

Viewer patches from Jim Wilson:

Think my brain is getting clogged with matrices :-).  Well I've got the funky
orientation offset bug out of the model code.  In the process the model.cxx
got optimized a bit.  At some point we'll need to liberate model.cxx from the
viewer class, but it is no longer hard coded to access the  "pilot view" to
get it's data. Instead it uses whatever the "current" view happens to be.  I
may try and do that final bit of having models rotate independant of the view
tomorrow night, or start right in on the viewmgr and get a tower view up and
running.  You guys have any preference?  My brother's family is coming to
visit for a few days so what I don't get done tomorrow night probably won't
get done until after the weekend.

Patch from Frederic Bouvier:

The compiler complains about too long names for instanciated templates
that result in name truncation. There are warnings but finally it ends
with a fatal errors. I found that ignoring the warning cure the
problem.

Fixed a bug report from Frederic Bouvier:

A sloppy coder already defined min and max as macros with 2 arguments

Fixed a bug report from Frederic Bouvier:

as a follow-up of my previous message, I found that in panel.cxx, function
const char *FGTextLayer::Chunk::getValue () const, there is the use of a
member variable _buf that seems to be uninitialized.

Removed outdated reference to fgSimTime.

Patch from Melchior Franz:

When the loop starts, n.type is still undefined, so the while statement
depends on unitialized garbage. The input operator cares for the [End]
bracket anyway (returns if the first character is a '['). So it is safe
to check for it after reading the line and break if necessary.

Patch from Melchior Franz:

Err ... 8 bytes isn't much for such a long string.
Makes a nice segfault.  ;-)

Added support for rotational rates and body access (pilot relative)
accelerations.

Added fields ... this is a work in progress.

Renamed README.xml to README.xmlsyntax, so that it wouldn't look like
an XML file itself.

More cleanups.  Removed the pui-* commands, and added a pass-through
property to indicate when mouse events should be offset to PUI and the
panel first.

Updated Melchior's entry with other stuff he's done.

First draft of a quick overview of XML syntax, not specific to
property lists.

Added Melchior (but I cannot remember what else he has contributed).

Patches from Melchior Franz:

The first hunk might not be necessary, but the light_coverage property
was the only one that wasn't explicitly defaulted, which is unfair.  ;-)
   The second hunk adds the missing initialization to the init routine.
This is necessary, because (unlike the material entries from material.xml)
the generated light entries don't get the light coverage set. Yet
obj.cxx:795 reads this information out fot every leaf, although not needed
in the case of lights. Avoiding this isn't worth the trouble.
   The last hunk sets the missing normal_index. The POINTS branch in
gen_leaf was always called with this int_list empty, which made the normals
handling use data garbage.

More work on the configurable mouse.  Mouse motion is now configurable
as well as mouse clicks.  You must configure with --with-new-mouse to
get this.

Setting offset or tilt always sets goal-offset or goal-tilt as well.

Commented out cout statements that were not in the std namespace.

I missed committing this one.

Oops, in the latest property manager rewrite, a typo crept in on
somewhat of a global scale.  This commit should fix all these up.

Ensure that all entries in the FGAircraftModel::Animation structure
are initialized (reported by Melchior Franz).

Patch from Melchior Franz:

It seems that the airport database was changed some day and the End?Flags
changed from floats to strings. The database definition, though, was not
adapted and still created number entries. Reading out these flags led to
access to memory, that was never initialized. While it didn't cause crashes
during normal use, it actually caused one when I ran fgfs in ddd. Seems,
that the concerned memory region wasn't zeroed out then and hence uncovered
the bug.

Of course, the runways.mk4 database has to be re-created with the new
definitions.

Patch from Erik Hofman:

This patch gets F8 working again.

Minor patch from David Megginson:

Fix FGViewer::update so that pitch offset and goal pitch offset work
together nicely (the offset was snapping to 90/-90 when only one of
the two was changed).

Viewer improvements from Jim Wilson:

These files get the 3d cockpit working and fix a few issues in the viewer
code.
XYZ offsets are now defined as follows: X -left/right+ (along wing axis), Y
-up/down+ perpendicular to the aircraft, Z is -in/out+ the aircraft's body axis.
I've also done some cleaning up of unused and mostly unusable interfaces,
added commentary to the *.hxx, combined together some duplicate code and
eliminated a couple unecessary operations.  I also moved what was left of the
"protected" zone to "private" since we aren't subclassing this anymore.

Fixed a small typo.

Support for assigning custom mouse cursors to mouse modes.

First steps towards configurable mouse input.  Soon, this new code
(mainly in src/Input/input.cxx) will make src/GUI/mouse.cxx obsolete
and bring the mouse into the same input system as the joystick and
keyboard.  This is just preliminary work allowing, covering mouse
clicks (no motion yet), and it actually crashes on a middle or right
click.

The new mouse support is disabled by default until it become stable;
to try it out, you need to configure --with-new-mouse.

Updates.  Includes property tree additions and fixes.  Also Jon's first cut at
multi-fdm capability.  Thanks also to Erik for pointing out a couple of typos.

Fix for flaps problem.

Removed unused function fgFindNode (the version used now is in model.cxx).

Removed unused files.

Initial revision of README describing the contents of the src/Main
directory.

Viewer update from Jim Wilson:

This patch renames the "goal" orientation offsets inputs/outputs to
conform with the naming of the new interface.

Minor updates to my own entry.

Added Erik Hofman's sound contributions.

Added Jim Wilson.

Removed obsolete files.

more ( string == "" ) to ( string.empty() ) conversions.

Found by inspection a bug in the generation of orientation matrices
for fuselage Surface objects.  If the fuselage wasn't aligned perpendicular
to the Y axis, the matrix wouldn't be orthonormal.  Since all of, perhaps,
three aircraft have ever been built this way, it's doubtful I would have
found this as a bug report. :)

Fix for the flaps problem.

Irix fix.

Model-loading patch from Dave Luff.

Use !strcmp for all property name comparisons.

Use !strcmp for all property string-value comparisons.

Converted if ( string == "" ) constructs to if ( string.empty() )
Fixed a warning in soundmgr.cxx.

Major viewer-code overhaul from Jim Wilson:

Description:

This update includes the new viewer interface as proposed by David M. and
a first pass at cleaning up the viewer/view manager code by Jim W.

Note that I have dropped Main/viewer_lookat.?xx and Main/viewer_rph.?xx and
modified the Makefile.am accordingly.

Detail of work:

Overall:
The code reads a little easier.  There are still some unnecessary bits in
there and I'd like to supplement the comments in the viewer.hxx with a tiny
bit on each interface group and what the groupings mean (similar but briefer
than what you emailed me the other day).  I tried not to mess up the style,
but there is an occasional inconsistency.  In general I wouldn't call it done
(especially since there's no tower yet! :)), but I'd like to get this out
there so others can comment, and test.

In Viewer:
The interface as you suggested has been implemented.  Basically everything
seems to work as it did visually.  There is no difference that I can see in
performance, although some things might be a tiny bit faster.

I've merged the lookat and rph (pilot view) code into the recalc for the
viewer.  There is still some redundancy between the two, but a lot has been
removed.  In some cases I've taken some code that we'd likely want to inline
anyway and left it in there in duplicate.  You'll see that the code for both
looks a little cleaner.  I need to take a closer look at the rotations in
particular.  I've cleaned up a little there, but I suspect more can be done
to streamline this.

The external declaration to the Quat_mat in mouse.cxx has been removed.  IMHO
the quat doesn't serve any intrinsic purpose in mouse.cxx, but I'm not about
to rip it out.  It would seem that there more conventional ways to get
spherical data that are just as fast.  In any case all the viewer was pulling
from the quat matrix was the pitch value so I modified mouse.cxx to output to
our pitchOffset input and that works fine.

I've changed the native values to degrees from radians where appropriate.
This required a conversion from degrees to radians in a couple modules that
access the interface.  Perhaps we should add interface calls that do the
conversion,  e.g. a getHeadingOffset_rad() to go along with the
getHeadingOffset_deg().

On the view_offset (now headingOffset) thing there are two entry points
because of the ability to instantly switch views or to scroll to a new view
angle (by hitting the numeric keys for example).   This leaves an anomaly in
the interface which should be resolved by adding "goal" settings to the
interface, e.g. a setGoalHeadingOffset_deg(), setGoalPitchOffset_deg(), etc.

Other than these two issues, the next step here will be to look at some
further optimizations, and to write support code for a tower view.  That
should be fairly simple at this point.  I was considering creating a
"simulated tower view" or "pedestrian view" that defaulted to a position off
to the right of whereever the plane is at the moment you switch to the tower
view.  This could be a fall back when we don't have an actual tower location
at hand (as would be the case with rural airports).

ViewManager:
Basically all I did here was neaten things up by ripping out excess crap and
made it compatible as is with the new interface.

The result is that viewmanager is now ready to be developed.  The two
preexisting views are still hardcoded into the view manager.  The next step
would be to design configuration xml (eg /sim/view[x]/config/blahblah) that
could be used to set up as many views as we want.  If we want to take the easy
way out, we might want to insist that view[0] be a pilot-view and have
viewmanager check for that.

Various Irix cleanups.
Float/Double clean ups for fg_sound.cxx

Added missing file from JSBSim CVS, with 'using namespace std;'
statement added for ANSI-conformant compilers.

Irix fixes.

Updates.

First commit of properties code.  JSBSim now has a basic property tree all
under /fdm/jsbsim

Major property-manager rewrite, using const char * throughout
interface instead of string.  This will result in a lot more
efficiency later, once I add in a simple hash table for caching
lookups, since it will avoid creating a lot of temporary string
objects.  The major considerations for users will be that they cannot
use

  node->getName() == "foo";

any more, and will have to use c_str() when setting a string value
from a C++ string.

Oops, "one third from the edge" means "one SIXTH from the center".

Somewhere along the line in the recent changes some std::cout were
inclueded in some of the files. Irix doesn't have cout in the std class,
so I changed it to "cout" and included SG_USING_NAMESPACE(std) at the
beginning of the files.
And some minor warning cleanups.

White space tweaks.

Here is a cleaned up hitlist that should solve the PLib conflict
and fixes a 'potential bug' if the FGFS View code were to change

I also consolidated the specialized IntersectLeaf()
as they really didn't gain us much outside of their having
'more direct access into the SSG controlled data'

I would like to see the fgCurrentElevation functions moved
out of hitlist.cxx.

The one obstacle is their being dependent on my PLib
auxillary functions

ssgGetEntityTransform()
ssgGetCurrentBSphere()

code has been run through astyle with the default options

Incorporated Norman's optimized line/geometry intersection code.

Initial revision.  Things are just beginning to be stubbed in now.

Oops, fixing the ghosts of bugs past. :-)  Good thing I remembered how I
fixed this when I made the mistake the first time.
The view code wasn't properly handling the transition across tile
boundaries so we'd get a 'flash' of the scene wrongly transformed for
one frame at every tile boundary crossing.  This is what
scenery.get_next_center() is for.

Removed at CM's request.

Major weather update from Christian Mayer, tying the weather code into
the property system, among other things.  A separate integration into
the FDMs will follow shortly.

This code will be used only if the --with-new-environment option is
*not* passed to configure.

Replaced some debugging structure David inadvertantly removed.

Patches from Tony Peden to separate property XML I/O operations into a
separate header file.  This change will help integrate properties into
JSBSim.

Also, I (David Megginson) removed most of the SimGear include
statements from globals.hxx, reducing the amount of recompilation
every time SimGear changes.  This required making minor changes to a
lot of files that were depending on the side-effects of the inclusions
in globals.hxx.

Renamed /cockpit property to /sim/view/internal.

Print an error message before terminating on parse errors.

More work on saving and restoring flights.  Restoring a flight is not
working in a running FlightGear session because of JSBSim trim-routine
problems, but it does work from the command-line now, i.e.

  fgfs fgfs.sav

Change fog punch through value a bit.  Either way it's a WAG for now, but
at least it's less obviously wrong.

Removed some cout statements left in accidentally.

Centralized most view-management code in FGViewMgr.  It's still a
mess, but the mess is all in one place now.

Combined and updated some code to calculate fog parameters.

Fix several signed vs. unsigned warning messages.

Autopilot cleanup:

- implement the standard FGSubsystem interface, for consistency
- eliminate current_autopilot and add get/set_autopilot to FGGlobals,
  for consistency
- use private methods rather than static functions for tying
  properties

There should be no change in functionality.

Add "(" and ")" to ensure the expression is evaluated correctly.

Eliminate dependencie on tying variable pointers (in preparation for
property-code simplification).

Added a final newline to avoid a compiler warning.

Working on the start of a 'standard' <grin> remote FDM interface.  This
is a work in progress and needs severe enhancement before it will be
useful.  It provides a UDP data channel that goes both ways between
flightgear and the fdm.  It also provides a TCP 'command' channel so
flightgear can 'reliably' send commands to the remote fdm (such as set
starting position, reset on ground, etc.)

Better support of the newer more flexible object file format.  This
includes the ability to specify per vertex normals rather than depending
the normals list being the same as the vertices list.  (Support for
previous binary file format scenery is maintained.)

Patch from Jim Wilson to fix internal cockpit view rotations:

This seems to pretty much correct the problem.  Part of the problem is that
rotations are occuring at the firewall (model origin) which seems a little
un-natural inside the cockpit.  The rest of the problem is I am just learning
how this stuff works (I know I've been saying this for a couple months
now...but hey I'm slow :-)).

*** empty log message ***

Fixes for Irix compatibility.
Fix several long standing irix compiler warnings.

Added a new 'delimiter' property to allow an alternative delimiter to
the comma, and fixed a constant that was causing CygWin compilation
problems.

Documented the new 'delimiter' property, allowing an alternative to
comma-delimited, and noted that no delimiter escaping is done in
property values.

Add support for elapsed time and logging.

Bind a new property, /sim/time/elapsed-ms, giving the number of
milliseconds elapsed since the simulation began.

Added new logging module.

*** empty log message ***

Patches from Erik.

Removed some extra debugging output that wasn't supposed to be committed.