Module Name: src Committed By: apb Date: Fri Oct 25 14:54:25 UTC 2013
Modified Files: src/sys/sys: cdefs.h Log Message: Now that these attributes are documented in attribute(3), shorten the comments for __used, __unused, __diagused, and __debugused. Also s/variable/item/ because the attributes may bve applied to functions. To generate a diff of this commit: cvs rdiff -u -r1.115 -r1.116 src/sys/sys/cdefs.h Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/sys/cdefs.h diff -u src/sys/sys/cdefs.h:1.115 src/sys/sys/cdefs.h:1.116 --- src/sys/sys/cdefs.h:1.115 Fri Oct 25 08:51:55 2013 +++ src/sys/sys/cdefs.h Fri Oct 25 14:54:25 2013 @@ -1,4 +1,4 @@ -/* $NetBSD: cdefs.h,v 1.115 2013/10/25 08:51:55 apb Exp $ */ +/* $NetBSD: cdefs.h,v 1.116 2013/10/25 14:54:25 apb Exp $ */ /* * Copyright (c) 1991, 1993 @@ -251,8 +251,7 @@ #endif /* - * __unused: Note that variable might be unused. - * Do not warn if variable is unused. + * __unused: Note that item or function might be unused. */ #if __GNUC_PREREQ__(2, 7) #define __unused __attribute__((__unused__)) @@ -261,9 +260,7 @@ #endif /* - * __used: Note that variable is needed, even if it appears to be unused. - * Valid only for static variables. Do not optimise variable away. - * Do not warn if variable is unused. + * __used: Note that item is needed, even if it appears to be unused. */ #if __GNUC_PREREQ__(3, 1) #define __used __attribute__((__used__)) @@ -272,17 +269,8 @@ #endif /* - * __diagused: Note that variable is used in diagnostic code, but may be + * __diagused: Note that item is used in diagnostic code, but may be * unused in non-diagnostic code. - * - * In the kernel, variables that are used when DIAGNOSTIC is defined, - * but unused when DIAGNOSTIC is not defined, may be declared with - * __diagused. In userland, variables that are used when NDEBUG is not - * defined, but unused when NDEBUG is defined, may be declared with - * __diagused. - * - * Variables used only in assert(3) or KASSERT(9) macros are likely - * candidates for being declared with __diagused. */ #if (defined(_KERNEL) && defined(DIAGNOSTIC)) \ || (!defined(_KERNEL) && !defined(NDEBUG)) @@ -292,16 +280,8 @@ #endif /* - * __debugused: Note that variable is used in debug code, but may be + * __debugused: Note that item is used in debug code, but may be * unused in non-debug code. - * - * In either the kernel or userland, variables that are used when DEBUG - * is defined, but unused when DEBUG is not defined, may be declared with - * __debugused. - * - * In the kernel, variables used only in KDASSERT(9) macros are likely - * candidates for being declared with __debugused. There is no - * established convention for the use of DEBUG in userland code. */ #if defined(DEBUG) #define __debugused /* empty */