Clément Fournier

Results 73 comments of Clément Fournier

> To be honest - this sounds very weird. I can understand, that one file can contain two languages (embedded), but the whole thing is only one (primary?) language at...

`intValue` is an instance method called on a variable called `integer` (of type `Integer`, capital `I`), declared at the start, so the lowercase is correct. I'll rename the var for...

> @oowekyala Do you have any idea about the changed violations in the regression report? It's only the rule LawOfDemeter... LawOfDemeter has been on my radar for a while... I...

This is a great idea! I have a few questions/ topics for general discussion. I'll add my comments to the kotlin ADR as review comments - What is the process...

> But I'm wondering, is there a better way to use the library of PMD to find out whether the declaration ```java private boolean isSerializable(ASTClassOrInterfaceDeclaration node) { return TypeHelper.isA(node, java.io.Serializable.class);...

This is probably because the type of the example classes used in the tests are not on the classpath. We should improve this function to look into the ExtendsList and...

Note: on master the visitor interfaces generated by Javacc are deprecated, but shouldn't be (eg [JavaParserVisitor](https://javadoc.io/doc/net.sourceforge.pmd/pmd-java/latest/net/sourceforge/pmd/lang/java/ast/JavaParserVisitor.html)) *(fixed by e1069b4a33878b7b3f49f685cc37adc2eaa50958)*

I think this can is done now. The only remaining thing is #4065

Quoting from https://www.shellscript.sh/exitcodes.html > Exit codes are a number between 0 and 255 [...]. Other numbers can be used, but these are treated modulo 256, so exit -10 is equivalent...

@ThanasisGkanos Tbh even with *while running the tool* I think the question is very vague. If it was the case that OP's looking for a progress indicator, then changing the...