threads / patch / 37044

patchenums: remove trailing ',' after last item in enum

Subject: [PATCH] enums: remove trailing ',' after last item in enum

## tl;dr

4 messages between Jul 2, 2014 and Jul 9, 2014. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Ronnie Sahlberg· Jul 2, 2014, 18:24 UTC · lore
Signed-off-by: Ronnie Sahlberg <sahlberg@google.com>
---
 builtin/clean.c | 2 +-
 builtin/tag.c   | 2 +-
 pretty.c        | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
Show changes to 3 files +3 −3

builtin/clean.c, builtin/tag.c, pretty.c

diff --git a/builtin/clean.c b/builtin/clean.c
index 9a91515..27701d2 100644
--- a/builtin/clean.c
+++ b/builtin/clean.c
@@ -48,7 +48,7 @@ enum color_clean {
 	CLEAN_COLOR_PROMPT = 2,
 	CLEAN_COLOR_HEADER = 3,
 	CLEAN_COLOR_HELP = 4,
-	CLEAN_COLOR_ERROR = 5,
+	CLEAN_COLOR_ERROR = 5
 };
 
 #define MENU_OPTS_SINGLETON		01
diff --git a/builtin/tag.c b/builtin/tag.c
index c6e8a71..ef76556 100644
--- a/builtin/tag.c
+++ b/builtin/tag.c
@@ -83,7 +83,7 @@ static int in_commit_list(const struct commit_list *want, struct commit *c)
 enum contains_result {
 	CONTAINS_UNKNOWN = -1,
 	CONTAINS_NO = 0,
-	CONTAINS_YES = 1,
+	CONTAINS_YES = 1
 };
 
 /*
diff --git a/pretty.c b/pretty.c
index 4f51287..924bc61 100644
--- a/pretty.c
+++ b/pretty.c
@@ -274,7 +274,7 @@ static void add_rfc822_quoted(struct strbuf *out, const char *s, int len)
 
 enum rfc2047_type {
 	RFC2047_SUBJECT,
-	RFC2047_ADDRESS,
+	RFC2047_ADDRESS
 };
 
 static int is_rfc2047_special(char ch, enum rfc2047_type type)
-- 
2.0.1.442.g7fe6834.dirty
Jonathan Nieder· Jul 2, 2014, 19:58 UTC · re: Ronnie Sahlberg · lore

Re: [PATCH] enums: remove trailing ',' after last item in enum

Ronnie Sahlberg wrote:
Show 6 quoted lines
> Signed-off-by: Ronnie Sahlberg <sahlberg@google.com>
> ---
>  builtin/clean.c | 2 +-
>  builtin/tag.c   | 2 +-
>  pretty.c        | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)

Is there some gcc option or other tool that can automatically detect this kind of problem so the regress/fix cycle doesn't have to repeat too many times?

Looks like v1.7.2-rc0~32^2~16 (2010-03-14) and v1.7.4.2~34 (2011-03-16) tried to fix this in the past.

Using the test from v1.7.4.2~34 also finds enums with trailing comma in

 grep.h
 log-tree.c

Thanks, Jonathan

Ronnie Sahlberg· Jul 2, 2014, 20:08 UTC · re: Jonathan Nieder · lore

Re: [PATCH] enums: remove trailing ',' after last item in enum

GCC can check/error for this with
--pedantic -Werror
On Wed, Jul 2, 2014 at 12:58 PM, Jonathan Nieder <jrnieder@gmail.com> wrote:
Show 24 quoted lines
> Ronnie Sahlberg wrote:
>
>> Signed-off-by: Ronnie Sahlberg <sahlberg@google.com>
>> ---
>>  builtin/clean.c | 2 +-
>>  builtin/tag.c   | 2 +-
>>  pretty.c        | 2 +-
>>  3 files changed, 3 insertions(+), 3 deletions(-)
>
> Is there some gcc option or other tool that can automatically detect
> this kind of problem so the regress/fix cycle doesn't have to repeat
> too many times?
>
> Looks like v1.7.2-rc0~32^2~16 (2010-03-14) and v1.7.4.2~34 (2011-03-16)
> tried to fix this in the past.
>
> Using the test from v1.7.4.2~34 also finds enums with trailing comma
> in
>
>  grep.h
>  log-tree.c
>
> Thanks,
> Jonathan
Junio C Hamano· Jul 9, 2014, 14:59 UTC · re: Ronnie Sahlberg · lore

Re: [PATCH] enums: remove trailing ',' after last item in enum

Ronnie Sahlberg <sahlberg@google.com> writes:
> Signed-off-by: Ronnie Sahlberg <sahlberg@google.com>
> ---
Looks good; thanks.
Show 44 quoted lines
>  builtin/clean.c | 2 +-
>  builtin/tag.c   | 2 +-
>  pretty.c        | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/builtin/clean.c b/builtin/clean.c
> index 9a91515..27701d2 100644
> --- a/builtin/clean.c
> +++ b/builtin/clean.c
> @@ -48,7 +48,7 @@ enum color_clean {
>  	CLEAN_COLOR_PROMPT = 2,
>  	CLEAN_COLOR_HEADER = 3,
>  	CLEAN_COLOR_HELP = 4,
> -	CLEAN_COLOR_ERROR = 5,
> +	CLEAN_COLOR_ERROR = 5
>  };
>  
>  #define MENU_OPTS_SINGLETON		01
> diff --git a/builtin/tag.c b/builtin/tag.c
> index c6e8a71..ef76556 100644
> --- a/builtin/tag.c
> +++ b/builtin/tag.c
> @@ -83,7 +83,7 @@ static int in_commit_list(const struct commit_list *want, struct commit *c)
>  enum contains_result {
>  	CONTAINS_UNKNOWN = -1,
>  	CONTAINS_NO = 0,
> -	CONTAINS_YES = 1,
> +	CONTAINS_YES = 1
>  };
>  
>  /*
> diff --git a/pretty.c b/pretty.c
> index 4f51287..924bc61 100644
> --- a/pretty.c
> +++ b/pretty.c
> @@ -274,7 +274,7 @@ static void add_rfc822_quoted(struct strbuf *out, const char *s, int len)
>  
>  enum rfc2047_type {
>  	RFC2047_SUBJECT,
> -	RFC2047_ADDRESS,
> +	RFC2047_ADDRESS
>  };
>  
>  static int is_rfc2047_special(char ch, enum rfc2047_type type)

← back to recent threads