Peter Paul Bakker

Results 44 issues of Peter Paul Bakker

**Affects PMD Version:** 7.0.x ~~(maybe also 6)~~ Make sure, to test with the latest PMD version. **Description:** During the development of `sonar-pmd` plugin, the integration tests that uses the `sonar-orchestrator`...

a:bug

The Kotlin PR is merged into pmd-7.x branch of pmd, so see if we can compile and run with latest-greatest pmd 7.0.0-SNAPSHOT

Current used version is 3.35.1.2719, latest version is [3.38.0.115](https://github.com/SonarSource/orchestrator/releases/tag/3.38.0.115) Might also solves some transitive vulnerable dependencies (test code only).

add github actions to build and release the sonar-pmd plugin

First addition. Question: if `XPathAPI.eval(doc, "//...")` is used: is also bad? The test case is now only `XPathAPI.eval(doc, "/...")` is _not_ bad.

Works on basic unit tests. Should we create more unit tests to avoid false positives and add more Kotlin syntaxes?

As decribed in the async section: https://github.com/jborgers/PMD-jPinpoint-rules/blob/master/docs/JavaCodePerformance.md#improper-asynchrony Can we make rules to check on new ThreadPoolTaskExector() without sizes being set (default constructor is effectively a single threaded pool). We can...

This is with latest pmd-7.0.0-SNAPSHOT build with Kotlin support: For this code: fun getSomething(): Something { return Something(Other.create()) } The Kotlin HttpClientBuilderWithoutDisableConnectionState fails with XPathException: Effective boolean value is not...

The Spring @Advice, used in AOP, creates one instance per default and fields in shared instances should be thread safe like a regular @Component. As the docs explain: "By default...

Both AvoidExpressionsInCacheable and SynchronizeForKeyInCacheable point to wrong issue when right clicking on "details": JavaCodePerformance.md#TMSU12 Seem docs for these items are missing on the documentation page?