On Thu, 20 Oct 2011 23:08:11 -0400 Mike Blumenkrantz <m...@zentific.com> said:
> On Fri, 21 Oct 2011 11:58:33 +0900 > Carsten Haitzler (The Rasterman) <ras...@rasterman.com> wrote: > > > On Fri, 21 Oct 2011 09:34:25 +0900 Daniel Juyung Seo <seojuyu...@gmail.com> > > said: > > > > > I raise this issue again. > > > We need pre/post-commit hook for formatting, whitespaces, and whatever. > > > webkit already does this job. > > > > that assumes you have something that can either 1. enforce formatting rules > > properly everywhere (and then compare post-commit code as-is vs post-commit > > code with formatting enforced and reject code based on that - or commit > > updated formatted version). or 2. a script/tool that can detect all your > > formatting wants. > > > > we have neither right now. we are CLOSE to having 1. mike worked a lot on > > ecrustify (uncrustify with changes), but last i threw it at some code it > > made a dogs breakfast of some things. > ecrustify has issues with function pointers and some comments. I think it > works perfectly aside from that, so if someone wants to try figuring that out > then that's the place to start. the ecore headers are botched because of an ecrustify run :( we may have to manually fix them. one problem is that ecrustify last i tried,m if u ran it once formatted to format A then if u ran it again on the result.. u got something different (format B). i need to poke at it again. but fn pointers inside function prototypes it definitely made a dogs breakfast of. > > if you want to fix formatting like above - you need an automated formatter > > that works for everything we have. get that working and we can talk :) > > > > > Daniel Juyung Seo (SeoZ) > > > > > > On Tue, Jun 21, 2011 at 10:00 PM, David Seikel <onef...@gmail.com> wrote: > > > > On Tue, 21 Jun 2011 13:43:13 +0200 Thomas Gstädtner > > > > <tho...@gstaedtner.net> wrote: > > > > > > > >> That is true, but the "Don't do that, nasty programmer. SPANK SPANK > > > >> SPANK" mail tells you that the hook triggered and also it hurts the > > > >> programmer who did produce the trailing whitespaces. People learn best > > > >> when it hurts :P > > > > > > > > Automated emails might just go to spam, don't hurt that much. Unless > > > > you intend to not accept the commit and make them do it over. That > > > > will hurt enough. > > > > > > > > -- > > > > A big old stinking pile of genius that no one wants > > > > coz there are too many silver coated monkeys in the world. > > > > > > > > ------------------------------------------------------------------------------ > > > > 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 > > > > > > > > > > > > > > ------------------------------------------------------------------------------ > > > The demand for IT networking professionals continues to grow, and the > > > demand for specialized networking skills is growing even more rapidly. > > > Take a complimentary Learning@Cisco Self-Assessment and learn > > > about Cisco certifications, training, and career opportunities. > > > http://p.sf.net/sfu/cisco-dev2dev > > > _______________________________________________ > > > enlightenment-devel mailing list > > > enlightenment-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > > > IMO starting with just whitespace stripping would be good. > > -- > Mike Blumenkrantz > Zentific: Doctor recommended, mother approved. > -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ The demand for IT networking professionals continues to grow, and the demand for specialized networking skills is growing even more rapidly. Take a complimentary Learning@Cisco Self-Assessment and learn about Cisco certifications, training, and career opportunities. http://p.sf.net/sfu/cisco-dev2dev _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel