Re: [vfs] Lcurly=no (was: svn commit: r1802440 - /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/jar/test/JarAttributesTestCase.java

2017-07-23 Thread Gary Gregory
om> >> Sent: Thursday, July 20, 2017 2:52:55 AM >> To: Commons Developers List >> Subject: Re: [vfs] Lcurly=no (was: svn commit: r1802440 - >> /commons/proper/vfs/trunk/core/src/test/java/org/apache/comm >> ons/vfs2/provider/jar/test/JarAttributesTestCase.java >> &

Re: [vfs] Lcurly=no (was: svn commit: r1802440 - /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/jar/test/JarAttributesTestCase.java

2017-07-20 Thread Gary Gregory
: Commons Developers List > Subject: Re: [vfs] Lcurly=no (was: svn commit: r1802440 - > /commons/proper/vfs/trunk/core/src/test/java/org/apache/ > commons/vfs2/provider/jar/test/JarAttributesTestCase.java > > We had talked a long time ago about changing the style to the {} style > bel

Re: [vfs] Lcurly=no (was: svn commit: r1802440 - /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/jar/test/JarAttributesTestCase.java

2017-07-20 Thread Bernd Eckenfels
: Gary Gregory <garydgreg...@gmail.com> Sent: Thursday, July 20, 2017 2:52:55 AM To: Commons Developers List Subject: Re: [vfs] Lcurly=no (was: svn commit: r1802440 - /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/jar/test/JarAttributesTestCase.java We had talked

Re: [vfs] Lcurly=no (was: svn commit: r1802440 - /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/jar/test/JarAttributesTestCase.java

2017-07-19 Thread Gary Gregory
We had talked a long time ago about changing the style to the {} style below but it was never done. Any IDE can automatically do that but I do not know how to tell Eclipse for example to ONLY do that. So if we can live we a tree-wide reformat, that's easy. Gary On Wed, Jul 19, 2017 at 5:15 PM,

[vfs] Lcurly=no (was: svn commit: r1802440 - /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/jar/test/JarAttributesTestCase.java

2017-07-19 Thread Bernd Eckenfels
Hello. I am not sure about changing that. It looks to me like most of the VFS code uses opening braces on new lines. I actually agree with that style. I would also accept to change it, but then completely. In fact the checkstyle template specifies curly = nl