Small cleanups to the SGGeo[dc] classes, provide more hooks to use them directly

Use function argument in va_start instead of local variable.

Remove now unused function

Modified Files:
simgear/scene/material/mat.cxx simgear/scene/material/mat.hxx
simgear/scene/material/matlib.cxx
simgear/scene/material/matlib.hxx simgear/scene/tgdb/leaf.cxx
simgear/scene/tgdb/obj.cxx
Attach userdata to groundtile scenegraph leafs that contains
a SGMaterial reference to the material of that leaf.
Add (physical) material properties to the material definitions.
Plug a memory leak with GlyphSigns.

Preliminary material lookup hooks - still unoptimized.

whoops, sorry (Yes, it *was* tested, but then I made another "trivial"
change and ...)

if we are going to die we better tell all our listeners

add optional position argument to SGRoute::add_waypoint(). Default is -1,
which appends the WP like it used to. Valid vector indices insert the WP
at this position.

Mac fix

Fix the initial texture path problem. Loaders are setting the one given to ssgLoad as the default one behind our back :-(

Redefine the default PLIB loader behavior : don't clear the texture cache after every model load

add method to delete any waypoint (last waypoint if n is out of range)

Pigeons remaining fix for the soundmanager crashes.

thanks to Erik's texture map I can now drop empty.rgb altogether and just
specify the same texture in the "foo.lighted" and "foo.unlighted" material
entry. This also allows to drop the state cloning and thereby solves the
most urgent apt_signs.cxx TODO. :-)

Add a texture cache mechanism. Fortunately this oly seems affective for  empty.rgb ...

don't allow new command name to overwrite material name

- don't use hard-coded emission values for unlighted signs, but load both
  states from material.xml (separate <material> entries for now)
- clone state less often: not once per sign element, but once per material
  switch (TODO: clone only once per material)

fix "unknown.rgb" path (the wrong path was the reason why we always only
got plib's lowres red-white chequer-board pattern along with an error
message, and not ours ... which is much prettier, but also bigger.
(Should we downscale it?)

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.)

rename OBJECT_TAXI_SIGN to OBJECT_SIGN. This isn't about taxi signs any
more, but all sorts of signs. Now is the best time to get rid of a
misleading name.

add <condition> support to textranslate & texrotate animation

set sign orientation such, that when the sign heading=0, one looks straight
on the sign face when looking North

lower signs

add minimalistic backside to signs as a temporary solution

re-add hard-coded vertical distance. The coordinates should be surface
points and not add this distance, which depends on the sign housing/hardware,
after all.

- commands do now have to start with @
- add commands @size, @material, @light
- make "BlackSign" texture default
- make @B, @R, @L, @Y open close their frames automatically (this can be
  avoided by setting the @material manually)
- add number variants for those 4 sign commands: @Y2, @B5, etc (according
  to the spec; defaulting to the respective biggest panel size, i.e. @B = @B3)
(detailed description will be added to $FG_ROOT/Docs/)

remove obsolete files (on request by Christian Mayer, who has introduced them):
- they are not used anywhere in sg/fgfs
- and are very clearly *not* GPL compatible!

Manabu Nishiyama (non-FlightGear Nasal user) discovered an
uninitialized data bug in naHash_cget().  When the hashcode field of
naStr was introduced, I forgot to set it in this function, which
creates a temporary naStr on the stack.

drop xscale member again, and use xsize/ysize instead. (One interface
element less to confuse people.)

rewrite of OBJECT_TAXI_SIGN code. The name is a bit misleading, as this
type can also create runway signs. (/me thinks about changing that ...)

support for font textures. They are normal (but rather lenghty) <material>,
but contain <glyph> entries with <name>, <left> and <right>. The latter two
describe where in the texture a letter or symbol begins and where it ends.
(range 0-1). <xscale> defines a horizontal scaling factor.

v0.3.10 changes.

