Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 24 Feb 2012 15:12:23 -0500
From:      John Baldwin <jhb@freebsd.org>
To:        Bruce Evans <brde@optusnet.com.au>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r232072 - head/usr.bin/kdump
Message-ID:  <201202241512.23271.jhb@freebsd.org>
In-Reply-To: <20120225044912.Y2319@besplex.bde.org>
References:  <201202232108.q1NL8LeB023212@svn.freebsd.org> <20120225044912.Y2319@besplex.bde.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Friday, February 24, 2012 12:57:23 pm Bruce Evans wrote:
> On Thu, 23 Feb 2012, John Baldwin wrote:
> 
> > Log:
> >  Pretty-print the advice constants passed to posix_fadvise(2).
> 
> > Modified: head/usr.bin/kdump/kdump.c
> > 
==============================================================================
> > --- head/usr.bin/kdump/kdump.c	Thu Feb 23 21:07:16 2012	(r232071)
> > +++ head/usr.bin/kdump/kdump.c	Thu Feb 23 21:08:21 2012	(r232072)
> > @@ -1014,6 +1014,15 @@ ktrsyscall(struct ktr_syscall *ktr, u_in
> > 				}
> > 				capname(arg);
> > 				break;
> > +			case SYS_posix_fadvise:
> > +				print_number(ip,narg,c);
> > +				print_number(ip,narg,c);
> > +				print_number(ip,narg,c);
> > ...
> 
> Any chance of prettyprinting the sources too?  The (macro) definition
> was already uglyprinted, but kdump is fairly KNF-conformant (unlike
> truss), and doesn't use uglyprinting when invoking print_number()
> except for here.

Oops, I originally did this on 8 where it still has the wrong style.
I'll fix.

-- 
John Baldwin



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