diff options
author | Tor Norbye <tnorbye@google.com> | 2014-09-08 19:28:21 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2014-09-08 19:28:22 +0000 |
commit | 4a5b724d9fc18eb12aa59ec23c74e39399ab5c03 (patch) | |
tree | 525f975af50498b0319435e03bf52d55c9a0a1c3 /java/java-analysis-impl/src/com/intellij/codeInspection/visibility | |
parent | 2250e7e055295bee73366d360d15ea0270573ef6 (diff) | |
parent | 145ec85b4790ed9ade912d24754b38588998dee1 (diff) | |
download | idea-4a5b724d9fc18eb12aa59ec23c74e39399ab5c03.tar.gz |
Merge "Don't suggest making activities/fragments private"
Diffstat (limited to 'java/java-analysis-impl/src/com/intellij/codeInspection/visibility')
-rw-r--r-- | java/java-analysis-impl/src/com/intellij/codeInspection/visibility/VisibilityInspection.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/java/java-analysis-impl/src/com/intellij/codeInspection/visibility/VisibilityInspection.java b/java/java-analysis-impl/src/com/intellij/codeInspection/visibility/VisibilityInspection.java index fd66f2806cb9..aa772f337610 100644 --- a/java/java-analysis-impl/src/com/intellij/codeInspection/visibility/VisibilityInspection.java +++ b/java/java-analysis-impl/src/com/intellij/codeInspection/visibility/VisibilityInspection.java @@ -44,7 +44,6 @@ import com.intellij.psi.util.PsiTreeUtil; import com.intellij.psi.util.PsiUtil; import com.intellij.usageView.UsageViewTypeLocation; import com.intellij.util.IncorrectOperationException; -import com.intellij.util.VisibilityUtil; import org.jetbrains.annotations.NonNls; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; @@ -179,6 +178,10 @@ public class VisibilityInspection extends GlobalJavaBatchInspectionTool { RefClass refClass = (RefClass) refElement; if (refClass.isAnonymous() || refClass.isEntry() || refClass.isTestCase() || refClass.isServlet() || refClass.isApplet() || refClass.isLocalClass()) return null; if (isTopLevelClass(refClass) && !SUGGEST_PACKAGE_LOCAL_FOR_TOP_CLASSES) return null; + + if (refClass.isAndroidPublic()) { + return null; + } } //ignore unreferenced code. They could be a potential entry points. |