Re: svn commit: r230037 - head/lib/libutil

2012-01-18 Thread Guy Helmer
On Jan 17, 2012, at 3:58 AM, Bruce Evans wrote: > On Mon, 16 Jan 2012, Pawel Jakub Dawidek wrote: > >> On Mon, Jan 16, 2012 at 11:14:32AM -0600, Guy Helmer wrote: >>> I've pasted the diff below that I think captures the majority of the issues >>> you have brought up. I have not attempted to tack

Re: svn commit: r230037 - head/lib/libutil

2012-01-17 Thread Bruce Evans
On Mon, 16 Jan 2012, Pawel Jakub Dawidek wrote: On Mon, Jan 16, 2012 at 11:14:32AM -0600, Guy Helmer wrote: I've pasted the diff below that I think captures the majority of the issues you have brought up. I have not attempted to tackle the property.3/properties.3 issues, nor the objections to

Re: svn commit: r230037 - head/lib/libutil

2012-01-17 Thread Bruce Evans
On Mon, 16 Jan 2012, Guy Helmer wrote: On Jan 14, 2012, at 3:02 PM, Bruce Evans wrote: ... I've pasted the diff below that I think captures the majority of the issues you have brought up. I have not attempted to tackle the property.3/properties.3 issues, nor the objections to the prefixes that

Re: svn commit: r230037 - head/lib/libutil

2012-01-16 Thread Pawel Jakub Dawidek
On Mon, Jan 16, 2012 at 11:14:32AM -0600, Guy Helmer wrote: > I've pasted the diff below that I think captures the majority of the issues > you have brought up. I have not attempted to tackle the > property.3/properties.3 issues, nor the objections to the prefixes that I > think would take consi

Re: svn commit: r230037 - head/lib/libutil

2012-01-16 Thread Guy Helmer
On Jan 14, 2012, at 3:02 PM, Bruce Evans wrote: > On Sat, 14 Jan 2012, Pawel Jakub Dawidek wrote: > >> On Sat, Jan 14, 2012 at 09:59:27PM +1100, Bruce Evans wrote: >>> ... >>> It's good to declare mode_t, since pidfile_open() uses it and we want >>> to remove the dependency on . However, this de

Re: svn commit: r230037 - head/lib/libutil

2012-01-14 Thread Bruce Evans
On Sat, 14 Jan 2012, Pawel Jakub Dawidek wrote: On Sat, Jan 14, 2012 at 09:59:27PM +1100, Bruce Evans wrote: ... It's good to declare mode_t, since pidfile_open() uses it and we want to remove the dependency on . However, this definition doesn't follow KNF or the style of all the other typedef

Re: svn commit: r230037 - head/lib/libutil

2012-01-14 Thread Pawel Jakub Dawidek
On Sat, Jan 14, 2012 at 09:59:27PM +1100, Bruce Evans wrote: > On Thu, 12 Jan 2012, Guy Helmer wrote: > > > Log: > > Move struct pidfh definition into pidfile.c, and leave a forward > > declaration > > for pidfh in libutil.h in its place. > > This allows us to hide the contents of the pidfh st

Re: svn commit: r230037 - head/lib/libutil

2012-01-14 Thread Bruce Evans
On Thu, 12 Jan 2012, Guy Helmer wrote: Log: Move struct pidfh definition into pidfile.c, and leave a forward declaration for pidfh in libutil.h in its place. This allows us to hide the contents of the pidfh structure, and also allowed removal of the "#ifdef _SYS_PARAM_H" guard from around th

svn commit: r230037 - head/lib/libutil

2012-01-12 Thread Guy Helmer
Author: ghelmer Date: Thu Jan 12 22:49:36 2012 New Revision: 230037 URL: http://svn.freebsd.org/changeset/base/230037 Log: Move struct pidfh definition into pidfile.c, and leave a forward declaration for pidfh in libutil.h in its place. This allows us to hide the contents of the pidfh struct