false positive

[LeFl0w]

j'ai identifié quelques faux positifs qui peuvent venir de bugs ou de remarques. https://sonar.linty-services.com/project/issues?resolutions=FALSE-POSITIVE&id=TEST_NEORV32

[racodond]

J'ai répondu en commentaire sur les règles où il y avait peu de violations.

Il faut qu'on voie ensemble pour les resets : https://sonar.linty-services.com/project/issues?resolutions=FALSE-POSITIVE&rules=vhdl%3AVHDL1024&id=TEST_NEORV32

https://sonar.linty-services.com/project/issues?resolutions=FALSE-POSITIVE&rules=vhdl%3AVHDL236&id=TEST_NEORV32 : Je viens de corriger dans ma branche de développement.

https://sonar.linty-services.com/project/issues?resolutions=FALSE-POSITIVE&rules=vhdl%3AVHDL1036&id=TEST_NEORV32 : Je te propose de supprimer cette règle. Il y a les cas de faux-positifs que tu as remontés. Mais il y a aussi les cas où tu initialises tes signaux avant le if et du coup tu n'as pas besoin du else. Ca fait trop de faux-positifs et peu de valeur ajoutée. Qu'est-ce que tu en penses ?

[LeFl0w]

https://sonar.linty-services.com/project/issues?resolutions=FALSE-POSITIVE&rules=vhdl%3AVHDL1036&id=TEST_NEORV32 pour cette règle il faudrait la séparer en deux :

• pour les process synchrones . Dans ce cas il faut enlever l'erreur de pas de else pour le clock event (ça éliminera beaucoup de cas)
• pour les process asynchrones een fonction on laissera les gens choisir.

par contre il faut que l'on voit pour la gestion du code non synthétisé (car sous generate) car c'est cela qui créé beaucoup de faut positif