Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 22 Apr 2017 14:09:36 -0700 (PDT)
From:      "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net>
To:        "Ngie Cooper (yaneurabeya)" <yaneurabeya@gmail.com>
Cc:        rgrimes@freebsd.org, Ngie Cooper <ngie@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r317290 - head/tools/regression/geom_gpt
Message-ID:  <201704222109.v3ML9ahr047899@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <3238ED82-1FAA-4E7C-81B9-A9E9E417E860@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
-- Start of PGP signed section.
[ Charset UTF-8 unsupported, converting... ]
> 
> > On Apr 22, 2017, at 13:06, Rodney W. Grimes <freebsd@pdx.rh.CN85.dnsmgr.net> wrote:
> > 
> >> Author: ngie
> >> Date: Sat Apr 22 20:00:52 2017
> >> New Revision: 317290
> >> URL: https://svnweb.freebsd.org/changeset/base/317290
> >> 
> >> Log:
> >>  Fix -Wimplicit-function-declaration compilation warning by moving libgeom.h
> >>  #include below the stdio.h #include.
> >> 
> >>  gctl_dump(3) needs stdio.h, per reasoning noted in r317289.
> >> 
> >>  MFC after:	5 weeks
> >>  PR:		218809
> >>  Submitted by:	Chang-Hsien Tsai <luke.tw@gmail.com>
> >>  Sponsored by:	Dell EMC Isilon
> >> 
> >> Modified:
> >>  head/tools/regression/geom_gpt/test.c
> >> 
> >> Modified: head/tools/regression/geom_gpt/test.c
> >> ==============================================================================
> >> --- head/tools/regression/geom_gpt/test.c	Sat Apr 22 19:32:23 2017	(r317289)
> >> +++ head/tools/regression/geom_gpt/test.c	Sat Apr 22 20:00:52 2017	(r317290)
> >> @@ -29,12 +29,12 @@ __FBSDID("$FreeBSD$");
> >> 
> >> #include <sys/param.h>
> >> #include <errno.h>
> >> -#include <libgeom.h>
> >> #include <limits.h>
> >> #include <stdio.h>
> >> #include <stdlib.h>
> >> #include <string.h>
> >> #include <unistd.h>
> >> +#include <libgeom.h>
> > #include <libgeom.h>	/* This is not in alphabetic order per r317289 */
> >> 
> >> struct retval {
> >> 	struct retval *retval;
> > 
> > Might it be a good idea to mark this in the test.c file as to why
> > these are not sorted in order to prevent regressions of this in
> > the future?
> 
> Please see r317289. There was an omission in the documentation that allowed this to happen.

I had seen that already when I made my post, that docuement is not going
to stop someone from going "Oh, these are out of order I am going to sort
them since I am here"  They well then probably have issues due to your next
statement about WARNS and go hum, what is that all about.  And either 
investigate and hopefully find the right thing, or do more wrong things.

When #includes are out of order for good reason the source code file should
be market as such and not dependend on the near 0 likelyhood someone
is going to go read a man page to find out why.

> 
> This also won?t happen again given that the test is now being compiled with WARNS?= 6 :).
> 
> -Ngie
-- End of PGP section, PGP failed!

-- 
Rod Grimes                                                 rgrimes@freebsd.org



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201704222109.v3ML9ahr047899>