svn commit: samba r23289 - in branches/SAMBA_4_0: . source/cluster/ctdb/common source/include source/lib/json source/lib/util source/script

jelmer at samba.org jelmer at samba.org
Fri Jun 1 12:01:54 GMT 2007 

Author: jelmer
Date: 2007-06-01 12:01:53 +0000 (Fri, 01 Jun 2007)
New Revision: 23289

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=23289

Log:
Provide support for GCC attributes _PURE_, _NONNULL_, _DEPRECATED_, _NORETURN_ and _WARN_UNUSED_RESULT_.
Modified:
   branches/SAMBA_4_0/
   branches/SAMBA_4_0/source/cluster/ctdb/common/ctdb_util.c
   branches/SAMBA_4_0/source/include/includes.h
   branches/SAMBA_4_0/source/lib/json/debug.c
   branches/SAMBA_4_0/source/lib/json/linkhash.c
   branches/SAMBA_4_0/source/lib/util/fault.c
   branches/SAMBA_4_0/source/lib/util/util_str.c
   branches/SAMBA_4_0/source/script/mkproto.pl


Changeset:

Property changes on: branches/SAMBA_4_0
___________________________________________________________________
Name: bzr:revision-info
   - timestamp: Mon 2007-05-28 23:02:10.542000055 +0200
committer: Jelmer Vernooij <jelmer at samba.org>
properties: 
	branch-nick: SAMBA_4_0

   + timestamp: Fri 2007-06-01 13:45:21.213999987 +0200
committer: Jelmer Vernooij <jelmer at samba.org>
properties: 
	branch-nick: SAMBA_4_0

Name: bzr:revision-id-v3
   - jelmer at samba.org-20070528142122-cf5oh79k3uyecr1j
jelmer at samba.org-20070528210210-rmhpmrglmhpe9aps

   + jelmer at samba.org-20070528142122-cf5oh79k3uyecr1j
jelmer at samba.org-20070528210210-rmhpmrglmhpe9aps
jelmer at samba.org-20070601114521-2xayni5somt1iwse

Name: bzr:merge
...skipped...

Modified: branches/SAMBA_4_0/source/cluster/ctdb/common/ctdb_util.c
===================================================================
--- branches/SAMBA_4_0/source/cluster/ctdb/common/ctdb_util.c	2007-06-01 10:17:02 UTC (rev 23288)
+++ branches/SAMBA_4_0/source/cluster/ctdb/common/ctdb_util.c	2007-06-01 12:01:53 UTC (rev 23289)
@@ -52,7 +52,7 @@
 /*
   a fatal internal error occurred - no hope for recovery
 */
-void ctdb_fatal(struct ctdb_context *ctdb, const char *msg)
+_NORETURN_ void ctdb_fatal(struct ctdb_context *ctdb, const char *msg)
 {
 	DEBUG(0,("ctdb fatal error: %s\n", msg));
 	fprintf(stderr, "ctdb fatal error: '%s'\n", msg);

Modified: branches/SAMBA_4_0/source/include/includes.h
===================================================================
--- branches/SAMBA_4_0/source/include/includes.h	2007-06-01 10:17:02 UTC (rev 23288)
+++ branches/SAMBA_4_0/source/include/includes.h	2007-06-01 12:01:53 UTC (rev 23289)
@@ -62,18 +62,46 @@
 #endif
 #endif
 
-#ifndef NORETURN_ATTRIBUTE
-#if (__GNUC__ >= 3)
-#define NORETURN_ATTRIBUTE __attribute__ ((noreturn))
+#ifndef _DEPRECATED_
+#if (__GNUC__ >= 3) && (__GNUC_MINOR__ >= 1 )
+#define _DEPRECATED_ __attribute__ ((deprecated))
 #else
-#define NORETURN_ATTRIBUTE
+#define _DEPRECATED_
 #endif
 #endif
 
-/* mark smb_panic() as noreturn, so static analysers know that it is
-   used like abort */
-_PUBLIC_ void smb_panic(const char *why) NORETURN_ATTRIBUTE;
+#ifndef _WARN_UNUSED_RESULT_
+#if (__GNUC__ >= 3) && (__GNUC_MINOR__ >= 1 )
+#define _WARN_UNUSED_RESULT_ __attribute__ ((warn_unused_result))
+#else
+#define _WARN_UNUSED_RESULT_
+#endif
+#endif
 
+#ifndef _NORETURN_
+#if (__GNUC__ >= 3) && (__GNUC_MINOR__ >= 1 )
+#define _NORETURN_ __attribute__ ((noreturn))
+#else
+#define _NORETURN_
+#endif
+#endif
+
+#ifndef _PURE_
+#if (__GNUC__ >= 3) && (__GNUC_MINOR__ >= 1)
+#define _PURE_ __attribute__((pure))
+#else
+#define _PURE_
+#endif
+#endif
+
+#ifndef NONNULL
+#if (__GNUC__ >= 3) && (__GNUC_MINOR__ >= 1)
+#define NONNULL(param) param __attribute__((nonnull))
+#else
+#define NONNULL(param) param
+#endif
+#endif
+
 #include "system/time.h"
 #include "system/wait.h"
 

Modified: branches/SAMBA_4_0/source/lib/json/debug.c
===================================================================
--- branches/SAMBA_4_0/source/lib/json/debug.c	2007-06-01 10:17:02 UTC (rev 23288)
+++ branches/SAMBA_4_0/source/lib/json/debug.c	2007-06-01 12:01:53 UTC (rev 23289)
@@ -41,7 +41,7 @@
   _syslog = syslog;
 }
 
