> From:    George Caswell <timbuktu@adamant.res.wpi.net>
> Subject: XV 3.10a XPM save bug
> 
>    Hi.  After a lot of frustration, I found the bug that's been causing
> XV3.10a to save corrupted XPM files.  There's an unterminated case in the
> switch statement of DoSave() in xvdir.c.  At line 1171 or thereabouts 
> there is the case F_XPM...
> 
>   case F_TARGA:
>     rv = WriteTarga (fp, thepic, ptype, w, h, rp, gp, bp, nc, col);    break;
>       
>   case F_XPM:
>     rv = WriteXPM   (fp, thepic, ptype, w, h, rp, gp, bp, nc, col,
>                      fullname, picComments);
>   case F_FITS:
>     rv = WriteFITS  (fp, thepic, ptype, w, h, rp, gp, bp, nc, col,
>                      picComments);
>     break;
> 
>   Need a break statement after the rv = WriteXPM() call, or else you get nice
> little xpm files with wonderful FITS files appended.  Patch follows.

Doh!

--jhb


*** xvdir.c.old	Tue Jan  3 16:21:39 1995
--- xvdir.c	Sun Mar 22 01:13:06 1998
***************
*** 1168,1174 ****
      
    case F_XPM:
      rv = WriteXPM   (fp, thepic, ptype, w, h, rp, gp, bp, nc, col, 
! 		     fullname, picComments);    
    case F_FITS:
      rv = WriteFITS  (fp, thepic, ptype, w, h, rp, gp, bp, nc, col, 
  		     picComments);    
--- 1168,1175 ----
      
    case F_XPM:
      rv = WriteXPM   (fp, thepic, ptype, w, h, rp, gp, bp, nc, col, 
! 		     fullname, picComments);                           break;
! 
    case F_FITS:
      rv = WriteFITS  (fp, thepic, ptype, w, h, rp, gp, bp, nc, col, 
  		     picComments);