On 1/8/07, Charles Manning <[EMAIL PROTECTED]> wrote: > On Sunday 07 January 2007 23:56, David Anderson wrote: > > Here is my first patch to the nxt platform code. Only indentation and > > formatting changes, no logical change. > > > > As the patch file is rather large, yahoo's mailman blocked it. I put > > the patch online at > > http://atlas.natulte.net/~dave/nxt_platform_indent.patch . > > It might just be easier for me to run indent and put the code back in svn
Of course. I've been meaning to say that the above patch was submitted when tired, and is obviously slightly insane. The commandline that formats the code according to the style I've seen in most of the code is: indent -bad -bap -bbb -sc -br -ce -cdw -npcs -cs -nprs -di2 -nbc -psl -T U8 -T S8 -T U16 -T S16 -T U32 -T S32 -brs -i2 -lp -ppi2 --line-length78 *.c *.h That produces strange output in uart.c, because of the two incomplete statements 'if (up->' and 'v v v' that get excluded at compile time, but still confuse indent. If you comment these out and run indent again, the output is sane. Let me know once you've submitted the reindentation, and I'll start working on smaller scale documentation tweaks. - Dave ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Lejos-discussion mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/lejos-discussion
