[OpenAFS-devel] [PATCH] flexelint: punctuation fixes

Jim Rees rees@umich.edu
Mon, 01 Dec 2003 11:32:38 -0500


We actually do have a coding standard, although I don't think it is
documented anywhere except the cvs logs and the mailing list archive.  Maybe
someone will be good enough to add it to the wiki if it's not already there.
This is our coding standard:

gnu indent 2.2.9 options:
-npro -nbad -bap -nbc -bbo -br -ce -cdw -brs -ncdb -cp1 -ncs -di2 -ndj -nfc1
-nfca -i4 -lp -npcs -nprs -psl -sc -nsob -ts8

There is nothing there about semicolon on a one-line for/while, and the only
option gindent gives you is to put a blank before the semicolon.  We are not
using this option.

I think it's entirely appropriate to discuss what we want to do about this
case, and if we can reach agreement, fix the source.  But in the absence of
agreement we probably should leave the existing code alone.