From: Jakub Narebski Date: Thu, 05 Aug 2010 12:16:38 GMT Subject: Re: [PATCH 1/5] diff: parse separate options like -S foo Message-ID: In-Reply-To: <1280996576-1165-1-git-send-email-Matthieu.Moy@imag.fr> Matthieu Moy writes: > +static inline int short_opt(char opt, const char **argv, > + const char **optarg) [...] > +int parse_long_opt(const char *opt, const char **argv, > + const char **optarg) Just a nitpick question: why it is 'parse_long_opt' but just 'short_opt'? -- Jakub Narebski Poland ShadeHawk on #git