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] gethostid (Linux variant): Switch to struct scratch_buffer [BZ #18023]


fweimer@redhat.com (Florian Weimer) writes:
>	[BZ #18023]
>	* sysdeps/unix/sysv/linux/gethostid.c (gethostid): Use struct
>	scratch_buffer instead of extend_alloca.  Update comments.

LGTM

A question about style, though... you have an if/else inside an else{}
block.  Would it be more readable to do if {} elseif {} else {} instead?


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