summaryrefslogtreecommitdiff
path: root/java/java-analysis-impl/src/com/intellij/codeInspection/visibility
diff options
context:
space:
mode:
authorTor Norbye <tnorbye@google.com>2014-09-08 19:28:21 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2014-09-08 19:28:22 +0000
commit4a5b724d9fc18eb12aa59ec23c74e39399ab5c03 (patch)
tree525f975af50498b0319435e03bf52d55c9a0a1c3 /java/java-analysis-impl/src/com/intellij/codeInspection/visibility
parent2250e7e055295bee73366d360d15ea0270573ef6 (diff)
parent145ec85b4790ed9ade912d24754b38588998dee1 (diff)
downloadidea-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.java5
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.