Want to enforce space between class and brace

[Andreas Jonsson]

Hi,

With my current Checkstyle configuration (very similar to Google's) i'm unable to catch the following case:

public class TestClass extends SomeClass{

but it warns when i do (Whitespace Around: WhitespaceAround: '{' is not preceded with whitespace.):

public String toString(){

Is this a bug or what configuration am I missing?

Regards

Andreas

[Roman Ivanov]

Hi Andreas,

You need to share exact file and exact config and exact version checkstyle.

Here is your case, all works as expected:

04:46 /var/tmp $ java -jar checkstyle-6.11.2-all.jar -c 3.xml 3.java

Starting audit...

/var/tmp/3.java:1:41: error: '{' is not preceded with whitespace.

/var/tmp/3.java:3:25: error: '{' is not preceded with whitespace.

Audit done.

Checkstyle ends with 2 errors.

04:46 /var/tmp $ cat 3.xml 

<?xml version="1.0"?>

<!DOCTYPE module PUBLIC

        "-//Puppy Crawl//DTD Check Configuration 1.2//EN"

        "http://jee-qa/jee-qa/trunk/checkstyle/configuration_1_2.dtd">

<module name="Checker">

    <module name="TreeWalker">

        <module name="WhitespaceAround" />

    </module>

</module>

04:46 /var/tmp $ cat 3.java 

public class TestClass extends SomeClass{

public String toString(){

}

}

04:46 /var/tmp $ 

thanks,

Roman Ivanov

​

[Andreas Jonsson]

Hi Roman,

Thanks for your answer. We are using 6.11.2, here's my code & configuration (attached):

public class CheckstyleTest{ // Doesn't warn - incorrect

 private static final int SOMETHING = 1;

 public CheckstyleTest(){ //warns here - correct

 System.out.println("CheckstyleTest " + SOMETHING);

 }


 public void test1() { // Doesn't warn - correct

 if (3 > 2) { // Doesn't warn - correct

 System.out.println("test1");

 }

 }


 public void test2(){ //warns here - correct

 if (3 > 2){ //warns here - correct

 System.out.println("test2");

 }

 }


}

Looking forward to your answer,

Andreas

[Roman Ivanov]

https://github.com/checkstyle/checkstyle/issues/2480

--

You received this message because you are subscribed to the Google Groups "checkstyle" group.
To unsubscribe from this group and stop receiving emails from it, send an email to checkstyle+...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.