Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 1 Dec 2012 13:24:36 GMT
From:      Robert Watson <rwatson@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 219871 for review
Message-ID:  <201212011324.qB1DOaeo042678@skunkworks.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://p4web.freebsd.org/@@219871?ac=10

Change 219871 by rwatson@rwatson_cinnamon on 2012/12/01 13:23:46

	Make endian.h tests more robust against -Wall -Werror.

Affected files ...

.. //depot/projects/trustedbsd/openbsm/configure#68 edit
.. //depot/projects/trustedbsd/openbsm/configure.ac#65 edit

Differences ...

==== //depot/projects/trustedbsd/openbsm/configure#68 (xtext) ====

@@ -1,5 +1,5 @@
 #! /bin/sh
-# From configure.ac P4: //depot/projects/trustedbsd/openbsm/configure.ac#63 .
+# From configure.ac P4: //depot/projects/trustedbsd/openbsm/configure.ac#64 .
 # Guess values for system-dependent variables and create Makefiles.
 # Generated by GNU Autoconf 2.69 for OpenBSM 1.2alpha3.
 #
@@ -13559,12 +13559,14 @@
 main ()
 {
 
-	bswap16(0);
-	bswap32(0);
-	bswap64(0);
+	int i;
+
+	i = bswap16(0);
+	i = bswap32(0);
+	i = bswap64(0);
 	be32enc(NULL, 0);
-	htole64(0);
-	le64toh(0);
+	i = htole64(0);
+	i = le64toh(0);
 
   ;
   return 0;

==== //depot/projects/trustedbsd/openbsm/configure.ac#65 (text+ko) ====

@@ -3,7 +3,7 @@
 
 AC_PREREQ(2.59)
 AC_INIT([OpenBSM], [1.2alpha3], [trustedbsd-audit@TrustesdBSD.org],[openbsm])
-AC_REVISION([$P4: //depot/projects/trustedbsd/openbsm/configure.ac#64 $])
+AC_REVISION([$P4: //depot/projects/trustedbsd/openbsm/configure.ac#65 $])
 AC_CONFIG_MACRO_DIR([m4])
 AC_CONFIG_SRCDIR([bin/auditreduce/auditreduce.c])
 AC_CONFIG_AUX_DIR(config)
@@ -217,12 +217,14 @@
 	#endif
 	#include <stdlib.h>
 ], [
-	bswap16(0);
-	bswap32(0);
-	bswap64(0);
+	int i;
+
+	i = bswap16(0);
+	i = bswap32(0);
+	i = bswap64(0);
 	be32enc(NULL, 0);
-	htole64(0);
-	le64toh(0);
+	i = htole64(0);
+	i = le64toh(0);
 ], [], [
 	AC_DEFINE(USE_COMPAT_ENDIAN_ENC_H,, Define if compat/endian_enc.h is required)
 	AC_MSG_RESULT([using compat/endian_enc.h])



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