   I decided to take a shot at NVIZ, since the codebase seems to be
stable, and (of course) I have a patch for it.  There is the usual
prototyping and 'return' fixing, along with two little items that
somebody else needs to look at:

src.contrib/GMSL/NVIZ2.2/src/lights.c
line 30:    return(switchlight(data, interp, argc, argv));
   switchlight() is defined to have two arguments, (int, char**)
    so I think that the first two arguments in this call are
    incorrect.  I could be wrong, though.
  
src.contrib/GMSL/NVIZ2.2/src/position.c
line 178:   GS_moveto(atof(argv[1]), atof(argv[2]), atof(argv[3]));
   GS_moveto() is defined to have only one argument, (float *) so
    I know that this call is wrong.  Somehow, we need to get a
    pointer to the float value of one of these argv-s, but I don't
    really understand everything that is going on in this module.

The patch is from the beta6h code, these two items are still present
on CVS today (2/5/2000.)  The patch is on my homepage at
http://www.grasshoppernet.com/~bhughes/patchNVIZ.5.0b6h.  I have
changed these two calls in the patchfile, but I have little hope that
the new versions are right - they do compile cleanly, though.

I'd appreciate it if someone familiar with NVIZ could give these
the once-over and let me know what needs to be done.

Bill Hughes

