
4 May
2011
4 May
'11
9:58 a.m.
Detlev Zundel wrote:
I'll start a new thread to discuss this. Hopefully we then come up with a policy to stick into that wiki page.
Now that the checkpatch policy is much more clear, I started to clean up the networking stuff, on top of which the TFTP server sits.
net/net.c alone counts 76 errors and 197 warnings. In terms of modified lines, it's going to be a big job, so I am doing it in separate patches, one per each category of errors/warnings (whitespace issues, lines >80 chars, parentheses issues etc). Is this choice ok?
Also, it's going to be a bigger job than the TFTP server itself, so I'll send a separate patchset for the cleanup work, and hold the TFTP server until the cleanup is worked out.
Luca