search

MovingBlocks / Terasology

Terasology - open source voxel world
http://terasology.org
Apache License 2.0
3.64k stars 1.33k forks source link

qa: parametrize logger usages #5173

Closed jdrueckert closed 7 months ago

jdrueckert commented 7 months ago

PMD Rule Docs: https://docs.pmd-code.org/pmd-doc-6.55.0/pmd_rules_java_bestpractices.html#guardlogstatement

Theoretically fixes rule findings by using substituting parameters (parametrized logs). Practically, false-positive findings still show up due to PMD bug: https://github.com/pmd/pmd/issues/4703

jdrueckert commented 7 months ago

Addresses the remaining findings that were not part of #5170