Date: Mon, 2 Jul 2001 16:00:19 -0700 (PDT) From: Dima Dorfman <dima@unixfreak.org> To: freebsd-bugs@FreeBSD.org Subject: Re: kern/28628: small indentation change to /sys/kern/kern_sysctl.c Message-ID: <200107022300.f62N0Jh64672@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
The following reply was made to PR kern/28628; it has been noted by GNATS. From: Dima Dorfman <dima@unixfreak.org> To: Giorgos Keramidas <keramida@ceid.upatras.gr> Cc: FreeBSD-gnats-submit@freebsd.org Subject: Re: kern/28628: small indentation change to /sys/kern/kern_sysctl.c Date: Mon, 02 Jul 2001 15:55:52 -0700 Giorgos Keramidas <keramida@ceid.upatras.gr> writes: > Index: kern_sysctl.c > =================================================================== > RCS file: /home/ncvs/src/sys/kern/kern_sysctl.c,v > retrieving revision 1.110 > diff -c -t -r1.110 kern_sysctl.c > *** kern_sysctl.c 2001/06/22 19:54:38 1.110 > --- kern_sysctl.c 2001/07/02 09:11:51 > *************** > *** 1027,1033 **** > if (!(oid->oid_kind & CTLFLAG_ANYBODY) && > req->newptr && req->p && > (error = suser_xxx(0, req->p, > ! (oid->oid_kind & CTLFLAG_PRISON) ? PRISON_ROOT : 0))) > return (error); > > if (!oid->oid_handler) > --- 1027,1034 ---- > if (!(oid->oid_kind & CTLFLAG_ANYBODY) && > req->newptr && req->p && > (error = suser_xxx(0, req->p, > ! (oid->oid_kind & CTLFLAG_PRISON) ? PRISON_ROOT : 0))) > ! > return (error); This change introduces two style bugs: (1) secondary indents are four, not eight spaces, and (2) lines should be <= 80 characters. Obviously, fixing (1) will fix (2) as well. That said, I don't think it makes sense to file PRs for minor style fixes. There are a lot more places in the tree where more serious violations are made, and not only would it take you a long time to find and fix all of them, I'm not sure anybody would want to commit them for you; it's just too much trouble for too little gain. Dima Dorfman dima@unixfreak.org To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-bugs" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200107022300.f62N0Jh64672>