On 7/31/16 14:36, Pedro F. Giffuni wrote: > Author: pfg > Date: Sun Jul 31 21:36:40 2016 > New Revision: 303600 > URL: https://svnweb.freebsd.org/changeset/base/303600 > > Log: > indent(1): replace function call to bzero with memset. > > Reference: > > https://github.com/pstef/freebsd_indent/commit/7422f42f80099c69d34833d7106035dc09230235 > > Differential Revision: https://reviews.freebsd.org/D6966 (Partial) > Submitted by: Piotr Stefaniak > > Modified: > head/usr.bin/indent/io.c > > Modified: head/usr.bin/indent/io.c > ============================================================================== > --- head/usr.bin/indent/io.c Sun Jul 31 21:29:10 2016 (r303599) > +++ head/usr.bin/indent/io.c Sun Jul 31 21:36:40 2016 (r303600) > @@ -629,7 +629,7 @@ parsefont(struct fstate *f, const char * > const char *s = s0; > int sizedelta = 0; > > - bzero(f, sizeof *f); > + memset(f, 0, sizeof(struct fstate)); ^^^^^^^^^^^^^^^^^^^^^ This is much more error-prone than sizeof(*f) IMHO.
Cheers,
signature.asc
Description: OpenPGP digital signature
