bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] getopt: for ambiguous options, enumerate the possibilities.


From: Bruno Haible
Subject: Re: [PATCH] getopt: for ambiguous options, enumerate the possibilities.
Date: Sat, 4 Jun 2011 15:34:52 +0200
User-agent: KMail/1.9.9

> > One nit that emacs highlights (while reading this via Gnus)
> > is that there is a line with trailing spaces above.
> > 
> > Other than that, it looks fine.
> 
> Pushed with that fixed.

Another nit is that "gcc -Wall" gives a warning (seen on OpenBSD 4.9):

  getopt.c: In function '_getopt_internal_r':
  getopt.c:491: warning: unused variable 'ambig'

I'm fixing it like this:


2011-06-04  Bruno Haible  <address@hidden>

        getopt: Avoid gcc warning.
        * lib/getopt.c (_getopt_internal_r): Remove unused variable 'ambig'.

--- lib/getopt.c.orig   Sat Jun  4 15:31:22 2011
+++ lib/getopt.c        Sat Jun  4 15:31:19 2011
@@ -488,7 +488,6 @@
         struct option_list *next;
       } *ambig_list = NULL;
       int exact = 0;
-      int ambig = 0;
       int indfound = -1;
       int option_index;
 

-- 
In memoriam Mordechai Gebirtig <http://en.wikipedia.org/wiki/Mordechai_Gebirtig>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]