* lib/sgroupio.c: Avoid assignments in comparisons.

	* lib/sgroupio.c: Add brackets.
	* lib/sgroupio.c: Avoid implicit conversion of pointers / integers to booleans.
	* lib/sgroupio.c: The changed, isopen, locked, and readonly fields of the db are booleans.
	* lib/sgroupio.h, lib/sgroupio.c: sgr_file_present returns a bool.
	* lib/sgroupio.h: Add protection against multiple inclusions.

diff --git a/ChangeLog b/ChangeLog
index e141aafba00e8cb295224d3f0186b2b3f6154995..e352390e0d35d2a57615108ef69806e51a37c284 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2008-05-26  Nicolas François  <nicolas.francois@centraliens.net>
+
+       * lib/sgroupio.c: Avoid assignments in comparisons.
+       * lib/sgroupio.c: Add brackets.
+       * lib/sgroupio.c: Avoid implicit conversion of pointers / integers
+       to booleans.
+       * lib/sgroupio.c: The changed, isopen, locked, and readonly fields
+       of the db are booleans.
+       * lib/sgroupio.h, lib/sgroupio.c: sgr_file_present returns a bool.
+       * lib/sgroupio.h: Add protection against multiple inclusions.
+
 2008-05-26  Nicolas François  <nicolas.francois@centraliens.net>
 
        * lib/gshadow.c: nis_used and nis_bound are booleans.

diff --git a/lib/sgroupio.h b/lib/sgroupio.h
index 7bcb9c23635d08db376464b541958c83b23a6f0e..01954f11a30c702581c0601fbe142826a2ba1007 100644 (file)
--- a/lib/sgroupio.h
+++ b/lib/sgroupio.h
@@ -31,8 +31,12 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+/* $Id$ */
+#ifndef _SGROUPIO_H
+#define _SGROUPIO_H
+
 extern int sgr_close (void);
-extern int sgr_file_present (void);
+extern bool sgr_file_present (void);
 extern const struct sgrp *sgr_locate (const char *);
 extern int sgr_lock (void);
 extern int sgr_name (const char *);
@@ -44,3 +48,4 @@ extern int sgr_unlock (void);
 extern int sgr_update (const struct sgrp *);
 extern int sgr_sort (void);
 
+#endif