protect ssg pointers to avoid occasional crashes  (of course it would be
nicer if the Occluder would always get removed before its model branch,
but that's not easily enforcable)

v0.3.10-pre3 updates.

Olaf Flebbe:

Unfortunatly the original SimGear Project I submitted contained local
changes which I didn't intend to submit. Please use this file instead...

If the author of this message isn't alerted enough to *fix* this, then I'm
sure the users won't do that either. This is regularly triggerd and leads
to meaningless error reports.

more guarded pointers (we are still getting spurious crashes on exit because
of that)

Initial revision.

remove so that complete dir can be dropped

Updates to remove unneeded and old version of zlib source.

Remove old depricated files.  There is a much newer version of zlib
available and just about every unix-based platform already comes with this.

v0.3.10-pre2 updates.

Vassilii KHACHATUROV: rain fix, cleanup, read formerly hard-coded values
                      from provided node

"* minor redundant gl call in DrawCone2 optimized away (twice per frame)
* corrected the glRotatef() order in drawRain even further (a less
obvious mistake than before, which is noticed by looking skywards and wiggling
the mouse in the view direction change mode)
* all the "magic numbers" used in the rain rendering code have been
provided a default (based on the old hardcoded value) in a form of a
define, and a meaningfully named static member in SGEnviro"

Vassilii KHACHATUROV: fix typos, add/fix comments, change variable names

Oops, implement the previously-mentioned fix without breaking support
for omitting a length argument to mean "the rest of the vector".

The original code (rather oddly) interprets a length of zero in
subvec() to mean "the whole vector".  Melchior showed a use case
(removal of the first element from a vector) where getting a
zero-length subvector is actually desired.  And since I can't come up
with a good reason for why the "feature" was there in the first place,
out it goes...

Melchior discovered that cmp() was just wrong, failing to actually
inspect the string pointers.  It also failed to properly sort strings
where one is a prefix of the other.  It looks to me like I just never
finished this, and it ended up in CVS because it just happened to
compile...

Updates for v0.3.10-pre1.

- better error message when submodel loading failed
- use alignmainmodel node in callback (not used anywhere yet)

The handle gets nulled out if the user has closed the file; don't pass that
null to fclose() in a garbage collection destructor too...

Fix broken checkin in iolib.c.  Also add the 'bits' library, which has a buf()
function needed to make convenient use of io.read().

Melchior found the first bug -- report EOF as nil in readln().

Add the Nasal I/O library so Melchior can play with it.  Not enabled currently (see NasalSys.cxx in the flightgear CVS)

--warnings

--warnings

close zone.tab file after having read all entries

prevent animations from losing nodes, because other processes removed them
(We are currently getting a lot of aborts in the condition code when
running MP. I don't expect this to fix it, but a bug is a bug.)

(reviewed by Fred, who also fixed *my* bugs :-)

Better use a node that is very clearly not used by other services
(e.g. animations). The data class doesn't mind.

activate model load/unload callback again. It turned out *not* to be the
cause for the MP crashes -- the same crashes did still occur without it.

Mac OS X fixes from Ima Sudonim.

commenting out Nasal in scenery models for now. This could be responsible
for an MP/AI crash. Still investigating.

warning--

Alexander Powell:

Add MAC OS X Render Texture support:

Most texture modes seem to work on my Powerbook, but I don't have a wide array
of machines to test it on otherwise.

If you have problems, please let me know and I'll see if I can help track down
the source of the bug.  I'd love to keep working on it if time permits (I use
RenderTexture in a few other projects), so I'll keep you informed if there are
any changes that I make for the better.

model.[ch]xx:
  add abstract class SGModelData. If a pointer to such a class is handed over
  to sgLoad3DModel, then its modelLoaded() method is called with path, property
  node and branch. And then it's added to the scene graph so that it's
  destroyed when the model branch is removed from the graph.

modellib.[ch]xx:
  only cache objects when asked to. This is the case for OBJECT_SHARED
  and random objects (like before), but no longer for OBJECT_STATIC.
  These are now removed from the graph when they are "out of sight". This
  prevents accumulation of static models, and makes destroying model data
  possible (e.g. removing Nasal modules)

matmodel.cxx:
  set cache flag for random objects (same behavior as before)

- new FSF addresses
- coplied license headers from h(xx) files to respective c(xx) files
- removed trailing spaces
- fixe $Id$
- fixed typos

Ignore generated binaries.

Mathias Fröhlich:

zero out all parent pointers, else they might be dangling.

Mathias Fröhlich:

I have some bugfixes which will avoid fg just crashing if the sound device
could not be opened.

Wrong config file name

Fix from Melchior: Set static values at every condition change, not only initially.

Melchior FRANZ:

- don't unlock an already unlocked mutex (Someone wanted to be on
  the safe side with this, but the result is undefined and makes
  pthread_mutex_destroy fail. Reference: manpage for
  pthread_mutexattr_gettype/The Open Group[1]: "Attempting to
  unlock a mutex of this type which is not locked results in
  undefined behaviour.")

- re-enabled all subsystem destructors again (this has been disabled
  because fgfs hung on exit, due to the mutex destroy failure from
  above.)

Reference:
  http://www.opengroup.org/onlinepubs/007908799/xsh/pthread_mutexattr_gettype.html

Back out the previous patch.

Declare specified functions, otherwise MIPSpro bails out.

Melchior FRANZ:

- new FSF address
- removed a few hundred trailing spaces
- fixed a few $Id$ lines
- copied two license headers from *.hxx files to their respective
  *.cxx counterparts
- added two test aps to .cvsignore
- don't unlock an already unlocked mutex (Someone wanted to be on
  the safe side with this, but the result is undefined and makes
  pthread_mutex_destroy fail. Reference: manpage for
  pthread_mutexattr_gettype/The Open Group[1]: "Attempting to
  unlock a mutex of this type which is not locked results in
  undefined behaviour.")
- re-enabled all subsystem destructors again (this has been disabled
  because fgfs hung on exit, due to the mutex destroy failure)

Mathias Froehlich: Add license information.

Mathias Fröhlich:

This patch makes use of the vectors now available in simgear with that past
patch. And using that it simplyfies the carrier code somehow.

- Small additional factory's to the quaternion code are done in the simgear
  part. Also more explicit unit names in the factory functions.
- The flightgear part makes use of them and simplyfies some computations
  especially in the carrier code.
- The data part fixes the coordinate frames I used for the park positions in
  the carrier to match the usual ones. I believed that I had done so, but it
  was definitly different. Also there are more parking positions avaliable now.

Melchior FRANZ:

- change SG back to FG (this was accidently changed in three wrong
  places when I prepared the file for SG)
- correct length for the proxy id detection
- set (guessed) deposit depth < 1mm correctly
- set deposit type string
- formatting

Missing defines

Add missing include files needed by the new math code under windows

Fix Code generation option for debug build

Remove compiler warnings

Clean up some compiler warnings.

Remove unused files.

Melchior FRANZ:

This patch fixes the sound of 737, Concorde and others, if fgfs
was compiled with newer gcc versions (e.g. gcc 4.0.2). These compilers
implement the c++ standard more strictly, and thus don't guarantee
that c-style casted pointers to different data types really point
to the same address. This is only guaranteed for union members.

Mathias Fröhlich:

The patch adds a vector lib I have put together during the last time,
it is just handy and interfaces well with s(s)g*. Together with some small
modifications this will later also interface well with OpenSceneGraphs
vectors/matrices. Using this vector kernel is targeted to have a handy
matrix/vector lib available and to provide a scenegraph independent vector
type for use with a small scenegraph wrapper aimed for a smooth transition to
openscenegraph.

That vector code also includes an improoved geodetic conversion routine I
have found some time ago published in the 'journal of geodesy' which avoids
iterative computations for that purpose.

Also the geodetic position class is more typesafe and unitsafe than the
Point3D currently is.

That part is relatively old and in use in my local trees for several months
now.

Olaf Flebbe:

This patch makesFlightGear at least compile on MSVC. I hope I have removed
reference of my other local changes. DSP and DSW files are included for
reference. They have been reconstructed with am2dsp.pl. I had to introduce a
change to am2dsp because of the need of filenames with embedded spaces. (Yuck)

The major direction is to remove clutter like the _USE_MATH_DEFINES and
have it on the compiler command line sice there is no central include
file. You will have to put it on the command line for your locale
Project files, if it not there, already. I added the
_CRT_SECURE_NO_DEPRECATE define for 2005, since it does no harm to other
VC version.

Third Party Libs like plib, OpenALSDK, freeglut, pthreads-win32, zlib
are unpacked as is side by side. Only change put the includes of OpenAL
into include/AL rather directly into include.

Outputing 6 digits is not enough for a double

Vassilii Khachaturov:

Fix the current buggy rain orientation behaviour for the views attached to the
aircraft (while still inheriting bugs with the views attached to anything else).

Expose the sun halo texture handle.

Expose GL_COORD_REPLACE

create an absolute value before calling log or log10, this adds support for sound on negative numbers (thrust reverse for example).

Melchior FRANZ:

add optional arg to SGPropertyNode::addChangeListener that triggers
the listener function call initially. This is useful for cases where
a freshly installed listener wants to treat the current property
value as changed from 'unknown' to the actual value right away.

Examples can be found in the Nasal incarnation setlistener(),
where we have for example this (in $FG_ROOT/Nasal/gui.nas):

  INIT = func {
      ...
      setlistener("/sim/rendering/fps-display", fpsDisplay);
      if (getprop("/sim/rendering/fps-display")) {
          fgcommand("dialog-show", props.Node.new({"dialog-name": "fps"}));
      }
  }

That is: we first attach a listener that cares for changes to the FPS
display switch, but then we have to manually open the dialog initially.
That's a duplication of code and could be as simple as this
(INIT part only):

  INIT = func {
      ...
      setlistener("/sim/rendering/fps-display", fpsDisplay, 1);
  }

That is: the optional third arg makes fpsDisplay be called initially,
and then again with every write action. My first solution was in the
Nasal code only, but Andy (rightfully) says that this should rather
be in sg.

On a second thought, disable smooth shaded lines for all segments of the lightning.

Disable smooth lines for certain lightning segments.

Add support for point sprites.

The sample is now managed by a SGSharedPtr and shouldn't be deleted explicitely

Mathias Fröhlich:

Incorporating the shared ptr code:
- All scenegraph references from SimGear
- SGMaterial which already had a reference counter uses now that common
  infrastructure.
- SGMatModel is now counted.
- SGSoundSample from SimGear
- And the corresponding change for the sound samples in flightgear which fixes
  a latent crash if FGBeacon would evern be deleted.

Mathias Fröhlich:

Use the new automatic reference counter instead of doing
that ourselfes.

MSVC vsnprintf ( in fact _vsnprinft ) returns -1 when the buffer is too short

Use the new SGPath::create_dir function
Ensure no triangles array could have more than 32767 vertices, a PLIB limit.