James Turner:
Convert FGRunway to be heap-based, and inherit FGPositioned. This is a large, ugly change, since FGRunway was essentially a plain struct, with no accessors or abstraction. This change adds various helpers and accessors to FGRunway, but doesn't change many places to use them - that will be a follow up series of patches. It's still a large patch, but outside of FGAirport and FGRunway, mostly mechanical search-and-replace.

An interesting part of this change is that reciprocal runways now exist as independent objects, rather than being created on the fly by the search methods. This simplifies some pieces of code that search for and iterate runways. For users who only want one 'end' of a runway, the new 'isReciprocal' predicate allows them to ignore the 'other' end. Current the only user of this is the 'ground-radar' ATC feature. If we had data on which runways are truly 'single-ended', it would now be trivial to use this in the airport loader to *not* create the reciprocal.

James Turner:

Here's part 2 - converting FGFix (the simplest one) to be both heap-based and inherit FGPositioned. One minor benefit from this is replacing some dangerous code in FGFixList which used to return the address of an iterator member ('&it->second'). To keep the diff a sensible size, I'm not updating the callers to use the richer FGPositioned types - i.e replacing separate lat/lon handling with SGGeod. I will make those cleanups, but in future patches.

Update MSVC 7.1 project

Add required include for lower_bound

James Turner:
Here's the very first FGPositioned patch - adding the files, and building them, but nothing else.

James Turner :
If someone could kindly apply the attached patch, that'll keep this from crashing, I believe. The fix is easy since FGAirport can now always provide an active runway - there's no need to guess at random, or rely on the tower having set one up.

Update MSVC 7.1 projects

Commit the proper patch stupid :( now with the positioning and orientation code included...

Don't update twice..

revert the move of the sound positioning code, at least till i found out why it doesn't work this way.

Move the soundmanager update code back to the main loop, just prior to updating all the SGSubsystems. Hopefuly this silves the sound problem.

Add a MSVC 7.1 project for the metar program

Stefan C. Müller :
Small patch fixing bugs I've encountered while getting the current CVS to build in MSVC.
* std::lower_bound was used with the key-type of a map, but lower_bound expects the value-type of the collection it works on, with is std::pair. MSVC seems to be more strict about this.
* Added an missing include statement.
* Replaced an rint() call with floor() (MSVC does not offer rint).

Update MSVC 7.1 projects

Torsten Dreyer:

Here is a little patch that changes the behaviour of the VOR CDI and OFF-flag
for indicators like the HSI when getting outside the range of the VOR
station.

Currently, when flying at a distance between the effective_range and twice the
effective_range of a VOR station, the in-range property is computed based on
a random value, causing the OFF Flag and the CDI bar to perform an ugly
jitter.

The attached patch introduces a new property signal-quality-norm which is
computed based on the distance to the station and the range. It is 1.0 when
the distance is less than the range and decreases by 1/x^2 for distances
greater than the range leading to a signal-quality-norm of 0.25 for distances
two times the range, 0.125 for three times the range and so on.
The in-range flag is tied to a signal-quality-norm greater than 0.2 (fixed
squelch).
The CDI and GS needle deflection is multiplied with the signal-quality-norm.

The benefit is:
- Ability to animate the OFF-Flag with a smooth transition.
- CDI and GS needle deflection shows correct values when in range
(signal-quality-norm=1.0) and show some wrong indication when the range is
exceeded
- CDI and GS needle start to move, even when the OFF flag is visible
- No more jitter for flag and needles

See the new SenecaII ki525a hsi as an example at
http://www.t3r.de/fg/navpatch.jpg
The numbers on the image are:
(1) the new property signal-quality-norm
(2) distance exceeds the effective-range by 30%
(3) NAV flag has a rotation animation bound to signal-quality-norm and is
partially visible
(4) CDI is partially deflected even with NAV flag shown

This implementation better matches reality - at least, how I observed it ;-)

Updates to match prerelease version number.

"make dist" fixes.

- Update FlightGear.dsp
- Assign an arbitrary prerelease version number
- Tweaks to top level Makefile.am

Sync. with JSBSim cvs

Fix missing parameter reported by Alex Buzin

Not a right but a left hand system

James Turner:

Good news: I'm working on some automatic testing of the 'core' FG pieces, especially those I'm likely to break in my Navaids / airports / runways work

Bad news: I already broke something, in my runways refactoring. (But my tests caught it!)

Attached patch fixes it - it's (of course) the stupidest thing in the world. Incidentally, standardising this kind of code into some (inlined) header is becoming more and more of a priority  for me - I've lost count of the number of times I've seen the 'clamp heading to 0..360.0' and 'reverse a heading and clamp it' idioms in the code. The KLN89 and MkVIII code have (of course) their own helpers for this.

remove redundant semi-colon

Fix a compile problem for lower_bound()

Update angles of inner and outer cone. they run from 0-180 degrees.

