Community
Participate
Working Groups
It would be good to be able to configure the compiler to emit warnings on redundant modifiers (such as public on interface fields).
I don't think its a very good idea to have the compiler emit such specific warnings. I think its better if other static analysis tools such as FindBugs, etc do this. Too much fine grained analysis will unnecessary slow down the compiler even for those who don't stand to gain anything from such a warning.
Surely if you set the setting to "Ignore" then there should be no slow down. Unless you are telling me that the compiler evaluates all warnings/errors regardless of the level they are set to.
The redundant modifiers don't harm other than being redundant, and a warning for them would rarely help finding a real problem in the vicinity. I think the Clean Up / Save Action from bug 267335 would be good enough to deal with this.
(In reply to comment #3) > The redundant modifiers don't harm other than being redundant, and a warning > for them would rarely help finding a real problem in the vicinity. I think the > Clean Up / Save Action from bug 267335 would be good enough to deal with this. +1 for a clean up. Closing this as WONTFIX.
