On Tue, May 24, 2011 at 05:07:58PM -0400, Chris Wilson wrote: > Sentry picked up this dead assignment commited yesterday in ba67aaf. > I've provided a patch to remove it. It might also be a good idea to > ask the author if that value was supposed to be used for something > in particular before pulling it out. Oops, I see others putting the patches inline. Here you go. Chris -- Chris Wilson http://vigilantsw.com/ Vigilant Software, LLC diff --git a/sh-i18n--envsubst.c b/sh-i18n--envsubst.c index 7125093..5829463 100644 --- a/sh-i18n--envsubst.c +++ b/sh-i18n--envsubst.c @@ -67,9 +67,6 @@ static void subst_from_stdin (void); int main (int argc, char *argv[]) { - /* Default values for command line options. */ - unsigned short int show_variables = 0; - switch (argc) { case 1: @@ -88,7 +85,6 @@ main (int argc, char *argv[]) /* git sh-i18n--envsubst --variables '$foo and $bar' */ if (strcmp(argv[1], "--variables")) error ("first argument must be --variables when two are given"); - show_variables = 1; print_variables (argv[2]); break; default: -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html