StevenLooman / magik-tools

Sonar Magik plugin
GNU General Public License v3.0
18 stars 10 forks source link

When setting UnusedVariableCheck.checkParameters to true, abstract methods are checked as well #123

Closed sebastiaanspeck closed 1 week ago

sebastiaanspeck commented 1 week ago
_abstract _method a.b(p_param1)
_endmethod

This snippet raises an issue on p_param1 that it is unused, but an abstract method will always be empty. IMO it is a good habit to show in the abstract method which parameters the subclass needs to implement.

122 implements this behaviour.

StevenLooman commented 1 week ago

Agreed.