Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/dev/videomode



hi christos,

On Sat, Apr 09, 2011 at 04:53:39PM -0400, Christos Zoulas wrote:
> - use struct copy instead of memcpy

i haven't looked into this specific instance, but won't gcc (at least)
produce the same code for memcpy as struct copy, and the memcpy call
has the (admittedly arguable) distinction of being more readable.

> - snprintf("%s") -> strlcpy

i use snprintf("%s") since it's more portable, and avoids having to
indirect through autoconf HAVE_STRLCPY goo just to invoke
snprintf("%s") in a stub re-implementation of strlcpy(3). feel the
glibc luurrrrrrve.

regards,
al


Home | Main Index | Thread Index | Old Index