James Turner:
This is a little intrusive on the KLN89 code, but avoids the wasteful cloning of the airports, runways and navaids which current happens, and also combines the ugly string ordering code.

fix a last minute typo

Nothing fancy, it all seemed to work properly already...

update coordinate system

Czaba Halasz:
Somebody please add config.h to AirportList.cxx, because the #include
<plib/puAux.h> (in AirportList.hxx) needs the PU_USE_NONE define.

DT: Done :-)

James Turner: Here's a trivial patch, when you have a moment:

- removes various members from FGRunway which no-one was using
- any of these can be trivially re-instated if and when someone
actually wants to use them - but right now they're simply bloating up
FGRunway, which we have lots of, because it currently includes all the
taxiways in Robin's data.
- that's it.

Move SGSoundMgr stuff oevr to FGFX

lsprop: version math.pi  (change %'v -> %q and %"V -> %Q; cosmetics)

lsprop: make included paths absolute

lsprop: also consider includes from <PropertyList>; help improvements

lsprop: fix -p long option

XMLErrors on one line

lsprop: list properties of <PropertyList> XML files

Update MSVC 7.1 projects

James Turner:
Switches the MkVIII code to use the same 'closest airport' logic as everything
else, instead of its own special version.

James Turner: Improved runway management code:
 - Runways are now part of an airport, instead of a separate list
 - Runways are no longer represented as a boring struct, but as a class
   of their own.
 -Improved runway access to unify various runway access methods.

Sync. with JSBSim CVS

nasal.vim: add <?...?> and <!--...--> to XML embedded mode
and only enable that if the file extension is xml

Allow generic file protocol to terminate fg after a number of repetitions.

Ron JENSEN: s/hide/!enabled/

use <enabled>false</enabled> flag for widgets that shouldn't be drawn
instead of <hide>true</hide>. This is consistent with other places
in fgfs, like menu entries, hud elements, subsystem switches, etc.

use <enabled>false</enabled> flag for widgets that shouldn't be drawn
instead of <hide>true</hide>. This is consistent with other places
in fgfs, like menu entries, hud elements, subsystem switches, etc.

<foo/> are also comments in XML embedded Nasal mode

- fix function argument order (top & bottom were swapped)
- drop wrong and superfluous rounding (sprintf() rounds already)
- remove redundant default values to property getters (default is 0 already)

James Turner:
Attached patch + new file make FGNavRecord have a .cxx file, and a constructor w
hich allows all the parameters to be supplied. Along the way I also cleaned up t
he navrecord.hxx header, lots more header pollution has been killed.

Some long methods are no longer inline, but were all suspiciously long to meet c
ompiler inlining criteria (I'm not clear if the 'inline' keyword is advisory or
mandatory in this situation) - I don't expect this to affect performance in any
way whatsoever.

The constructor addition is to support some hacking I'm doing improving the star
tup performance of the navDB by lazily loading the data, and caching it in a mor
e efficient format than text. I'm submitting this change (and probably some othe
r small tweaks in the future) since they are worthwhile as cleanups regardless o
f how my current experiments work out.

Add support for squared damping coefficients for gears.

mark obvious XML tags as comments

Restore sky

Don't clear BACKGROUND_BIT in the master camera's cull
mask. ViewPartitionNode does that where needed.

Update MSVC 7.1 projects - Adapt to OSG 2.6.0-rc1 : location of header files should now be searched in the install directory, here ..\..\..\install\msvc71\OpenSceneGraph\include
3dconvert not built anymore. Use osgconv

MSVC 7.1 fix

simplification

CameraGroup class for managing multiple cameras.

CameraGroup supports cameras opened in different windows or in the
same window, with flexible view and perspective specification.

Clean up mouse click handling via osgViewer. Don't let any camera
"have focus;" this forces events to be reported in window coordinates
and simplifies life. Don't use the osgViewer::View::requestWarpPointer
method; instead use our own code which only deals with window
coordinates.

Make glut and sdl versions work with CameraGroup too.

Make sure 'make dist' keeps working.

header cleanups

Explicitly set traversal mask for near camera.

Don't depend on camera inheritance; set the mask in the traverse() method.

better use unset() for unsetting ...

disable clip planes after use (Doesn't seem to make a difference,
and is only a temporary measure, as it was planned to OSGify the
HUD and put it in the scenegraph etc.)

s/clip/clipping/

- extract clipping code to HUD.cxx, allowing generic clipping
  and separate ladder clipping
- fix acceleration node names
- draw 90/-90 bars if there's no zenith/nadir

Changed ViewPartition to use two cameras instead of three.

I thought that this would fix the "black hole in the sky" problem,
which turned out to be caused by an OpenSceneGraph bug. Nevertheless
it is a simplification.

