From 55f6eefe12d4864724fc4cb0584ef1426e27ff7b Mon Sep 17 00:00:00 2001
From: Martin Pool <mbp@samba.org>
Date: Wed, 19 Dec 2001 04:55:58 +0000
Subject: Doc. (This used to be commit
 5b6c22a209a26cb9adbf6d7733d396038c729633)

---
 source3/include/includes.h | 4 +++-
 source3/lib/talloc.c       | 6 ++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

(limited to 'source3')

diff --git a/source3/include/includes.h b/source3/include/includes.h
index ec4628e51f..3b2b1db877 100644
--- a/source3/include/includes.h
+++ b/source3/include/includes.h
@@ -54,8 +54,10 @@
 #endif
 #endif
 
-/* use gcc attribute to check printf fns */
 #ifdef __GNUC__
+/** Use gcc attribute to check printf fns.  a1 is the 1-based index of
+ * the parameter containing the format, and a2 the index of the first
+ * argument.  **/
 #define PRINTF_ATTRIBUTE(a1, a2) __attribute__ ((format (__printf__, a1, a2)))
 #else
 #define PRINTF_ATTRIBUTE(a1, a2)
diff --git a/source3/lib/talloc.c b/source3/lib/talloc.c
index 13393c1a69..ff1bdd0000 100644
--- a/source3/lib/talloc.c
+++ b/source3/lib/talloc.c
@@ -34,6 +34,12 @@
    TALLOC_ALIGN alignment
 */
 
+/* TODO: We could allocate both the talloc_chunk structure, and the
+ * memory it contains all in one allocation, which might be a bit
+ * faster and perhaps use less memory overhead.
+ *
+ * That smells like a premature optimization, though.  -- mbp */
+
 #include "includes.h"
 
 /** Create a new talloc context. **/
-- 
cgit