[PATCH] mkpasswd.c - allows selection of specific user
Mark Bradshaw
bradshaw@staff.crosswalk.com
Mon Oct 15 16:36:00 GMT 2001
Ah. I see what you mean. I've always done ifs like that, and never
considered that the braces would be a "thing". If you or Corinna want, I'll
redo em and resubmit. 15th times the charm...
Mark
> -----Original Message-----
> From: Christopher Faylor [mailto:cgf@redhat.com]
> Sent: Tuesday, November 20, 2001 10:35 PM
> To: cygwin-patches@cygwin.com
> Subject: Re: [PATCH] mkpasswd.c - allows selection of specific user
>
>
> The code looks good to me, but it seems like you you're using
> K&R formatting rather than GNU formatting, i.e., the curly
> braces don't match the rest of the code.
>
> This is pretty minor, and normally I would just apply the
> patch, fix the couple of formatting glitches, and check this
> in, however, since mkpasswd.c is sort of owned by Corinna,
> I'll let her have final approval.
>
> Btw, the ChangeLog entry looks fine.
>
> Thanks for this patch. I'm looking forward to getting it
> into the main distribution.
>
> cgf
More information about the Cygwin-patches
mailing list