-void mc_abort(const char *msg, ...)
+_NORETURN_ void mc_abort(const char *msg, ...)
 {
   va_list ap;
   va_start(ap, msg);

Modified: branches/SAMBA_4_0/source/lib/json/linkhash.c
===================================================================
--- branches/SAMBA_4_0/source/lib/json/linkhash.c	2007-06-01 10:17:02 UTC (rev 23288)
+++ branches/SAMBA_4_0/source/lib/json/linkhash.c	2007-06-01 12:01:53 UTC (rev 23289)
@@ -20,7 +20,7 @@
 
 #include "linkhash.h"
 
-void lh_abort(const char *msg, ...)
+_NORETURN_ void lh_abort(const char *msg, ...)
 {
 	va_list ap;
 	va_start(ap, msg);

Modified: branches/SAMBA_4_0/source/lib/util/fault.c
===================================================================
--- branches/SAMBA_4_0/source/lib/util/fault.c	2007-06-01 10:17:02 UTC (rev 23288)
+++ branches/SAMBA_4_0/source/lib/util/fault.c	2007-06-01 12:01:53 UTC (rev 23289)
@@ -118,7 +118,7 @@
 /**
  Something really nasty happened - panic !
 **/
-_PUBLIC_ void smb_panic(const char *why)
+_PUBLIC_ _NORETURN_ void smb_panic(const char *why)
 {
 	int result;
 

Modified: branches/SAMBA_4_0/source/lib/util/util_str.c
===================================================================
--- branches/SAMBA_4_0/source/lib/util/util_str.c	2007-06-01 10:17:02 UTC (rev 23288)
+++ branches/SAMBA_4_0/source/lib/util/util_str.c	2007-06-01 12:01:53 UTC (rev 23289)
@@ -36,16 +36,16 @@
 /**
  Trim the specified elements off the front and back of a string.
 **/
-_PUBLIC_ BOOL trim_string(char *s,const char *front,const char *back)
+_PUBLIC_ bool trim_string(char *s, const char *front, const char *back)
 {
-	BOOL ret = False;
+	bool ret = false;
 	size_t front_len;
 	size_t back_len;
 	size_t len;
 
 	/* Ignore null or empty strings. */
 	if (!s || (s[0] == '\0'))
-		return False;
+		return false;
 
 	front_len	= front? strlen(front) : 0;
 	back_len	= back? strlen(back) : 0;
@@ -58,7 +58,7 @@
 			 * easily overlap. Found by valgrind. JRA. */
 			memmove(s, s+front_len, (len-front_len)+1);
 			len -= front_len;
-			ret=True;
+			ret=true;
 		}
 	}
 	
@@ -66,7 +66,7 @@
 		while ((len >= back_len) && strncmp(s+len-back_len,back,back_len)==0) {
 			s[len-back_len]='\0';
 			len -= back_len;
-			ret=True;
+			ret=true;
 		}
 	}
 	return ret;
@@ -75,7 +75,7 @@
 /**
  Find the number of 'c' chars in a string
 **/
-_PUBLIC_ size_t count_chars(const char *s, char c)
+_PUBLIC_ _PURE_ size_t count_chars(const char *s, char c)
 {
 	size_t count = 0;
 
@@ -218,7 +218,7 @@
 /** 
  * Parse a hex string and return a data blob. 
  */
-_PUBLIC_ DATA_BLOB strhex_to_data_blob(const char *strhex) 
+_PUBLIC_ _PURE_ DATA_BLOB strhex_to_data_blob(const char *strhex) 
 {
 	DATA_BLOB ret_blob = data_blob(NULL, strlen(strhex)/2+1);
 

Modified: branches/SAMBA_4_0/source/script/mkproto.pl
===================================================================
--- branches/SAMBA_4_0/source/script/mkproto.pl	2007-06-01 10:17:02 UTC (rev 23288)
+++ branches/SAMBA_4_0/source/script/mkproto.pl	2007-06-01 12:01:53 UTC (rev 23289)
@@ -180,7 +180,8 @@
 		}
 
 		next unless ( $is_public || $line =~ /
-			      (_DEPRECATED_ )?^(void|BOOL|bool|int|struct|char|const|\w+_[tT]\s|uint|unsigned|long|NTSTATUS|
+			      (_DEPRECATED_ |_NORETURN_ |_WARN_UNUSED_RESULT_ |_PURE_ )*^(
+				  void|BOOL|bool|int|struct|char|const|\w+_[tT]\s|uint|unsigned|long|NTSTATUS|
 				  ADS_STATUS|enum\s.*\(|DATA_BLOB|WERROR|XFILE|FILE|DIR|
 			      double|TDB_CONTEXT|TDB_DATA|TALLOC_CTX|NTTIME|FN_|init_module|
 			      GtkWidget|GType|smb_ucs2_t|krb5_error_code)
@@ -224,6 +225,10 @@
 }
 
 public("#ifndef _PUBLIC_\n#define _PUBLIC_\n#endif\n\n");
+public("#ifndef _PURE_\n#define _PURE_\n#endif\n\n");
+public("#ifndef _NORETURN_\n#define _NORETURN_\n#endif\n\n");
+public("#ifndef _DEPRECATED_\n#define _DEPRECATED_\n#endif\n\n");
+public("#ifndef _WARN_UNUSED_RESULT_\n#define _WARN_UNUSED_RESULT_\n#endif\n\n");
 
 process_file(\&public, \&private, $_) foreach (@ARGV);
 print_footer(\&public, $public_define);

More information about the samba-cvs mailing list