Checkstyle

[François Thillen]

Hello,

 

I just applied the Checkstyle on the wsdl2dll and I still get an error  (mvn install -Pitests,checkstyle,licenseCheck) that a “{“ is not correct but I am sure that it is correct. It is in a else statement. Does anybody have an idea to solve this?

 

Best regards

François

[Andreas Pieber]

What's the exact error? I assume you're placing it at the wrong line or miss a space, or something like this.

Kind regards,
Andreas

--
You received this message because you are subscribed to the Google Groups "OpenEngSB developer discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openengsb-de...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
 
 

[François Thillen]

Here is the message from the log:

[INFO] Starting audit...

C:\Users\frenz\openengsb\cxf-plugin\openengsb-framework\tooling\wsdl2dll\src\main\java\org\openengsb\loom\csharp\comon\wsdltodll\W

sdlToDll.java:357:13: '}' should be on the same line.

C:\Users\frenz\openengsb\cxf-plugin\openengsb-framework\tooling\wsdl2dll\src\main\java\org\openengsb\loom\csharp\comon\wsdltodll\W

sdlToDll.java:360:13: '}' should be on the same line.

Audit done.

 

And here are the corresponding lines in the code

 

355.            if (isClass) {

356.                index = matcher.start();

357.            }

358.            else if (isAbstractClass) {

359.                index = abstractMatcher.start();

360.            }

361.            else {

362.                index = inheritedMatcher.start();

363.            }

 

I do not see the problem.

 

Best regards

François

[Andreas Pieber]

Thats actually pretty easy :-)

instead of...

357.            }

358.            else if (isAbstractClass) {

359.                index = abstractMatcher.start();

360.            }

...it should be...

357.            } else if (isAbstractClass) {

359.                index = abstractMatcher.start();

360.            }

Kind regards,

Andreas