Thread: [tuxdroid-user] Coding style automation
Status: Beta
Brought to you by:
ks156
From: Philippe T. <ph...@te...> - 2007-04-15 11:15:42
|
Hi all, After some discussions with neimad, here is a proposed set of options for indent. This defines most of the current coding style of the major part of the code so if nobody objects, we'll run indent against all the current svn code, at least the C files, to fix the other parts not yet coding-style compliant. Look at the man page of indent for the option description. indent -nut -i4 -bad -bap -bbb -sob -sc -bl -cli0 -cbi4 -nce -ncdw -ss -bls -npsl -npcs -bli0 -ncs -nprs -saf -sai -saw Phil |
From: Philippe T. <ph...@te...> - 2007-04-15 11:19:49
|
I forgot also: -l80 |
From: Srikanta P. <sri...@gm...> - 2007-04-15 16:10:31
|
Perfect! Except, imho, I feel -ncdw is not needed. I feel, } while () is natural, though, I can live if it is done otherwise too :-) On 4/15/07, Philippe Teuwen <ph...@te...> wrote: > Hi all, > > After some discussions with neimad, here is a proposed set > of options for indent. > This defines most of the current coding style of the major part > of the code so if nobody objects, we'll run indent against > all the current svn code, at least the C files, to fix the other > parts not yet coding-style compliant. > > Look at the man page of indent for the option description. > > indent > -nut > -i4 > -bad > -bap > -bbb > -sob > -sc > -bl > -cli0 > -cbi4 > -nce > -ncdw > -ss > -bls > -npsl > -npcs > -bli0 > -ncs > -nprs > -saf > -sai > -saw > > Phil > > > ------------------------------------------------------------------------- > This SF.net email is sponsored by DB2 Express > Download DB2 Express C - the FREE version of DB2 express and take > control of your XML. No limits. Just data. Click to get it now. > http://sourceforge.net/powerbar/db2/ > _______________________________________________ > tux-droid-user mailing list > tux...@li... > https://lists.sourceforge.net/lists/listinfo/tux-droid-user > |
From: David B. <da...@ja...> - 2007-04-16 10:46:44
|
On Sun, 15 Apr 2007 13:15:19 +0200, Philippe Teuwen <ph...@te...> wrote: > After some discussions with neimad, here is a proposed set > of options for indent. Sounds good to me, glad also to discover that great tool :-) Now there's a tuxsetup release planned for this week (well, already planned 2 weeks ago...) and that will include all stuff that we have now on svn. I know they're testing it now, there've been some great contributions since the first beta and they want to be sure it somehow works for the newbies that are going to use tuxsetup a bit blindly. I know Rémi is having some problems now with the trunk, he told me the daemon sometimes quit. Don't know if that comes from a recent patch or a bug lying there that didn't show up before for any reason. Anybody else got that problem? As soon as he could fix that, he'll do the release. So it's probably better to wait for his tag before applying the indent changes otherwise they'll have to test everything again. I'll keep you posted with the release status in order to apply that patch. cheers, david |
From: Philippe T. <ph...@te...> - 2007-04-16 16:32:50
|
> I know Rémi is having some problems now with the trunk, he told me the > daemon sometimes quit. Don't know if that comes from a recent patch or a > bug lying there that didn't show up before for any reason. Anybody else > got that problem? As soon as he could fix that, he'll do the release. Nope. Only the ttsdaemon eating all my CPU sometimes. If Rémi manages to reproduce the bug he can post the steps so can can try out too. > So it's probably better to wait for his tag before applying the indent > changes otherwise they'll have to test everything again. I'll keep you > posted with the release status in order to apply that patch. Sir yes sir! Phil |
From: Damien <ror...@gm...> - 2007-04-16 17:32:16
|
Le lundi 16 avril 2007, "David Bourgeois" <da...@ja...> a =C3=A9crit : =20 [...] > reason. Anybody else got that problem? As soon as he could fix that, No such problem here. > So it's probably better to wait for his tag before applying the > indent changes otherwise they'll have to test everything again. I'll > keep you posted with the release status in order to apply that patch. Ack. Damien |
From: David B. <da...@ja...> - 2007-04-16 19:42:41
|
On Mon, 16 Apr 2007 19:32:08 +0200, Damien <ror...@gm...> wrote: > Le lundi 16 avril 2007, > "David Bourgeois" <da...@ja...> a écrit : > [...] >> reason. Anybody else got that problem? As soon as he could fix that, > > No such problem here. Well, didn't get it myself neither :-) I'll check again with Rémi tomorrow and push for the release. |
From: David B. <da...@ja...> - 2007-04-18 14:53:39
|
The daemon and the api have been tagged as new versions so the trunks are free again. Let's patch :D |