One example of what I  was talking: evas_render_updates_internal() in
evas/sr/lib/canvas/evas_render.c:

1) You have a function of *~400 lines length*.  This is not even
warned by uncrustify.
2) Some points inside this function have an indentation of *8 levels*.
What uncrustify really does when you tell it has to "fix" the 80-col
issue is to just break lines and turn things uglier yet. Worse, it
will shut up the warning "oww... this is more than 80 cols, think
better about this code because most likely it's difficult to read"
which is the real problem here.



Lucas De Marchi

------------------------------------------------------------------------------
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

Reply via email to