Remove how to format K&R stuff. The project hasn't been using it in
new code for a long time.
Details
Details
Diff Detail
Diff Detail
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 14579 Build 14713: arc lint + arc unit
Event Timeline
Comment Actions
Only question I have before we remove this, is the tree void of code in this format?
If it is not then I think it would be unwise to remove this,if it is then yes, absolutely, remove this.
Comment Actions
There's plenty of stuff in the tree that doesn't conform to style(9). I don't we should make compliance a prerequisite to changing bits of style(9).
Comment Actions
No new code has it.
There's odd corners of the tree that may still have it, but anybody changing it would remove the K&R style and go with full ANSI, so I'm not sure I understand how it would be useful to retain.
Comment Actions
Yes please.
All K&R function declarations are style bugs. Due to incomplete compiler support for function signature checking of K&R definitions, they can easily mask real bugs (but probably don't very much in our tree).
Comment Actions
As in any documentation it would be useful to retain as it documents the state of things. Though the style guide is mostly thought of as how one should write code, it is also one of how the code was written and is of use to someone reading the code to understand why somethings are in the form that they are. Committers should not be considered the only consumer of style.9.
Comment Actions
(sorry for the late noise. For some odd reason, today is the first day I've been able to connect to this page, although I've tried since last Thursday. I suspect there was a problem in the browser on my machine).