Re: CVS commit: src/sys/kern

[Martin Husemann <martin%netbsd.org@localhost>]

On Wed, Jan 30, 2013 at 08:07:14AM +0000, David Laight wrote:
> On Tue, Jan 29, 2013 at 11:00:31PM +0000, Lars Heidieker wrote:
> > Module Name:        src
> > Committed By:       para
> > Date:               Tue Jan 29 23:00:31 UTC 2013
> > 
> > Modified Files:
> >     src/sys/kern: kern_sysctl.c
> > 
> > Log Message:
> > fix the sysctl_setup_func typedef
> 
> -typedef void (*sysctl_setup_func)(struct sysctllog **);
> +typedef void sysctl_setup_func(struct sysctllog **);
> 
> IIRC you are only supposed to be able to typedef pointers to functions.

You can use typdef for a function instead of a function pointer, but then
you can only declare args/variables as sysctl_setup_func*.

This does not look like an improvement to me.

Martin