svn commit: r230037 - head/lib/libutil

[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 structure, and also
  allowed removal of the "#ifdef _SYS_PARAM_H" guard from around the
  pidfile_* function prototypes.
  
  Suggested by pjd.

Modified:
  head/lib/libutil/libutil.h
  head/lib/libutil/pidfile.c

Modified: head/lib/libutil/libutil.h
==============================================================================
--- head/lib/libutil/libutil.h	Thu Jan 12 22:30:41 2012	(r230036)
+++ head/lib/libutil/libutil.h	Thu Jan 12 22:49:36 2012	(r230037)
@@ -48,6 +48,11 @@ typedef	__gid_t		gid_t;
 #define	_GID_T_DECLARED
 #endif
 
+#ifndef _MODE_T_DECLARED
+typedef __mode_t	mode_t;
+#define _MODE_T_DECLARED
+#endif
+
 #ifndef _PID_T_DECLARED
 typedef	__pid_t		pid_t;
 #define	_PID_T_DECLARED
@@ -73,21 +78,12 @@ typedef struct _property {
 	char	*value;
 } *properties;
 
-#ifdef _SYS_PARAM_H_
-/* for pidfile.c */
-struct pidfh {
-	int	pf_fd;
-	char	pf_path[MAXPATHLEN + 1];
-	__dev_t	pf_dev;
-	ino_t	pf_ino;
-};
-#endif
-
 /* Avoid pulling in all the include files for no need */
 struct in_addr;
 struct kinfo_file;
 struct kinfo_proc;
 struct kinfo_vmentry;
+struct pidfh;
 struct sockaddr;
 struct termios;
 struct winsize;
@@ -174,14 +170,12 @@ struct group
 int	gr_tmp(int _mdf);
 #endif
 
-#ifdef _SYS_PARAM_H_
 int	pidfile_close(struct pidfh *_pfh);
 int	pidfile_fileno(const struct pidfh *_pfh);
 struct pidfh *
 	pidfile_open(const char *_path, mode_t _mode, pid_t *_pidptr);
 int	pidfile_remove(struct pidfh *_pfh);
 int	pidfile_write(struct pidfh *_pfh);
-#endif
 
 #ifdef _UFS_UFS_QUOTA_H_
 struct fstab;

Modified: head/lib/libutil/pidfile.c
==============================================================================
--- head/lib/libutil/pidfile.c	Thu Jan 12 22:30:41 2012	(r230036)
+++ head/lib/libutil/pidfile.c	Thu Jan 12 22:49:36 2012	(r230037)
@@ -41,6 +41,13 @@ __FBSDID("$FreeBSD$");
 #include <errno.h>
 #include <libutil.h>
 
+struct pidfh {
+	int	pf_fd;
+	char	pf_path[MAXPATHLEN + 1];
+	dev_t	pf_dev;
+	ino_t	pf_ino;
+};
+
 static int _pidfile_remove(struct pidfh *pfh, int freeit);
 
 static int