From 7e80575a0882d3c0d3253dcee9d1c61a26961218 Mon Sep 17 00:00:00 2001
From: Mikhail Glukhikh <mikhail.glukhikh@jetbrains.com>
Date: Sat, 23 Apr 2016 12:57:47 +0300
Subject: [PATCH] Minor refactoring: MakeVisibleFactory

---
 .../jetbrains/kotlin/idea/quickfix/MakeVisibleFactory.kt    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/idea/src/org/jetbrains/kotlin/idea/quickfix/MakeVisibleFactory.kt b/idea/src/org/jetbrains/kotlin/idea/quickfix/MakeVisibleFactory.kt
index 4c7a956e449..3c7bc07d46c 100644
--- a/idea/src/org/jetbrains/kotlin/idea/quickfix/MakeVisibleFactory.kt
+++ b/idea/src/org/jetbrains/kotlin/idea/quickfix/MakeVisibleFactory.kt
@@ -44,8 +44,10 @@ object MakeVisibleFactory  : KotlinIntentionActionsFactory() {
         val declaration = DescriptorToSourceUtils.getSourceFromDescriptor(descriptor) as? KtModifierListOwner ?: return emptyList()
 
         val module = DescriptorUtils.getContainingModule(descriptor)
-        val targetVisibilities = if (module != usageModule || descriptor.visibility != PRIVATE) listOf(PUBLIC)
-                                 else listOf(PUBLIC, INTERNAL)
+        val targetVisibilities = when (descriptor.visibility) {
+            PRIVATE -> if (module != usageModule) listOf(PUBLIC) else listOf(PUBLIC, INTERNAL)
+            else -> listOf(PUBLIC)
+        }
 
         return targetVisibilities.mapNotNull { ChangeVisibilityFix.create(declaration, descriptor, it) }
     }