Re: [bug-diffutils] Missing comma (?)

[Jim Meyering]

Santiago Vila wrote:
> While updating es.po I've noticed you have added commas everywhere
> between short and long options, so this is likely a miss:
>
> diff --git a/src/diff.c b/src/diff.c
> index fb8a2bc..6a08f36 100644
> --- a/src/diff.c
> +++ b/src/diff.c
> @@ -880,7 +880,7 @@ static char const * const option_help_msgid[] = {
>    "",
>    N_("-i, --ignore-case ignore case differences in file contents"),
>    N_("-E, --ignore-tab-expansion      ignore changes due to tab expansion"),
> - N_("-b --ignore-space-change ignore changes in the amount of white
> space"),
> + N_("-b, --ignore-space-change ignore changes in the amount of white
> space"),

Good catch.
I'll push this in your name:

>From 8832cf045d62a4a40e6df948e6cf8b85485653a0 Mon Sep 17 00:00:00 2001
From: Santiago Vila <address@hidden>
Date: Mon, 8 Aug 2011 17:59:46 +0200
Subject: [PATCH] doc: insert comma in --help line for --ignore-space-change
 (-b)

* src/diff.c (option_help_msgid): Insert omitted comma between
"-b" and "--ignore-space-change".  (tiny change)
---
 src/diff.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/diff.c b/src/diff.c
index fb8a2bc..6a08f36 100644
--- a/src/diff.c
+++ b/src/diff.c
@@ -880,7 +880,7 @@ static char const * const option_help_msgid[] = {
   "",
   N_("-i, --ignore-case               ignore case differences in file 
contents"),
   N_("-E, --ignore-tab-expansion      ignore changes due to tab expansion"),
-  N_("-b  --ignore-space-change       ignore changes in the amount of white 
space"),
+  N_("-b, --ignore-space-change       ignore changes in the amount of white 
space"),
   N_("-w, --ignore-all-space          ignore all white space"),
   N_("-B, --ignore-blank-lines        ignore changes whose lines are all 
blank"),
   N_("-I, --ignore-matching-lines=RE  ignore changes whose lines all match 
RE"),
--
1.7.4.4