Re: kern/38900: Build of `sys/dev/wscons/wskbd.c' brocken if NWSDISPLAY == 0

["Arnaud Lacombe" <lacombar%gmail.com@localhost>]

The following reply was made to PR kern/38900; it has been noted by GNATS.

From: "Arnaud Lacombe" <lacombar%gmail.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: kern/38900: Build of `sys/dev/wscons/wskbd.c' brocken if 
NWSDISPLAY == 0
Date: Sun, 8 Jun 2008 22:56:58 +0200

 On Sun, Jun 8, 2008 at 10:55 PM,  <gnats-admin%netbsd.org@localhost> wrote:
 > Thank you very much for your problem report.
 > It has the internal identification `kern/38900'.
 > The individual assigned to look at your
 > report is: kern-bug-people.
 >
 >>Category:       kern
 >>Responsible:    kern-bug-people
 >>Synopsis:       Build of `sys/dev/wscons/wskbd.c' brocken if NWSDISPLAY == 0
 >>Arrival-Date:   Sun Jun 08 20:55:00 +0000 2008
 >
 here is the correct patch (fix an "unused variable" gcc's error):
 
 Index: /src/sys/dev/wscons/wskbd.c
 ===================================================================
 RCS file: /cvsroot/src/sys/dev/wscons/wskbd.c,v
 retrieving revision 1.119
 diff -u -r1.119 wskbd.c
 --- /src/sys/dev/wscons/wskbd.c     4 May 2008 13:19:17 -0000       1.119
 +++ /src/sys/dev/wscons/wskbd.c     8 Jun 2008 20:53:43 -0000
 @@ -1503,7 +1503,7 @@
  internal_command(struct wskbd_softc *sc, u_int *type, keysym_t ksym,
         keysym_t ksym2)
  {
 -#ifdef WSDISPLAY_SCROLLSUPPORT
 +#if NWSDISPLAY > 0 && defined(WSDISPLAY_SCROLLSUPPORT)
         u_int state = 0;
  #endif
         switch (ksym) {
 @@ -1519,7 +1519,7 @@
                 if (*type == WSCONS_EVENT_KEY_DOWN)
                         pmf_event_inject(NULL, PMFE_AUDIO_VOLUME_DOWN);
                 break;
 -#ifdef WSDISPLAY_SCROLLSUPPORT
 +#if NWSDISPLAY > 0 && defined(WSDISPLAY_SCROLLSUPPORT)
         case KS_Cmd_ScrollFastUp:
         case KS_Cmd_ScrollFastDown:
                 if (*type == WSCONS_EVENT_KEY_DOWN) {