This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Unbound alloca in crypt routines


On Thu, Mar 29, 2012 at 1:10 PM, Roland McGrath <roland@hack.frob.com> wrote:
> The pontificating comments are inappropriate. ?The use of
> __libc_use_alloca is well established throughout the code.
> It does not require any special comment, let alone a snarky one.

I agree here.

The comments should be professional and informative.

Please update the patch.

Cheers,
Carlos.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]