On 06/15/2011 12:36 AM, Carsten Haitzler (The Rasterman) wrote: > uncrustify solves all the formatting issues, we just need to apply it, but to > apply it we need to have no pending work, patches or branches as it basically > goes and edits pretty much every line of code.
While having consistently formatted code is nice, I don't think using a tool to automatically reformat all the code is worth it. It will break "blame" functionality of revision control and as you point out, break any pending patches. If there's something really ugly, then maybe it's worth identifying and fixing, but otherwise, fixing as you go is good. Commit hooks can be used for pointing out the problem, but shouldn't block commits. thanks, Mike ------------------------------------------------------------------------------ EditLive Enterprise is the world's most technically advanced content authoring tool. Experience the power of Track Changes, Inline Image Editing and ensure content is compliant with Accessibility Checking. http://p.sf.net/sfu/ephox-dev2dev _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
