Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#138 Module RedundantModifier should be configurable

release_3.0
closed
nobody
None
5
2012-10-10
2003-02-05
Christian Hujer
No

The module RedundantModifier should be configurable
wether to complain about a certain modifier or not.

For instance like this:
<module name="RedundantModifier">
<property name="static" value="allow" />
<property name="abstract" value="forbid" />
<property name="public" value="allow" />
</module>

Thanks

/Christian

Discussion

  • Lars Kühne
    Lars Kühne
    2003-02-20

    Logged In: YES
    user_id=401384

    I don't see why this could be useful.

    The check currently finds all redundant modifiers. Why are
    "public" modifiers OK even if they are redundant? In other
    words, why do you think "abstract" modifiers are more
    redundant than the others?

    We are trying to find a balance between checkstyle's code
    complexity and usefulnes. Right now I feel that adding this
    feature would only add complexity without being useful for
    many people, but maybe you can convince us.