diff mbox

[V2] doc: copy-edit text related to applying sizeof to a _Bool

Message ID 20180408085416.vbomceitg5l5672t@gmail.com (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Joey Pabalinas April 8, 2018, 8:54 a.m. UTC 
Clean up the grammar/capitalization of the -Wsizeof-bool sections and
italicize the size (1) so that it is consistent with the surrounding
text.

CC: Al Viro <viro@ZenIV.linux.org.uk>
CC: Christopher Li <sparse@chrisli.org>
CC: Joey Pabalinas <joeypabalinas@gmail.com>
CC: Luc Van Oostenryck <luc.vanoostenryck@gmail.com>
Signed-off-by: Joey Pabalinas <joeypabalinas@gmail.com>
Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com>

 evaluate.c | 2 +-
 sparse.1   | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

Comments

Luc Van Oostenryck April 8, 2018, 11:55 a.m. UTC | #1 
On Sat, Apr 07, 2018 at 10:54:16PM -1000, Joey Pabalinas wrote:
> Clean up the grammar/capitalization of the -Wsizeof-bool sections and
> italicize the size (1) so that it is consistent with the surrounding
> text.
> 

Thanks.
With the corresponding change to validation/sizeof-bool.c
(see output of 'make check'): 
Acked-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com>

Cheers,
-- Luc
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/evaluate.c b/evaluate.c
index 4e1dffe9c5416380df..9ee47c611958c7e8bf 100644
--- a/evaluate.c
+++ b/evaluate.c
@@ -2199,7 +2199,7 @@  static struct symbol *evaluate_sizeof(struct expression *expr)
 
 	if (size == 1 && is_bool_type(type)) {
 		if (Wsizeof_bool)
-			warning(expr->pos, "expression using sizeof bool");
+			warning(expr->pos, "expression using sizeof _Bool");
 		size = bits_in_char;
 	}
 
diff --git a/sparse.1 b/sparse.1
index 88343f3170f195297a..01f8869825a1dc9e37 100644
--- a/sparse.1
+++ b/sparse.1
@@ -329,7 +329,7 @@  Sparse does not issue these warnings by default.
 .B \-Wsizeof-bool
 Warn when checking the sizeof a _Bool.
 
-C99 does not specify the sizeof a _Bool.  gcc uses 1.
+C99 does not specify the size of a _Bool. GCC, by default, uses \fI1\fR.
 
 Sparse does not issue these warnings by default.
 .