Re: lynx-dev (patch) simplify ifdefing with new USE_MOUSE symbol

[Klaus Weide]

On Sun, 2 Apr 2000, Leonid Pauzner wrote:

> At least I do not understand this DOSPATH
> in LYStrings.c fragment below:
> 
> > @@ -1464,7 +1478,7 @@
> >
> >  #if defined(IGNORE_CTRL_C) || defined(USE_GETCHAR) || !defined(NCURSES) || 
> > \
> >      (HAVE_KEYPAD && defined(KEY_RESIZE)) || \
> > -    (defined(NCURSES_MOUSE_VERSION) && !defined(DOSPATH))
> > +    (defined(NCURSES) && defined(USE_MOUSE) && !defined(DOSPATH))
> >  re_read:
> >  #endif /* IGNORE_CTRL_C || USE_GETCHAR etc. */
> >  #if !defined(UCX) || !defined(VAXC) /* errno not modifiable ? */

That's just for ifdefing out the label 're_read:', for all cases
where it never gets referenced, to avoid compiler warnings; the ifdef
isn't really necessary at all, if you can live with the warning (that
there is a label that isn't being used).  If the ifdef is growing
beyond being mamageable - looks like it - just get rid of it completely,
or take out some off the &&-ed conditions.

   Klaus