Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 23 May 2018 07:35:06 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        Eitan Adler <eadler@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r334078 - head/sbin/devd
Message-ID:  <CANCZdfoKvBo2XWo%2BARzyJmvDAYYxkAH=jd02W3ga-fX0PKtLYA@mail.gmail.com>
In-Reply-To: <201805230739.w4N7d5Z1062496@repo.freebsd.org>
References:  <201805230739.w4N7d5Z1062496@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, May 23, 2018 at 1:39 AM, Eitan Adler <eadler@freebsd.org> wrote:

> Author: eadler
> Date: Wed May 23 07:39:05 2018
> New Revision: 334078
> URL: https://svnweb.freebsd.org/changeset/base/334078
>
> Log:
>   devd: compile at WARNS=6
>
>   Verified with "make universe TARGETS='amd64 arm arm64 i386 sparc64'"
>
> Modified:
>   head/sbin/devd/Makefile
>   head/sbin/devd/devd.cc
>   head/sbin/devd/token.l
>
> ...

> Modified: head/sbin/devd/token.l
> ============================================================
> ==================
> --- head/sbin/devd/token.l      Wed May 23 07:39:02 2018        (r334077)
> +++ head/sbin/devd/token.l      Wed May 23 07:39:05 2018        (r334078)
> @@ -38,6 +38,7 @@
>  #include "devd.h"
>  #include "y.tab.h"
>
> +extern int lineno;
>  int lineno = 1;
>
>  static void


This is completely bogus.  Never ever ever just put extern in the line
before to avoid the warning. That's wrong 100% of the time.

Fix it right by putting this in devd.h.

Warner



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfoKvBo2XWo%2BARzyJmvDAYYxkAH=jd02W3ga-fX0PKtLYA>