On Mon, Oct 05, 2015 at 11:50:49AM -0400, Rob Pierce wrote: > There are some offending braces. I just added leading tabs in the right > places to correct indentation. > > Rob >
why are you indenting? the point of "-offset indent" in the list/display is to do just that. jmc > Index: style.9 > =================================================================== > RCS file: /cvs/src/share/man/man9/style.9,v > retrieving revision 1.62 > diff -u -p -r1.62 style.9 > --- style.9 5 Oct 2015 01:22:34 -0000 1.62 > +++ style.9 5 Oct 2015 15:46:43 -0000 > @@ -377,10 +377,10 @@ k = !(l & FLAGS); > .Pp > Exits should be 0 on success, or non-zero for errors. > .Bd -literal -offset indent > -exit(0); /* > - * Avoid obvious comments such as > - * "Exit 0 on success." > - */ > + exit(0); /* > + * Avoid obvious comments such as > + * "Exit 0 on success." > + */ > } > .Ed > .Pp > @@ -526,11 +526,11 @@ and > family of functions. > Don't roll your own! > .Bd -literal -offset indent > -if ((four = malloc(sizeof(struct foo))) == NULL) > - err(1, NULL); > -if ((six = (int *)overflow()) == NULL) > - errx(1, "Number overflowed."); > -return (eight); > + if ((four = malloc(sizeof(struct foo))) == NULL) > + err(1, NULL); > + if ((six = (int *)overflow()) == NULL) > + errx(1, "Number overflowed."); > + return (eight); > } > .Ed > .Pp > @@ -600,8 +600,8 @@ The > .Li __progname > string may be used instead of hard-coding the program name. > .Bd -literal -offset indent > -(void)fprintf(stderr, "usage: %s [-ab]\en", __progname); > -exit(1); > + (void)fprintf(stderr, "usage: %s [-ab]\en", __progname); > + exit(1); > } > .Ed > .Pp >