On Thu, 29 Jul 2010 15:53:15 -0300 Lucas De Marchi <lucas.demar...@profusion.mobi> said:
> On Wed, Jul 28, 2010 at 6:36 PM, Carsten Haitzler <ras...@rasterman.com> > wrote: > > looks like EINA_ARG_NONNULL(), EINA_MALLOC etc. and friends really mess up > > the formatter - they can't make sense of it as its a func del with yet > > another fucn behind it and stuff before a ; which isn't actually normal to > > see in c. > > > > I think that if we don't be so strict about line width, we can > work-around this. Also, sometimes it's really bad to break lines when > you can have a much more readable source code with, let's say 85 > chars. Not desirable, right, but it's not the worst thing. > > Maybe we can just emit warnings about line width, as check-patch from > kernel guys does. then you wrap at 85.. then you say "but if we made it a little wider it'd be nicer" so it become 90, then "if its a bit wider" 95.. and so on. it doesn't end until you cease wrapping entirely. i think trying to stick to 80 wide is good. it's the standard term width. it's not a magic number invented for efl coding. uncrustify needs to be able to handle the above case properly. the only question is... how to do it? -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ The Palm PDK Hot Apps Program offers developers who use the Plug-In Development Kit to bring their C/C++ apps to Palm for a share of $1 Million in cash or HP Products. Visit us here for more details: http://p.sf.net/sfu/dev2dev-palm _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
