summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorPádraig Brady <P@draigBrady.com>2011-01-07 01:25:06 +0000
committerPádraig Brady <P@draigBrady.com>2011-01-07 11:07:20 +0000
commitd0270e7b630dd9c27e70ef46cc3ce5dac70dd574 (patch)
tree13c24980428e47f696d4aec8b19b6a31d1831f58 /lib
parent939dedd9a2fba934e5ff0313155f9f5f39cb6f6f (diff)
downloadgnulib-d0270e7b630dd9c27e70ef46cc3ce5dac70dd574.tar.gz
ignore-value: clarify some comments
* lib/ignore-value.h: Add Eric Blake as an author since he rewrote the macros. (ignore_value): Specify exactly when the GCC warn_unused_result feature became available. (ignore_ptr): Mention that ignore_value supports aggregates.
Diffstat (limited to 'lib')
-rw-r--r--lib/ignore-value.h10
1 files changed, 7 insertions, 3 deletions
diff --git a/lib/ignore-value.h b/lib/ignore-value.h
index 52804dc23b..7fd4772f01 100644
--- a/lib/ignore-value.h
+++ b/lib/ignore-value.h
@@ -15,7 +15,7 @@
You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
-/* Written by Jim Meyering and Pádraig Brady. */
+/* Written by Jim Meyering, Eric Blake and Pádraig Brady. */
/* Use "ignore_value" to avoid a warning when using a function declared with
gcc's warn_unused_result attribute, but for which you really do want to
@@ -47,13 +47,17 @@
# endif
# endif
-# if __GNUC__ < 3 || (__GNUC__ == 3 && __GNUC_MINOR__ < 1)
+/* The __attribute__((__warn_unused_result__)) feature
+ is available in gcc versions 3.4 and newer,
+ while the typeof feature has been available since 2.7 at least. */
+# if __GNUC__ < 3 || (__GNUC__ == 3 && __GNUC_MINOR__ < 4)
# define ignore_value(x) ((void) (x))
# else
# define ignore_value(x) (({ __typeof__ (x) __x = (x); (void) __x; }))
# endif
-/* ignore_value works for both scalars and pointers; deprecate ignore_ptr. */
+/* ignore_value works for scalars, pointers and aggregates;
+ deprecate ignore_ptr. */
static inline void ATTRIBUTE_DEPRECATED
ignore_ptr (void *p) { (void) p; } /* deprecated: use ignore_value */