Re: Code style

2014-03-12 Thread Julian Reschke
On 2014-03-11 16:41, Felix Meschberger wrote: +1 Coding style is part of the corporate identity of the code, in a sense. Regards Felix ... I like consisting coding style, but then, I also like to be able to violate a rule in some cases. That being said: there are more things we should

Re: Code style

2014-03-11 Thread Felix Meschberger
+1 Coding style is part of the corporate identity of the code, in a sense. Regards Felix Am 04.03.2014 um 09:49 schrieb Thomas Mueller muel...@adobe.com: Hi, Question: why don't we use the maven checkstyle plugin? for what? we are quite liberal in coding style and respect personal

Re: Code style

2014-03-04 Thread Davide Giannella
On 03/03/2014 21:50, Jukka Zitting wrote: As a general rule, if you're changing existing code, stick with the existing formatting (i.e. no line in a path should be about formatting, unless the patch is explicitly about that). If introducing new code, we use the standard Java coding

Re: Code style

2014-03-04 Thread Davide Giannella
On 03/03/2014 17:20, Tobias Bocanegra wrote: Hi, On Mon, Mar 3, 2014 at 6:14 AM, Davide Giannella giannella.dav...@gmail.com wrote: I just spent some time in fixing formatting issues... what would that be? Well, spaces, line wrapping, indentation, that changed on an existing class and

Re: Code style

2014-03-04 Thread Davide Giannella
On 04/03/2014 08:49, Thomas Mueller wrote: Sure, the exact formatting rules are a matter of personal taste. But why do companies such as Google *enforce* specific rules? http://google-styleguide.googlecode.com/svn/trunk/javaguide.html Becase most of the time, developers *read* the code.

Re: Code style

2014-03-04 Thread Davide Giannella
Hi Jukka, On 04/03/2014 15:12, Jukka Zitting wrote: Hi, On Tue, Mar 4, 2014 at 8:11 AM, Davide Giannella giannella.dav...@gmail.com wrote: I raised the topic as the issue I'm working with (OAK-1263) is costing me quite some time in fixing formatting. The formatting wasn't broken to begin

Re: Code style

2014-03-04 Thread Jukka Zitting
Hi, On Tue, Mar 4, 2014 at 10:47 AM, Davide Giannella giannella.dav...@gmail.com wrote: I'm saying that with the proper tool in place this would have not happened as the build would (potentially) break if the formatting is not good. The trouble with such tooling is avoiding false positives.

Code style

2014-03-03 Thread Davide Giannella
Hello team, I just spent some time in fixing formatting issues for a pull request ;) Question: why don't we use the maven checkstyle plugin? In this way it can check the styles (we could plug it in the pedantic profile) and it could even fail the build if some exceptions occurred. As an