On Fri, 21 Oct 2011 09:34:25 +0900 Daniel Juyung Seo <[email protected]>
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.

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 <[email protected]> wrote:
> > On Tue, 21 Jun 2011 13:43:13 +0200 Thomas Gstädtner
> > <[email protected]> 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
> > [email protected]
> > 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
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    [email protected]


------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to