Hi Niklaus,
On Monday 25 June 2007, Niklaus Giger wrote:
The only lines which I exceed 80 characters are lines that my patch does not touch or removes them. Is it my duty to fix all lines which are too long in a source file? Or did I miss a line?
I'm not talking about too long coding lines, but the line wrap most likely caused by your mail client. For example the sequoia.h file (even if removed) has some line wraps which case that the patch can't be applied clean.
Does the limit of 80 characters also apply to *.h-files? If yes, we have about 485 lines to fix in include/ppc*.h.
Normally the 80 characters line length also apply the the headers (all source code files). But you don't have to clean up existing code. Please just make sure that your mail client doesn't warp the lines upon sending.
Hint: Please send the mail first to yourself and try to apply the patch to your repository. If this works without any problems you can send it to the list.
Thanks.
Best regards, Stefan
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office@denx.de =====================================================================