Re: [PATCH 5/5] Changed argp parsing policy

[olafBuddenhagen]

Hi,

On Fri, May 29, 2009 at 12:09:04AM +0300, Sergiu Ivanov wrote:

> +/*---------------------------------------------------------------------------*/
> +#include "unionmount.h"
> +/*---------------------------------------------------------------------------*/
> +
> +/*---------------------------------------------------------------------------*/
> +/*---------Global 
> Variables--------------------------------------------------*/
> +/*The command line for starting the translator. */
> +char * trans_argz;
> +size_t trans_argz_len;
> +/*---------------------------------------------------------------------------*/
[...]

Please drop these crazy comment "lines"... I'm not going to discuss the
aesthetic value of such ornaments :-) -- but please try to keep
consistent with the style of the existing code.

Also, I don't think a comment saying "global variables" is exactly
useful -- this is really obvious to any C programmer...

-antrik-