Le 28/07/15 19:54, Emmanuel Lécharny a écrit : > >> Beside checkstyle we should also add PMD check, you already started to >> cleanup some issues. > PMD is a bit a PITA. There are many so called violations that are just > plain BS, and many others that are going to push us to rewrite the whole > code... > > This wil need some serious tuning.
FTR, I ran PMD on one plugin (aciitemeditor) this morning before breakfast, it gave me 1550 lines of violations... Out of which we have : 288 times : Parameter 'xxx' is not assigned and could be declared final 61 times : Comment is too large: Too many lines (this is mainly because of the AL 2.0 license header) etc... in any case, there will be hundreds of things to fix. Perfect when you have long insomnias or when at in the smaller room of your appartment ;-)
