From 30d8bf5fea53bc3c0e9fe1d57cdb4854ed525aac Mon Sep 17 00:00:00 2001 From: Bas Leijdekkers Date: Sat, 14 Apr 2012 14:56:42 +0200 Subject: [PATCH] do not offer to make top level classes private --- .../src/com/siyeh/ipp/modifiers/ModifierPredicate.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/plugins/IntentionPowerPak/src/com/siyeh/ipp/modifiers/ModifierPredicate.java b/plugins/IntentionPowerPak/src/com/siyeh/ipp/modifiers/ModifierPredicate.java index 975743ce29a3..dea48f76c821 100644 --- a/plugins/IntentionPowerPak/src/com/siyeh/ipp/modifiers/ModifierPredicate.java +++ b/plugins/IntentionPowerPak/src/com/siyeh/ipp/modifiers/ModifierPredicate.java @@ -18,6 +18,7 @@ package com.siyeh.ipp.modifiers; import com.intellij.psi.*; import com.intellij.psi.javadoc.PsiDocComment; import com.siyeh.ipp.base.PsiElementPredicate; +import org.jetbrains.annotations.NotNull; /** * @author Bas Leijdekkers @@ -26,7 +27,7 @@ class ModifierPredicate implements PsiElementPredicate { private final String myModifier; - public ModifierPredicate(@PsiModifier.ModifierConstant String modifier) { + public ModifierPredicate(@NotNull @PsiModifier.ModifierConstant String modifier) { myModifier = modifier; } @@ -45,6 +46,9 @@ class ModifierPredicate implements PsiElementPredicate { if (brace != null && brace.getTextOffset() < element.getTextOffset()) { return false; } + if (myModifier.equals(PsiModifier.PRIVATE) && aClass.getContainingClass() == null) { + return false; + } } final PsiModifierListOwner owner = (PsiModifierListOwner)parent; final PsiModifierList modifierList = owner.getModifierList();