svn commit: r243134 - head/sys/sys

mdf at FreeBSD.org mdf at FreeBSD.org
Fri Nov 16 16:02:40 UTC 2012


On Thu, Nov 15, 2012 at 10:25 PM, Konstantin Belousov <kib at freebsd.org> wrote:
> Author: kib
> Date: Fri Nov 16 06:25:20 2012
> New Revision: 243134
> URL: http://svnweb.freebsd.org/changeset/base/243134
>
> Log:
>   Alphabetically reorder the forward-declarations of the structures.
>   Add the declaration for enum idtype, to be used later.

Forward declarations of enums isn't an ISO C feature, but a gcc
extension.  While the kernel uses many gcc extensions, it hides most
under a #define so unsupported compilers can continue along.  This
unsupported feature can't be hidden.

Does the forward declaration prevent another warning?

Thanks,
matthew

>   Reported and reviewed by:     bde
>   MFC after:    28 days
>
> Modified:
>   head/sys/sys/syscallsubr.h
>
> Modified: head/sys/sys/syscallsubr.h
> ==============================================================================
> --- head/sys/sys/syscallsubr.h  Fri Nov 16 06:22:14 2012        (r243133)
> +++ head/sys/sys/syscallsubr.h  Fri Nov 16 06:25:20 2012        (r243134)
> @@ -35,25 +35,26 @@
>  #include <sys/mount.h>
>
>  struct file;
> +enum idtype;
>  struct itimerval;
>  struct image_args;
>  struct jail;
> +struct kevent;
> +struct kevent_copyops;
> +struct kld_file_stat;
> +struct ksiginfo;
>  struct mbuf;
>  struct msghdr;
>  struct msqid_ds;
> +struct ogetdirentries_args;
>  struct rlimit;
>  struct rusage;
> -struct __wrusage;
>  union semun;
> +struct sendfile_args;
>  struct sockaddr;
>  struct stat;
> -struct kevent;
> -struct kevent_copyops;
> -struct kld_file_stat;
> -struct ksiginfo;
> -struct sendfile_args;
>  struct thr_param;
> -struct ogetdirentries_args;
> +struct __wrusage;
>
>  int    kern___getcwd(struct thread *td, u_char *buf, enum uio_seg bufseg,
>             u_int buflen);


More information about the svn-src-head mailing list