James Turner:
 * experimental clean-up / reduction on two of the FG headers:
   (I'm going to await feedback on the developers list before doing more of
    these, to avoiding going over files multiple times, but in principle it
    seems pretty straightforward.)

 * final fixes for SG_USING_STD removal

- read x/y/width/height as floats
- don't write bars past nadir/zenith (better MILSTD compliance)
- introduce tick-length/zero-bar-overlength/enable-dive-bar-angle
  configuration parameters
- simplification

SG_GLUT_H removed

- remove the SG_GLxxxx_H #defines, since OSG provides its own versions
 - this exposed a bizarre issue on Mac where dragging in <AGL/agl.h> in
   extensions.hxx was pulling in all of Carbon to the global namespace
   - very scary. As a result, I now need to explicitly include CoreFoundation
   in fg_init.cxx.
 - change SG_USING_STD(x) to using std::x

Restore inclusion of GLU.h

Remove unneeded inclusions of windows.h, GL.h and GLU.H

switch back to "<key> in <hash>" notation (this had failed in one test,
but the reason must have been unrelated)

Enable Radio #2.

Remove unused variables

get rid of mouse.cxx and pu{Show|Hide}Cursor()

remove all the FX and XMESA cruft relating to Glide support.

add cvs source urls

One occurence of Balloon.cxx is enough

But I do think we need to remove *this* reference to BaloonSim. :-)

Rvert last change. The Balloon and Magic Carpet are there. They didn't
show up during my previous commit though...

Add the BalloonSim and MagicCarpet fdm's back in (i seriously thought this had been done, oops)

Balloon and Magic Carpet no longer exist.

Update MSVC 7.1 projects - Balloon, BalloonSim and MagicCarpet not build anymore

Balloon and MagicCarpet already moved to SP_FDM

drop dependence on PyXML and use standard lib methods

This should apply, and everything should build cleanly, in isolation from the
SimGear change. It changes all the SG_xxxx to be the 'real' includes, and gets
rid of many #ifdef SG_HAVE_STD_INCLUDES. As an added bonus, rather than
replacing 'SG_USING_NAMESPACE(std)' with 'using namespace std', I just fixed
the small number of places to use std:: explicitly. So we're no longer polluting
the global namespace with the entire contents of std, in many cases.

There is one more 'mechanical' change to come - getting rid of SG_USING_STD(X),
but I want to keep that separate from everything else. (There's another
mechnical change, replacing <math.h> with <cmath> and so on *everywhere*, but
one step at a time)

Attached patches remove BORLANDC, and hence SG_MATH_EXCEPTION_CLASH and SG_INCOM
PLETE_FUNCTIONAL from SimGear and FlightGear.

As a result, SG_HAVE_STD_INCLUDES is now *always* set, so I will get the boring
fixes for that done, but separately. I'm still auditing the other things in comp
ilers.h - there's a lot that can die now BORLAND is gone.

Some Mac / __APPLE___ cleanups:

    - remove the OSX_BUNDLE crap *I* introduced years ago - we're always a a bun
dle on Mac now.

    - fix up the default fg-root on Mac to be FlightGear.app/Contents/Resources/
data - i.e the location used by the macflightgear.org distro, and indeed the obv
ious 'correct' location. Not sure why I didn't use that in the first place, back
 in the day.

    - remove the CPSForegroundEnable hack. For one thing, we're a bundle and don
't need it, and for another, osgViewer on Mac does the same logic using a newer,
 public API rather than a hack into the OS.

    - remove the strange logic for doing fgOSInit 'early' (in bootstrap rather t
han main) when running from the command line on Mac; again this is obsolete, and
 no one seems to know why it was ever necessary. I guess it was an interaction w
ith SDL when running without a bundle.

    - (not Mac related) remove obsolete code bracketed by ENABLE_PLIB_JOYSTICK a
nd USE_GLIDE (neither of which are ever set, even from config.h that I can see)
in main.cxx

Remove the remaining macintosh (not MacOS-X) and MWERKS workarounds.

typo: s/atl/alt/  (for "alternatives")

remove macintosh (note: not MacOS-X) and MWERKS workarounds.

argument & parameter completion for bash (instructions inside)

let removeChild() return the detached node (like the c++ version)

add printf

add thisfunc() and props.condition()

- use path validation for parsexml() nasal command
- util.cxx: add comment

Sync. with JSBSim CVS (header cleanups).

Clean up a couple warnings.

  * change a number of function parameters to const where appropriate
  * fix a problem where the wrong node-name length was returned
  * xmlgrep now also works when only the -e options is specified
  * fix xmlgrep to show the correct node-name (it reported the parent
    node-name in the previous version)

* fix __xmlSkipComment to properly find the end of comment tag.
* add the xmlGetNodeName and xmlCopyNodeName functions
* add the xmlCopyString function
* clean up some code

'xmlgrep -v' now returns the proper node name

Update MSVC 7.1 projects - new JSBsim