Comments on: Global unused declaration inspection http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/ tips & tricks, news, how-to's Mon, 13 Feb 2012 12:27:56 +0000 http://wordpress.org/?v=2.6.5 By: Alexey Kudravtsev http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-124676 Alexey Kudravtsev Fri, 29 Jan 2010 16:17:06 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-124676 To disable the inspection for public methods, turn off the "Settings|Inspections|Declaration redundancy|Unused declaration" global inspection. To disable the inspection for public methods, turn off the “Settings|Inspections|Declaration redundancy|Unused declaration” global inspection.

]]> By: Chris http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-124666 Chris Fri, 29 Jan 2010 14:00:45 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-124666 +1 to Jan. A great feature rendered virtually useless. And there doesn't appear to be a fix in the 9.0.2 release.. +1 to Jan. A great feature rendered virtually useless. And there doesn’t appear to be a fix in the 9.0.2 release..

]]> By: n http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-123638 n Wed, 13 Jan 2010 08:42:58 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-123638 Partly agree with Jan - but with a caveat: The optimal solution would be if this was configurable so that a project could specify whether: - all public declarations will be excluded from inspection (suitable for some libraries etc, or for projects with lots of reflection or injection etc) - declarations that are tagged as unused could be marked as "ignore usage - it really is in use I say!" - suitable for projects that have some reflection/injection/other reasons - the project has a set of "entry points", e.g. a set of interfaces/classes that define the "really public" part of your library, and remaining public declarations are checked for usage. This is due to the missing modularization support in Java, with Java7 this feature would be even more useful (but require modification/extension). The last option would be extremely useful for me in my project! Partly agree with Jan - but with a caveat:

The optimal solution would be if this was configurable so that a project could specify whether:

- all public declarations will be excluded from inspection (suitable for some libraries etc, or for projects with lots of reflection or injection etc)

- declarations that are tagged as unused could be marked as “ignore usage - it really is in use I say!” - suitable for projects that have some reflection/injection/other reasons

- the project has a set of “entry points”, e.g. a set of interfaces/classes that define the “really public” part of your library, and remaining public declarations are checked for usage. This is due to the missing modularization support in Java, with Java7 this feature would be even more useful (but require modification/extension).

The last option would be extremely useful for me in my project!

]]> By: Tomasz http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-121216 Tomasz Mon, 14 Dec 2009 11:28:08 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-121216 Agreed with Jan - this feature made one of the most important inspection useless. Public methods should be excludable from the inspection! Hope Jetbrains issue a fix soon. Agreed with Jan - this feature made one of the most important inspection useless. Public methods should be excludable from the inspection!

Hope Jetbrains issue a fix soon.

]]> By: Lucas http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-120971 Lucas Wed, 09 Dec 2009 20:32:19 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-120971 I agree with you, Jan. There should be an option to disable this feature for public methods. I agree with you, Jan. There should be an option to disable this feature for public methods.

]]> By: Andreas http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-120017 Andreas Mon, 23 Nov 2009 21:16:44 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-120017 +1 to Jan. There should be an option to disable public declarations, but to keep it for other scopes. +1 to Jan. There should be an option to disable public declarations, but to keep it for other scopes.

]]> By: Osvaldas Grigas http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-119655 Osvaldas Grigas Tue, 17 Nov 2009 09:34:22 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-119655 I second Jan. There are many public methods in our project that are invoked through reflection, e.g. through Spring XML configuration. So now I see warnings everywhere, which is frustrating because I know these methods are used. I could turn \Unused declarations\ inspection off, but then I would also lose notifications of unused private methods, and that would be a shame. I second Jan. There are many public methods in our project that are invoked through reflection, e.g. through Spring XML configuration. So now I see warnings everywhere, which is frustrating because I know these methods are used. I could turn \Unused declarations\ inspection off, but then I would also lose notifications of unused private methods, and that would be a shame.

]]> By: Ran Biron http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-117645 Ran Biron Thu, 15 Oct 2009 19:25:29 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-117645 What about recursive global unused declarations? possibly with "remove entire declaration tree" action? What about recursive global unused declarations? possibly with “remove entire declaration tree” action?

]]> By: Jan http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-117064 Jan Wed, 07 Oct 2009 11:36:36 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-117064 Is there a way to disable this inspection for public methods? I still like the inspection to warn me about unused private/protected/package_private methods, but not for public methods. The problem with this feature is that it doesn't work well if you are developing an API of some sort, on which other projects (outside the scope of the currently opened IDEA project) depend. Don't get me wrong, I like this feature, but I would also like to configure it differently for public and non-public methods. Is there a way to disable this inspection for public methods? I still like the inspection to warn me about unused private/protected/package_private methods, but not for public methods.

The problem with this feature is that it doesn’t work well if you are developing an API of some sort, on which other projects (outside the scope of the currently opened IDEA project) depend.

Don’t get me wrong, I like this feature, but I would also like to configure it differently for public and non-public methods.

]]> By: James http://blogs.jetbrains.com/idea/2009/04/global-unused-declaration-inspection/#comment-112794 James Fri, 10 Jul 2009 13:52:47 +0000 http://blogs.jetbrains.com/idea/?p=775#comment-112794 Very exciting news! I can't wait to run it against our diseased codebase :) Very exciting news! I can’t wait to run it against our diseased codebase :)

]]>