kotlin speedup: specify return type for public/protected APIs
authorSergey Ignatov <sergey.ignatov@jetbrains.com>
Thu, 24 May 2018 16:38:11 +0000 (19:38 +0300)
committerSergey Ignatov <sergey.ignatov@jetbrains.com>
Thu, 24 May 2018 16:38:47 +0000 (19:38 +0300)
151 files changed:
platform/diff-impl/src/com/intellij/diff/impl/DiffSettingsHolder.kt
platform/diff-impl/src/com/intellij/diff/tools/util/base/TextDiffSettingsHolder.kt
platform/diff-impl/src/com/intellij/openapi/vcs/ex/LineStatusTrackerBase.kt
platform/dvcs-impl/src/com/intellij/dvcs/actions/SetRepositoryChangesGroupingAction.kt
platform/dvcs-impl/src/com/intellij/dvcs/ui/RepositoryChangesBrowserNode.kt
platform/dvcs-impl/src/com/intellij/dvcs/ui/RepositoryChangesGroupingPolicy.kt
platform/editor-ui-api/src/com/intellij/ide/ui/UISettings.kt
platform/external-system-api/src/com/intellij/openapi/externalSystem/ExternalSystemModulePropertyManager.kt
platform/external-system-impl/src/com/intellij/openapi/externalSystem/service/project/settings/FacetConfigurationHandler.kt
platform/external-system-impl/src/com/intellij/openapi/externalSystem/service/project/settings/RunConfigurationHandler.kt
platform/lang-impl/src/com/intellij/codeInsight/daemon/impl/DaemonTooltipWithActionRenderer.kt
platform/lang-impl/src/com/intellij/codeInsight/hints/PopupActions.kt
platform/lang-impl/src/com/intellij/execution/compound/CompoundRunConfiguration.kt
platform/lang-impl/src/com/intellij/execution/impl/ExecutionManagerKtImpl.kt
platform/lang-impl/src/com/intellij/execution/impl/RunConfigurable.kt
platform/lang-impl/src/com/intellij/execution/impl/RunManagerImpl.kt
platform/lang-impl/src/com/intellij/execution/impl/RunnerAndConfigurationSettingsImpl.kt
platform/lang-impl/src/com/intellij/execution/impl/runConfigurableHelper.kt
platform/lang-impl/src/com/intellij/formatting/blocks/CStyleComment.kt
platform/lang-impl/src/com/intellij/formatting/blocks/TextLineBlock.kt
platform/lang-impl/src/com/intellij/ide/projectView/impl/ModuleGroupingTreeHelper.kt
platform/lang-impl/src/com/intellij/ide/projectView/impl/ProjectViewSharedSettings.kt
platform/lang-impl/src/com/intellij/index/PrebuiltIndex.kt
platform/lang-impl/src/com/intellij/index/PrebuiltIndexAwareIdIndexer.kt
platform/lang-impl/src/com/intellij/openapi/module/impl/AutomaticModuleUnloader.kt
platform/lang-impl/src/com/intellij/platform/ModuleAttachProcessor.kt
platform/lang-impl/src/com/intellij/psi/stubs/PrebuiltStubs.kt
platform/platform-impl/src/com/intellij/codeInsight/hints/filtering/MethodMatcher.kt
platform/platform-impl/src/com/intellij/codeInsight/hints/filtering/StringMatcherBuilder.kt
platform/platform-impl/src/com/intellij/configurationStore/storageUtil.kt
platform/platform-impl/src/com/intellij/diagnostic/DebugLogManager.kt
platform/platform-impl/src/com/intellij/diagnostic/ITNReporter.kt
platform/platform-impl/src/com/intellij/ide/actions/EditCustomSettingsAction.kt
platform/platform-impl/src/com/intellij/ide/actions/project/convertModuleGroupsToQualifiedNames.kt
platform/platform-impl/src/com/intellij/ide/ui/laf/darcula/ui/DarculaOptionButtonUI.kt
platform/platform-impl/src/com/intellij/ide/ui/laf/intellij/MacIntelliJOptionButtonUI.kt
platform/platform-impl/src/com/intellij/ide/ui/laf/intellij/WinIntelliJOptionButtonUI.kt
platform/platform-impl/src/com/intellij/internal/performance/Latenciometer.kt
platform/platform-impl/src/com/intellij/internal/statistic/collectors/legacy/jdk/JdkSettingsUsageCollector.kt
platform/platform-impl/src/com/intellij/internal/statistic/eventLog/LogEvents.kt
platform/platform-impl/src/com/intellij/openapi/keymap/impl/DefaultKeymapImpl.kt
platform/platform-impl/src/com/intellij/openapi/keymap/impl/KeymapImpl.kt
platform/platform-impl/src/com/intellij/openapi/keymap/impl/KeymapManagerImpl.kt
platform/platform-impl/src/com/intellij/openapi/keymap/impl/MacOSDefaultKeymap.kt
platform/platform-impl/src/com/intellij/openapi/updateSettings/impl/UpdateChecker.kt
platform/platform-impl/src/com/intellij/openapi/updateSettings/impl/UpdateInfo.kt
platform/platform-impl/src/com/intellij/openapi/updateSettings/impl/UpdateOptions.kt
platform/platform-impl/src/com/intellij/openapi/util/extensions.kt
platform/platform-impl/src/com/intellij/openapi/util/io/fileUtil.kt
platform/platform-impl/src/com/intellij/openapi/wm/impl/ProjectFrameBounds.kt
platform/platform-impl/src/com/intellij/openapi/wm/impl/WindowInfoImpl.kt
platform/platform-impl/src/com/intellij/openapi/wm/impl/content/AdditionalIcon.kt
platform/platform-impl/src/com/intellij/ui/components/BasicOptionButtonUI.kt
platform/platform-impl/src/com/intellij/ui/components/components.kt
platform/platform-impl/src/com/intellij/ui/layout/Cell.kt
platform/platform-impl/src/com/intellij/ui/layout/LayoutBuilder.kt
platform/platform-impl/src/com/intellij/ui/layout/layoutImpl.kt
platform/platform-impl/src/com/intellij/ui/layout/migLayout/patched/MigLayout.kt
platform/platform-impl/src/com/intellij/util/Urls.kt
platform/platform-impl/src/com/intellij/util/io/netty.kt
platform/platform-impl/src/com/intellij/util/net/net.kt
platform/platform-impl/src/org/jetbrains/ide/HttpRequestHandler.kt
platform/platform-impl/src/org/jetbrains/io/netty.kt
platform/projectModel-api/src/com/intellij/configurationStore/StreamProvider.kt
platform/projectModel-api/src/com/intellij/openapi/components/BaseState.kt
platform/projectModel-api/src/com/intellij/openapi/diagnostic/logger.kt
platform/projectModel-api/src/com/intellij/openapi/module/ModuleGrouper.kt
platform/projectModel-api/src/com/intellij/openapi/project/ExternalStorageConfigurationManager.kt
platform/projectModel-api/src/com/intellij/util/containers/util.kt
platform/projectModel-api/src/com/intellij/util/io/io.kt
platform/projectModel-api/src/com/intellij/util/io/path.kt
platform/projectModel-api/src/com/intellij/util/jdom.kt
platform/projectModel-api/src/org/jetbrains/concurrency/AsyncPromise.kt
platform/projectModel-api/src/org/jetbrains/concurrency/promise.kt
platform/projectModel-impl/src/com/intellij/configurationStore/BinaryXmlOutputter.kt
platform/projectModel-impl/src/com/intellij/configurationStore/StateStorageManager.kt
platform/projectModel-impl/src/com/intellij/configurationStore/StreamProviderFactory.kt
platform/projectModel-impl/src/com/intellij/configurationStore/scheme-impl.kt
platform/projectModel-impl/src/com/intellij/openapi/module/EmptyModuleManager.kt
platform/projectModel-impl/src/com/intellij/openapi/module/impl/LoadedModuleDescriptionImpl.kt
platform/projectModel-impl/src/com/intellij/openapi/module/impl/ModulePointerImpl.kt
platform/projectModel-impl/src/com/intellij/openapi/module/impl/ModulePointerManagerImpl.kt
platform/projectModel-impl/src/com/intellij/openapi/module/impl/ModuleRenamingHistoryState.kt
platform/projectModel-impl/src/com/intellij/openapi/module/impl/UnloadedModuleDescriptionImpl.kt
platform/projectModel-impl/src/com/intellij/openapi/options/ExternalizableSchemeAdapter.kt
platform/projectModel-impl/src/com/intellij/openapi/options/SchemeManager.kt
platform/projectModel-impl/src/com/intellij/openapi/options/scheme.kt
platform/projectModel-impl/src/com/intellij/openapi/roots/impl/libraries/ProjectLibraryTable.kt
platform/script-debugger/backend/src/debugger/Breakpoint.kt
platform/script-debugger/backend/src/debugger/BreakpointManager.kt
platform/script-debugger/backend/src/debugger/BreakpointManagerBase.kt
platform/script-debugger/backend/src/debugger/CallFrameBase.kt
platform/script-debugger/backend/src/debugger/EvaluateContextBase.kt
platform/script-debugger/backend/src/debugger/ObjectPropertyImpl.kt
platform/script-debugger/backend/src/debugger/ScriptBase.kt
platform/script-debugger/backend/src/debugger/ScriptManager.kt
platform/script-debugger/backend/src/debugger/ScriptManagerBaseEx.kt
platform/script-debugger/backend/src/debugger/ScriptRegExpBreakpointTarget.kt
platform/script-debugger/backend/src/debugger/SuspendContextManager.kt
platform/script-debugger/backend/src/debugger/SuspendContextManagerBase.kt
platform/script-debugger/backend/src/debugger/ValueModifierUtil.kt
platform/script-debugger/backend/src/debugger/VmBase.kt
platform/script-debugger/backend/src/debugger/sourcemap/MappingList.kt
platform/script-debugger/backend/src/debugger/sourcemap/NestedSourceMap.kt
platform/script-debugger/backend/src/debugger/sourcemap/SourceMap.kt
platform/script-debugger/backend/src/debugger/sourcemap/SourceResolver.kt
platform/script-debugger/backend/src/debugger/values/FunctionValue.kt
platform/script-debugger/backend/src/debugger/values/ObjectValue.kt
platform/script-debugger/backend/src/debugger/values/ObjectValueBase.kt
platform/script-debugger/backend/src/debugger/values/PrimitiveValue.kt
platform/script-debugger/backend/src/debugger/values/ValueManager.kt
platform/script-debugger/backend/src/debugger/values/ValueType.kt
platform/script-debugger/backend/src/rpc/CommandProcessor.kt
platform/script-debugger/backend/src/rpc/MessageManager.kt
platform/script-debugger/backend/src/rpc/MessageManagerBase.kt
platform/script-debugger/debugger-ui/src/BasicDebuggerViewSupport.kt
platform/script-debugger/debugger-ui/src/CallFrameView.kt
platform/script-debugger/debugger-ui/src/DebugProcessImpl.kt
platform/script-debugger/debugger-ui/src/DebuggerViewSupport.kt
platform/script-debugger/debugger-ui/src/MemberFilter.kt
platform/script-debugger/debugger-ui/src/ProcessHandlerWrapper.kt
platform/script-debugger/debugger-ui/src/RemoteVmConnection.kt
platform/script-debugger/debugger-ui/src/ScopeVariablesGroup.kt
platform/script-debugger/debugger-ui/src/SourceInfo.kt
platform/script-debugger/debugger-ui/src/SuspendContextView.kt
platform/script-debugger/debugger-ui/src/VariableView.kt
platform/script-debugger/debugger-ui/src/com/intellij/javascript/debugger/NameMapper.kt
platform/script-debugger/debugger-ui/src/com/intellij/javascript/debugger/execution/DebuggableProgramRunner.kt
platform/script-debugger/debugger-ui/src/com/jetbrains/javascript/debugger/FileUrlMapper.kt
platform/script-debugger/debugger-ui/src/com/jetbrains/javascript/debugger/JavaScriptDebugAware.kt
platform/script-debugger/debugger-ui/src/org/jetbrains/debugger/MemberFilterWithNameMappings.kt
platform/script-debugger/protocol/protocol-model-generator/src/BoxableType.kt
platform/script-debugger/protocol/protocol-model-generator/src/ClassNameScheme.kt
platform/script-debugger/protocol/protocol-model-generator/src/ClassScope.kt
platform/script-debugger/protocol/protocol-model-generator/src/DomainGenerator.kt
platform/script-debugger/protocol/protocol-model-generator/src/Generator.kt
platform/script-debugger/protocol/protocol-model-generator/src/ListType.kt
platform/script-debugger/protocol/protocol-model-generator/src/StandaloneType.kt
platform/script-debugger/protocol/protocol-reader-runtime/src/org/jetbrains/jsonProtocol/OutMessage.kt
platform/script-debugger/protocol/protocol-reader/src/FileUpdater.kt
platform/script-debugger/protocol/protocol-reader/src/JsonProtocolModelParseException.kt
platform/script-debugger/protocol/protocol-reader/src/ReaderGenerator.kt
platform/script-debugger/protocol/protocol-reader/src/TextOutput.kt
platform/script-debugger/protocol/protocol-reader/src/Util.kt
platform/testFramework/src/com/intellij/mock/MockRunManager.kt
platform/testFramework/src/com/intellij/project/IntelliJProjectConfiguration.kt
platform/testFramework/src/com/intellij/testFramework/FixtureRule.kt
platform/testFramework/src/com/intellij/testFramework/TemporaryDirectory.kt
platform/testFramework/src/com/intellij/testFramework/fsBuilder.kt
platform/testFramework/src/com/intellij/testFramework/inspections.kt
platform/testFramework/src/com/intellij/util/io/impl/DirectoryContentSpecImpl.kt

index 376675adffc976a2f4d91afca96211e3fc761661..8ee18999d439295bed80cdb4a9b2898f3b16db17 100644 (file)
@@ -74,7 +74,7 @@ class DiffSettingsHolder : PersistentStateComponent<DiffSettingsHolder.State> {
     @OptionTag
     @XMap
     @JvmField var PLACES_MAP: TreeMap<String, PlaceSettings> = TreeMap()
-    @JvmField var SHARED_SETTINGS = SharedSettings()
+    @JvmField var SHARED_SETTINGS: SharedSettings = SharedSettings()
   }
 
   private var myState: State = State()
index 80ba7aa374dc66e91b3d918b803cc398dcd9762d..4c9990156d0e7edd0bb0588deb4a69b30385bf7a 100644 (file)
@@ -174,7 +174,7 @@ class TextDiffSettingsHolder : PersistentStateComponent<TextDiffSettingsHolder.S
     @OptionTag
     @XMap
     @JvmField var PLACES_MAP: TreeMap<String, PlaceSettings> = TreeMap()
-    @JvmField var SHARED_SETTINGS = SharedSettings()
+    @JvmField var SHARED_SETTINGS: SharedSettings = SharedSettings()
   }
 
   private var myState: State = State()
index 00755f4012a92ab791138e1c7c5acc2bb03eede2..8bac924c5aec039081a9c8b25d9e8f9d11de2d9a 100644 (file)
@@ -17,6 +17,7 @@ package com.intellij.openapi.vcs.ex
 
 import com.intellij.diff.util.DiffUtil
 import com.intellij.diff.util.Side
+import com.intellij.openapi.Disposable
 import com.intellij.openapi.application.Application
 import com.intellij.openapi.application.ApplicationManager
 import com.intellij.openapi.application.runWriteAction
@@ -46,7 +47,7 @@ abstract class LineStatusTrackerBase<R : Range> {
   val document: Document
   val vcsDocument: Document
 
-  protected val disposable = Disposer.newDisposable()
+  protected val disposable: Disposable = Disposer.newDisposable()
   protected val documentTracker: DocumentTracker
   protected abstract val renderer: LineStatusMarkerRenderer
 
@@ -469,18 +470,18 @@ abstract class LineStatusTrackerBase<R : Range> {
 
 
   companion object {
-    @JvmStatic protected val LOG = Logger.getInstance("#com.intellij.openapi.vcs.ex.LineStatusTracker")
+    @JvmStatic protected val LOG: Logger = Logger.getInstance("#com.intellij.openapi.vcs.ex.LineStatusTracker")
 
     @JvmStatic protected val Block.start: Int get() = range.start2
     @JvmStatic protected val Block.end: Int get() = range.end2
     @JvmStatic protected val Block.vcsStart: Int get() = range.start1
     @JvmStatic protected val Block.vcsEnd: Int get() = range.end1
 
-    @JvmStatic protected fun Block.isSelectedByLine(line: Int) = DiffUtil.isSelectedByLine(line, this.range.start2, this.range.end2)
-    @JvmStatic protected fun Block.isSelectedByLine(lines: BitSet) = DiffUtil.isSelectedByLine(lines, this.range.start2, this.range.end2)
+    @JvmStatic protected fun Block.isSelectedByLine(line: Int): Boolean = DiffUtil.isSelectedByLine(line, this.range.start2, this.range.end2)
+    @JvmStatic protected fun Block.isSelectedByLine(lines: BitSet): Boolean = DiffUtil.isSelectedByLine(lines, this.range.start2, this.range.end2)
   }
 
 
   @TestOnly
-  fun getDocumentTrackerInTestMode() = documentTracker
+  fun getDocumentTrackerInTestMode(): DocumentTracker = documentTracker
 }
index e44c1e4c222a4aeb4bdc0386382ce7222e3689a9..a5676c065e3c89583edd3b514c7d5db5a45d0652 100644 (file)
@@ -6,9 +6,9 @@ import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.vcs.changes.actions.SetChangesGroupingAction
 
 class SetRepositoryChangesGroupingAction : SetChangesGroupingAction() {
-  override val groupingKey get() = "repository"
+  override val groupingKey: String get() = "repository"
 
-  override fun update(e: AnActionEvent) = super.update(e).also {
+  override fun update(e: AnActionEvent): Unit = super.update(e).also {
     val colorManager = e.project?.let(::getColorManager)
 
     e.presentation.isEnabledAndVisible = e.presentation.isEnabledAndVisible && colorManager?.isMultipleRoots ?: false
index 56614398cc0ea5547b2f9ba0b966b44b0644b45d..cfade195cdf98f619cd6228457c5e23b895d735f 100644 (file)
@@ -38,14 +38,14 @@ class RepositoryChangesBrowserNode(repository: Repository) : ChangesBrowserNode<
     appendCount(renderer)
   }
 
-  override fun getSortWeight() = REPOSITORY_SORT_WEIGHT
+  override fun getSortWeight(): Int = REPOSITORY_SORT_WEIGHT
 
-  override fun compareUserObjects(o2: Repository) = getShortRepositoryName(getUserObject()).compareTo(getShortRepositoryName(o2), true)
+  override fun compareUserObjects(o2: Repository): Int = getShortRepositoryName(getUserObject()).compareTo(getShortRepositoryName(o2), true)
 
-  override fun getTextPresentation() = getShortRepositoryName(getUserObject())
+  override fun getTextPresentation(): String = getShortRepositoryName(getUserObject())
 
   companion object {
-    fun getColorManager(project: Project) = VcsProjectLog.getInstance(project).logManager?.colorManager ?: VcsLogColorManagerImpl(
+    fun getColorManager(project: Project): VcsLogColorManagerImpl = VcsProjectLog.getInstance(project).logManager?.colorManager ?: VcsLogColorManagerImpl(
       findLogProviders(ProjectLevelVcsManagerImpl.getInstance(project).allVcsRoots.asList(), project).keys)
   }
 }
\ No newline at end of file
index 5eb81e306c08c5db24f04a52f071685c63a5d727..a1b37048210e3f974fb62a71e7ab64fd32ae90a8 100644 (file)
@@ -41,11 +41,11 @@ class RepositoryChangesGroupingPolicy(val project: Project, val model: DefaultTr
   }
 
   class Factory(val project: Project) : ChangesGroupingPolicyFactory() {
-    override fun createGroupingPolicy(model: DefaultTreeModel) = RepositoryChangesGroupingPolicy(project, model)
+    override fun createGroupingPolicy(model: DefaultTreeModel): RepositoryChangesGroupingPolicy = RepositoryChangesGroupingPolicy(project, model)
   }
 
   companion object {
-    val REPOSITORY_CACHE = NotNullLazyKey.create<MutableMap<Repository, ChangesBrowserNode<*>>, ChangesBrowserNode<*>>(
+    val REPOSITORY_CACHE: NotNullLazyKey<MutableMap<Repository, ChangesBrowserNode<*>>, ChangesBrowserNode<*>> = NotNullLazyKey.create<MutableMap<Repository, ChangesBrowserNode<*>>, ChangesBrowserNode<*>>(
       "ChangesTree.RepositoryCache") { _ -> mutableMapOf() }
 
   }
index f6de22330d3e49314027ff717ab2c34cf72f5942..2b9381b00f5d622cf34822247be0f2754ebeeb59 100644 (file)
@@ -35,81 +35,81 @@ class UISettings : BaseState(), PersistentStateComponent<UISettings> {
   // should be stored or shouldn't by the provided filter only.
   @get:Property(filter = FontFilter::class)
   @get:OptionTag("FONT_FACE")
-  var fontFace by string()
+  var fontFace: String? by string()
 
   @get:Property(filter = FontFilter::class)
   @get:OptionTag("FONT_SIZE")
-  var fontSize by property(defFontSize)
+  var fontSize: Int by property(defFontSize)
 
   @get:Property(filter = FontFilter::class)
   @get:OptionTag("FONT_SCALE")
-  var fontScale by property(0f)
-
-  @get:OptionTag("RECENT_FILES_LIMIT") var recentFilesLimit by property(50)
-  @get:OptionTag("CONSOLE_COMMAND_HISTORY_LIMIT") var consoleCommandHistoryLimit by property(300)
-  @get:OptionTag("OVERRIDE_CONSOLE_CYCLE_BUFFER_SIZE") var overrideConsoleCycleBufferSize by property(false)
-  @get:OptionTag("CONSOLE_CYCLE_BUFFER_SIZE_KB") var consoleCycleBufferSizeKb by property(1024)
-  @get:OptionTag("EDITOR_TAB_LIMIT") var editorTabLimit by property(10)
-
-  @get:OptionTag("REUSE_NOT_MODIFIED_TABS") var reuseNotModifiedTabs by property(false)
-  @get:OptionTag("ANIMATE_WINDOWS") var animateWindows by property(true)
-  @get:OptionTag("SHOW_TOOL_WINDOW_NUMBERS") var showToolWindowsNumbers by property(true)
-  @get:OptionTag("HIDE_TOOL_STRIPES") var hideToolStripes by property(false)
-  @get:OptionTag("WIDESCREEN_SUPPORT") var wideScreenSupport by property(false)
-  @get:OptionTag("LEFT_HORIZONTAL_SPLIT") var leftHorizontalSplit by property(false)
-  @get:OptionTag("RIGHT_HORIZONTAL_SPLIT") var rightHorizontalSplit by property(false)
-  @get:OptionTag("SHOW_EDITOR_TOOLTIP") var showEditorToolTip by property(true)
-  @get:OptionTag("SHOW_MEMORY_INDICATOR") var showMemoryIndicator by property(false)
-  @get:OptionTag("ALLOW_MERGE_BUTTONS") var allowMergeButtons by property(true)
-  @get:OptionTag("SHOW_MAIN_TOOLBAR") var showMainToolbar by property(false)
-  @get:OptionTag("SHOW_STATUS_BAR") var showStatusBar by property(true)
-  @get:OptionTag("SHOW_NAVIGATION_BAR") var showNavigationBar by property(true)
-  @get:OptionTag("ALWAYS_SHOW_WINDOW_BUTTONS") var alwaysShowWindowsButton by property(false)
-  @get:OptionTag("CYCLE_SCROLLING") var cycleScrolling by property(true)
-  @get:OptionTag("SCROLL_TAB_LAYOUT_IN_EDITOR") var scrollTabLayoutInEditor by property(true)
-  @get:OptionTag("HIDE_TABS_IF_NEED") var hideTabsIfNeed by property(true)
-  @get:OptionTag("SHOW_CLOSE_BUTTON") var showCloseButton by property(true)
-  @get:OptionTag("CLOSE_TAB_BUTTON_ON_THE_RIGHT") var closeTabButtonOnTheRight by property(true)
-  @get:OptionTag("EDITOR_TAB_PLACEMENT") var editorTabPlacement by property(SwingConstants.TOP)
-  @get:OptionTag("HIDE_KNOWN_EXTENSION_IN_TABS") var hideKnownExtensionInTabs by property(false)
-  @get:OptionTag("SHOW_ICONS_IN_QUICK_NAVIGATION") var showIconInQuickNavigation by property(true)
-
-  @get:OptionTag("CLOSE_NON_MODIFIED_FILES_FIRST") var closeNonModifiedFilesFirst by property(false)
-  @get:OptionTag("ACTIVATE_MRU_EDITOR_ON_CLOSE") var activeMruEditorOnClose by property(false)
+  var fontScale: Float by property(0f)
+
+  @get:OptionTag("RECENT_FILES_LIMIT") var recentFilesLimit: Int by property(50)
+  @get:OptionTag("CONSOLE_COMMAND_HISTORY_LIMIT") var consoleCommandHistoryLimit: Int by property(300)
+  @get:OptionTag("OVERRIDE_CONSOLE_CYCLE_BUFFER_SIZE") var overrideConsoleCycleBufferSize: Boolean by property(false)
+  @get:OptionTag("CONSOLE_CYCLE_BUFFER_SIZE_KB") var consoleCycleBufferSizeKb: Int by property(1024)
+  @get:OptionTag("EDITOR_TAB_LIMIT") var editorTabLimit: Int by property(10)
+
+  @get:OptionTag("REUSE_NOT_MODIFIED_TABS") var reuseNotModifiedTabs: Boolean by property(false)
+  @get:OptionTag("ANIMATE_WINDOWS") var animateWindows: Boolean by property(true)
+  @get:OptionTag("SHOW_TOOL_WINDOW_NUMBERS") var showToolWindowsNumbers: Boolean by property(true)
+  @get:OptionTag("HIDE_TOOL_STRIPES") var hideToolStripes: Boolean by property(false)
+  @get:OptionTag("WIDESCREEN_SUPPORT") var wideScreenSupport: Boolean by property(false)
+  @get:OptionTag("LEFT_HORIZONTAL_SPLIT") var leftHorizontalSplit: Boolean by property(false)
+  @get:OptionTag("RIGHT_HORIZONTAL_SPLIT") var rightHorizontalSplit: Boolean by property(false)
+  @get:OptionTag("SHOW_EDITOR_TOOLTIP") var showEditorToolTip: Boolean by property(true)
+  @get:OptionTag("SHOW_MEMORY_INDICATOR") var showMemoryIndicator: Boolean by property(false)
+  @get:OptionTag("ALLOW_MERGE_BUTTONS") var allowMergeButtons: Boolean by property(true)
+  @get:OptionTag("SHOW_MAIN_TOOLBAR") var showMainToolbar: Boolean by property(false)
+  @get:OptionTag("SHOW_STATUS_BAR") var showStatusBar: Boolean by property(true)
+  @get:OptionTag("SHOW_NAVIGATION_BAR") var showNavigationBar: Boolean by property(true)
+  @get:OptionTag("ALWAYS_SHOW_WINDOW_BUTTONS") var alwaysShowWindowsButton: Boolean by property(false)
+  @get:OptionTag("CYCLE_SCROLLING") var cycleScrolling: Boolean by property(true)
+  @get:OptionTag("SCROLL_TAB_LAYOUT_IN_EDITOR") var scrollTabLayoutInEditor: Boolean by property(true)
+  @get:OptionTag("HIDE_TABS_IF_NEED") var hideTabsIfNeed: Boolean by property(true)
+  @get:OptionTag("SHOW_CLOSE_BUTTON") var showCloseButton: Boolean by property(true)
+  @get:OptionTag("CLOSE_TAB_BUTTON_ON_THE_RIGHT") var closeTabButtonOnTheRight: Boolean by property(true)
+  @get:OptionTag("EDITOR_TAB_PLACEMENT") var editorTabPlacement: Int by property(SwingConstants.TOP)
+  @get:OptionTag("HIDE_KNOWN_EXTENSION_IN_TABS") var hideKnownExtensionInTabs: Boolean by property(false)
+  @get:OptionTag("SHOW_ICONS_IN_QUICK_NAVIGATION") var showIconInQuickNavigation: Boolean by property(true)
+
+  @get:OptionTag("CLOSE_NON_MODIFIED_FILES_FIRST") var closeNonModifiedFilesFirst: Boolean by property(false)
+  @get:OptionTag("ACTIVATE_MRU_EDITOR_ON_CLOSE") var activeMruEditorOnClose: Boolean by property(false)
   // TODO[anton] consider making all IDEs use the same settings
-  @get:OptionTag("ACTIVATE_RIGHT_EDITOR_ON_CLOSE") var activeRightEditorOnClose by property(PlatformUtils.isAppCode())
-
-  @get:OptionTag("IDE_AA_TYPE") var ideAAType by property(AntialiasingType.SUBPIXEL)
-  @get:OptionTag("EDITOR_AA_TYPE") var editorAAType by property(AntialiasingType.SUBPIXEL)
-  @get:OptionTag("COLOR_BLINDNESS") var colorBlindness by property<ColorBlindness?>()
-  @get:OptionTag("MOVE_MOUSE_ON_DEFAULT_BUTTON") var moveMouseOnDefaultButton by property(false)
-  @get:OptionTag("ENABLE_ALPHA_MODE") var enableAlphaMode by property(false)
-  @get:OptionTag("ALPHA_MODE_DELAY") var alphaModeDelay by property(1500)
-  @get:OptionTag("ALPHA_MODE_RATIO") var alphaModeRatio by property(0.5f)
-  @get:OptionTag("MAX_CLIPBOARD_CONTENTS") var maxClipboardContents by property(5)
-  @get:OptionTag("OVERRIDE_NONIDEA_LAF_FONTS") var overrideLafFonts by property(false)
-  @get:OptionTag("SHOW_ICONS_IN_MENUS") var showIconsInMenus by property(!PlatformUtils.isAppCode())
+  @get:OptionTag("ACTIVATE_RIGHT_EDITOR_ON_CLOSE") var activeRightEditorOnClose: Boolean by property(PlatformUtils.isAppCode())
+
+  @get:OptionTag("IDE_AA_TYPE") var ideAAType: AntialiasingType by property(AntialiasingType.SUBPIXEL)
+  @get:OptionTag("EDITOR_AA_TYPE") var editorAAType: AntialiasingType by property(AntialiasingType.SUBPIXEL)
+  @get:OptionTag("COLOR_BLINDNESS") var colorBlindness: ColorBlindness? by property<ColorBlindness?>()
+  @get:OptionTag("MOVE_MOUSE_ON_DEFAULT_BUTTON") var moveMouseOnDefaultButton: Boolean by property(false)
+  @get:OptionTag("ENABLE_ALPHA_MODE") var enableAlphaMode: Boolean by property(false)
+  @get:OptionTag("ALPHA_MODE_DELAY") var alphaModeDelay: Int by property(1500)
+  @get:OptionTag("ALPHA_MODE_RATIO") var alphaModeRatio: Float by property(0.5f)
+  @get:OptionTag("MAX_CLIPBOARD_CONTENTS") var maxClipboardContents: Int by property(5)
+  @get:OptionTag("OVERRIDE_NONIDEA_LAF_FONTS") var overrideLafFonts: Boolean by property(false)
+  @get:OptionTag("SHOW_ICONS_IN_MENUS") var showIconsInMenus: Boolean by property(!PlatformUtils.isAppCode())
   // IDEADEV-33409, should be disabled by default on MacOS
-  @get:OptionTag("DISABLE_MNEMONICS") var disableMnemonics by property(SystemInfo.isMac)
-  @get:OptionTag("DISABLE_MNEMONICS_IN_CONTROLS") var disableMnemonicsInControls by property(false)
-  @get:OptionTag("USE_SMALL_LABELS_ON_TABS") var useSmallLabelsOnTabs by property(SystemInfo.isMac)
-  @get:OptionTag("MAX_LOOKUP_WIDTH2") var maxLookupWidth by property(500)
-  @get:OptionTag("MAX_LOOKUP_LIST_HEIGHT") var maxLookupListHeight by property(11)
-  @get:OptionTag("HIDE_NAVIGATION_ON_FOCUS_LOSS") var hideNavigationOnFocusLoss by property(true)
-  @get:OptionTag("DND_WITH_PRESSED_ALT_ONLY") var dndWithPressedAltOnly by property(false)
-  @get:OptionTag("DEFAULT_AUTOSCROLL_TO_SOURCE") var defaultAutoScrollToSource by property(false)
-  @Transient var presentationMode = false
-  @get:OptionTag("PRESENTATION_MODE_FONT_SIZE") var presentationModeFontSize by property(24)
-  @get:OptionTag("MARK_MODIFIED_TABS_WITH_ASTERISK") var markModifiedTabsWithAsterisk by property(false)
-  @get:OptionTag("SHOW_TABS_TOOLTIPS") var showTabsTooltips by property(true)
-  @get:OptionTag("SHOW_DIRECTORY_FOR_NON_UNIQUE_FILENAMES") var showDirectoryForNonUniqueFilenames by property(true)
-  var smoothScrolling by property(SystemInfo.isMac && (SystemInfo.isJetBrainsJvm || SystemInfo.IS_AT_LEAST_JAVA9))
-  @get:OptionTag("NAVIGATE_TO_PREVIEW") var navigateToPreview by property(false)
-
-  @get:OptionTag("SORT_LOOKUP_ELEMENTS_LEXICOGRAPHICALLY") var sortLookupElementsLexicographically by property(false)
-  @get:OptionTag("MERGE_EQUAL_STACKTRACES") var mergeEqualStackTraces by property(true)
-  @get:OptionTag("SORT_BOOKMARKS") var sortBookmarks by property(false)
-  @get:OptionTag("PIN_FIND_IN_PATH_POPUP") var pinFindInPath by property(false)
+  @get:OptionTag("DISABLE_MNEMONICS") var disableMnemonics: Boolean by property(SystemInfo.isMac)
+  @get:OptionTag("DISABLE_MNEMONICS_IN_CONTROLS") var disableMnemonicsInControls: Boolean by property(false)
+  @get:OptionTag("USE_SMALL_LABELS_ON_TABS") var useSmallLabelsOnTabs: Boolean by property(SystemInfo.isMac)
+  @get:OptionTag("MAX_LOOKUP_WIDTH2") var maxLookupWidth: Int by property(500)
+  @get:OptionTag("MAX_LOOKUP_LIST_HEIGHT") var maxLookupListHeight: Int by property(11)
+  @get:OptionTag("HIDE_NAVIGATION_ON_FOCUS_LOSS") var hideNavigationOnFocusLoss: Boolean by property(true)
+  @get:OptionTag("DND_WITH_PRESSED_ALT_ONLY") var dndWithPressedAltOnly: Boolean by property(false)
+  @get:OptionTag("DEFAULT_AUTOSCROLL_TO_SOURCE") var defaultAutoScrollToSource: Boolean by property(false)
+  @Transient var presentationMode: Boolean = false
+  @get:OptionTag("PRESENTATION_MODE_FONT_SIZE") var presentationModeFontSize: Int by property(24)
+  @get:OptionTag("MARK_MODIFIED_TABS_WITH_ASTERISK") var markModifiedTabsWithAsterisk: Boolean by property(false)
+  @get:OptionTag("SHOW_TABS_TOOLTIPS") var showTabsTooltips: Boolean by property(true)
+  @get:OptionTag("SHOW_DIRECTORY_FOR_NON_UNIQUE_FILENAMES") var showDirectoryForNonUniqueFilenames: Boolean by property(true)
+  var smoothScrolling: Boolean by property(SystemInfo.isMac && (SystemInfo.isJetBrainsJvm || SystemInfo.IS_AT_LEAST_JAVA9))
+  @get:OptionTag("NAVIGATE_TO_PREVIEW") var navigateToPreview: Boolean by property(false)
+
+  @get:OptionTag("SORT_LOOKUP_ELEMENTS_LEXICOGRAPHICALLY") var sortLookupElementsLexicographically: Boolean by property(false)
+  @get:OptionTag("MERGE_EQUAL_STACKTRACES") var mergeEqualStackTraces: Boolean by property(true)
+  @get:OptionTag("SORT_BOOKMARKS") var sortBookmarks: Boolean by property(false)
+  @get:OptionTag("PIN_FIND_IN_PATH_POPUP") var pinFindInPath: Boolean by property(false)
 
   private val myTreeDispatcher = ComponentTreeEventDispatcher.create(UISettingsListener::class.java)
 
@@ -180,7 +180,7 @@ class UISettings : BaseState(), PersistentStateComponent<UISettings> {
     }
   }
 
-  override fun getState() = this
+  override fun getState(): UISettings = this
 
   override fun loadState(state: UISettings) {
     XmlSerializerUtil.copyBean(state, this)
@@ -247,10 +247,10 @@ class UISettings : BaseState(), PersistentStateComponent<UISettings> {
     @JvmStatic
     private fun verbose(msg: String, vararg args: Any) = if (JBUI.SCALE_VERBOSE) LOG.info(String.format(msg, *args)) else {}
 
-    const val ANIMATION_DURATION = 300 // Milliseconds
+    const val ANIMATION_DURATION: Int = 300 // Milliseconds
 
     /** Not tabbed pane.  */
-    const val TABS_NONE = 0
+    const val TABS_NONE: Int = 0
 
     private @Volatile var _instance: UISettings? = null
 
@@ -295,7 +295,7 @@ class UISettings : BaseState(), PersistentStateComponent<UISettings> {
       }
 
     @JvmField
-    val FORCE_USE_FRACTIONAL_METRICS = SystemProperties.getBooleanProperty("idea.force.use.fractional.metrics", false)
+    val FORCE_USE_FRACTIONAL_METRICS: Boolean = SystemProperties.getBooleanProperty("idea.force.use.fractional.metrics", false)
 
     @JvmStatic
     fun setupFractionalMetrics(g2d: Graphics2D) {
@@ -411,31 +411,31 @@ class UISettings : BaseState(), PersistentStateComponent<UISettings> {
   @Deprecated("Use hideToolStripes", replaceWith = ReplaceWith("hideToolStripes"))
   @JvmField
   @Transient
-  var HIDE_TOOL_STRIPES = true
+  var HIDE_TOOL_STRIPES: Boolean = true
 
   @Suppress("unused")
   @Deprecated("Use consoleCommandHistoryLimit", replaceWith = ReplaceWith("consoleCommandHistoryLimit"))
   @JvmField
   @Transient
-  var CONSOLE_COMMAND_HISTORY_LIMIT = 300
+  var CONSOLE_COMMAND_HISTORY_LIMIT: Int = 300
 
   @Suppress("unused")
   @Deprecated("Use cycleScrolling", replaceWith = ReplaceWith("cycleScrolling"))
   @JvmField
   @Transient
-  var CYCLE_SCROLLING = true
+  var CYCLE_SCROLLING: Boolean = true
 
   @Suppress("unused")
   @Deprecated("Use showMainToolbar", replaceWith = ReplaceWith("showMainToolbar"))
   @JvmField
   @Transient
-  var SHOW_MAIN_TOOLBAR = false
+  var SHOW_MAIN_TOOLBAR: Boolean = false
 
   @Suppress("unused")
   @Deprecated("Use showCloseButton", replaceWith = ReplaceWith("showCloseButton"))
   @JvmField
   @Transient
-  var SHOW_CLOSE_BUTTON = true
+  var SHOW_CLOSE_BUTTON: Boolean = true
 
   @Suppress("unused")
   @Deprecated("Use editorAAType", replaceWith = ReplaceWith("editorAAType"))
@@ -447,36 +447,36 @@ class UISettings : BaseState(), PersistentStateComponent<UISettings> {
   @Deprecated("Use presentationMode", replaceWith = ReplaceWith("presentationMode"))
   @JvmField
   @Transient
-  var PRESENTATION_MODE = false
+  var PRESENTATION_MODE: Boolean = false
 
   @Suppress("unused")
   @Deprecated("Use overrideLafFonts", replaceWith = ReplaceWith("overrideLafFonts"))
   @JvmField
   @Transient
-  var OVERRIDE_NONIDEA_LAF_FONTS = false
+  var OVERRIDE_NONIDEA_LAF_FONTS: Boolean = false
 
   @Suppress("unused")
   @Deprecated("Use presentationModeFontSize", replaceWith = ReplaceWith("presentationModeFontSize"))
   @JvmField
   @Transient
-  var PRESENTATION_MODE_FONT_SIZE = 24
+  var PRESENTATION_MODE_FONT_SIZE: Int = 24
 
   @Suppress("unused")
   @Deprecated("Use editorTabLimit", replaceWith = ReplaceWith("editorTabLimit"))
   @JvmField
   @Transient
-  var EDITOR_TAB_LIMIT = editorTabLimit
+  var EDITOR_TAB_LIMIT: Int = editorTabLimit
 
   @Suppress("unused")
   @Deprecated("Use overrideConsoleCycleBufferSize", replaceWith = ReplaceWith("overrideConsoleCycleBufferSize"))
   @JvmField
   @Transient
-  var OVERRIDE_CONSOLE_CYCLE_BUFFER_SIZE = false
+  var OVERRIDE_CONSOLE_CYCLE_BUFFER_SIZE: Boolean = false
 
   @Suppress("unused")
   @Deprecated("Use consoleCycleBufferSizeKb", replaceWith = ReplaceWith("consoleCycleBufferSizeKb"))
   @JvmField
   @Transient
-  var CONSOLE_CYCLE_BUFFER_SIZE_KB = consoleCycleBufferSizeKb
+  var CONSOLE_CYCLE_BUFFER_SIZE_KB: Int = consoleCycleBufferSizeKb
   //</editor-fold>
 }
\ No newline at end of file
index 09940c6ae6723a02f0f3b2de20eb27d5237d057e..fda15f93621ecf72dfd84fe1c1dfc6d94405d369 100644 (file)
@@ -11,21 +11,22 @@ import com.intellij.openapi.module.ModuleServiceManager
 import com.intellij.openapi.project.isExternalStorageEnabled
 import com.intellij.openapi.roots.ExternalProjectSystemRegistry
 import com.intellij.openapi.roots.ProjectModelElement
+import com.intellij.openapi.roots.ProjectModelExternalSource
 import com.intellij.util.xmlb.annotations.Attribute
 import com.intellij.util.xmlb.annotations.Transient
 import kotlin.properties.ReadWriteProperty
 import kotlin.reflect.KProperty
 
-val EMPTY_STATE = ExternalStateComponent()
+val EMPTY_STATE: ExternalStateComponent = ExternalStateComponent()
 
 @Suppress("DEPRECATION")
 @State(name = "ExternalSystem")
 class ExternalSystemModulePropertyManager(private val module: Module) : PersistentStateComponent<ExternalStateComponent>, ProjectModelElement {
-  override fun getExternalSource() = store.externalSystem?.let { ExternalProjectSystemRegistry.getInstance().getSourceById(it) }
+  override fun getExternalSource(): ProjectModelExternalSource? = store.externalSystem?.let { ExternalProjectSystemRegistry.getInstance().getSourceById(it) }
 
   private var store = if (module.project.isExternalStorageEnabled) ExternalStateComponent() else ExternalStateModule(module)
 
-  override fun getState() = store as? ExternalStateComponent ?: EMPTY_STATE
+  override fun getState(): ExternalStateComponent = store as? ExternalStateComponent ?: EMPTY_STATE
 
   override fun loadState(state: ExternalStateComponent) {
     store = state
@@ -33,26 +34,26 @@ class ExternalSystemModulePropertyManager(private val module: Module) : Persiste
 
   companion object {
     @JvmStatic
-    fun getInstance(module: Module) = ModuleServiceManager.getService(module, ExternalSystemModulePropertyManager::class.java)!!
+    fun getInstance(module: Module): ExternalSystemModulePropertyManager = ModuleServiceManager.getService(module, ExternalSystemModulePropertyManager::class.java)!!
   }
 
   @Suppress("DEPRECATION")
-  fun getExternalSystemId() = store.externalSystem
+  fun getExternalSystemId(): String? = store.externalSystem
 
-  fun getExternalModuleType() = store.externalSystemModuleType
+  fun getExternalModuleType(): String? = store.externalSystemModuleType
 
-  fun getExternalModuleVersion() = store.externalSystemModuleVersion
+  fun getExternalModuleVersion(): String? = store.externalSystemModuleVersion
 
-  fun getExternalModuleGroup() = store.externalSystemModuleGroup
+  fun getExternalModuleGroup(): String? = store.externalSystemModuleGroup
 
-  fun getLinkedProjectId() = store.linkedProjectId
+  fun getLinkedProjectId(): String? = store.linkedProjectId
 
-  fun getRootProjectPath() = store.rootProjectPath
+  fun getRootProjectPath(): String? = store.rootProjectPath
 
-  fun getLinkedProjectPath() = store.linkedProjectPath
+  fun getLinkedProjectPath(): String? = store.linkedProjectPath
 
   @Suppress("DEPRECATION")
-  fun isMavenized() = store.isMavenized
+  fun isMavenized(): Boolean = store.isMavenized
 
   @Suppress("DEPRECATION")
   fun setMavenized(mavenized: Boolean) {
index 941c30dcb1b6b575c28eb69f648fd74b8fe382d2..4b96dbc02f24adf61b58c2761b4688c46057cff9 100644 (file)
@@ -33,7 +33,7 @@ import com.intellij.openapi.project.Project
 class FacetConfigurationHandler : ConfigurationHandler {
 
   companion object {
-    val LOG = Logger.getInstance(FacetConfigurationHandler::class.java)
+    val LOG: Logger = Logger.getInstance(FacetConfigurationHandler::class.java)
   }
 
   override fun apply(module: Module, modelsProvider: IdeModifiableModelsProvider, configuration: ConfigurationData) {
index 60ba4b788741da35d1baddbb70505f02e0ad4f51..650aec72c8f06457ae7d72f882af688e8f569e4e 100644 (file)
@@ -18,7 +18,7 @@ import com.intellij.util.ObjectUtils.consumeIfCast
 class RunConfigurationHandler : ConfigurationHandler {
 
   companion object {
-    val LOG = Logger.getInstance(RunConfigurationHandler::class.java)
+    val LOG: Logger = Logger.getInstance(RunConfigurationHandler::class.java)
   }
 
   private fun Any?.isTrue(): Boolean = this != null && this is Boolean && this
index 0074d94e2b60a108cec486811ea57204a5c4d3ba..8a9d0dbfe1901d778244d5168bebec22e549c081 100644 (file)
@@ -40,7 +40,7 @@ import javax.swing.MenuSelectionManager
 import javax.swing.event.HyperlinkEvent
 
 
-val runActionCustomShortcutSet = CustomShortcutSet(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, KeyEvent.SHIFT_DOWN_MASK or KeyEvent.ALT_DOWN_MASK ))
+val runActionCustomShortcutSet: CustomShortcutSet = CustomShortcutSet(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, KeyEvent.SHIFT_DOWN_MASK or KeyEvent.ALT_DOWN_MASK ))
 
 internal class DaemonTooltipWithActionRenderer(text: String?,
                                                private val tooltipAction: TooltipAction?,
index aab76dd2bb08d3a7708773d7a9212251fbb5b6c7..abba036702d5bd0384bc6b8b4f1499a81d15efa9 100644 (file)
@@ -145,7 +145,7 @@ class BlacklistCurrentMethodIntention : IntentionAction, LowPriorityAction {
     refreshAllOpenEditors()
   }
 
-  override fun startInWriteAction() = false
+  override fun startInWriteAction(): Boolean = false
 }
 
 
@@ -188,7 +188,7 @@ class DisableCustomHintsOption: IntentionAction, LowPriorityAction {
     refreshAllOpenEditors()
   }
 
-  override fun startInWriteAction() = false
+  override fun startInWriteAction(): Boolean = false
 }
 
 class EnableCustomHintsOption: IntentionAction, HighPriorityAction {
@@ -236,7 +236,7 @@ class EnableCustomHintsOption: IntentionAction, HighPriorityAction {
     refreshAllOpenEditors()
   }
 
-  override fun startInWriteAction() = false
+  override fun startInWriteAction(): Boolean = false
 
 }
 
@@ -322,7 +322,7 @@ fun PsiElement.isOwnsInlayInEditor(editor: Editor): Boolean {
 }
 
 
-fun MethodInfo.toPattern() = this.fullyQualifiedName + '(' + this.paramNames.joinToString(",") + ')'
+fun MethodInfo.toPattern(): String = this.fullyQualifiedName + '(' + this.paramNames.joinToString(",") + ')'
 
 
 private fun String.capitalize() = StringUtil.capitalizeWords(this, true)
index 3ddfc1efa6c371a0cd1685000447c35baf371256..2605484bc66836ee8cc97e37ed2e9ca987d2630a 100644 (file)
@@ -25,7 +25,7 @@ class CompoundRunConfiguration @JvmOverloads constructor(project: Project, name:
   RunConfigurationBase(project, factory, name), RunnerIconProvider, WithoutOwnBeforeRunSteps, Cloneable {
   companion object {
     @JvmField
-    val COMPARATOR = Comparator<RunConfiguration> { o1, o2 ->
+    val COMPARATOR: Comparator<RunConfiguration> = Comparator<RunConfiguration> { o1, o2 ->
       val i = o1.type.displayName.compareTo(o2.type.displayName)
       when {
         i != 0 -> i
@@ -84,7 +84,7 @@ class CompoundRunConfiguration @JvmOverloads constructor(project: Project, name:
     isInitialized = true
   }
 
-  override fun getConfigurationEditor() = CompoundRunConfigurationSettingsEditor(project)
+  override fun getConfigurationEditor(): CompoundRunConfigurationSettingsEditor = CompoundRunConfigurationSettingsEditor(project)
 
   override fun checkConfiguration() {
     if (sortedConfigurationsWithTargets.isEmpty()) {
index 0a761d42f5897851e2712f5200cc1c5ab4158971..1722ca43949b08d3e33622b05e5aad4dbed2e5e7 100644 (file)
@@ -25,7 +25,7 @@ import java.util.concurrent.atomic.AtomicBoolean
 
 class ExecutionManagerKtImpl(project: Project) : ExecutionManagerImpl(project) {
   @set:TestOnly
-  @Volatile var forceCompilationInTests = false
+  @Volatile var forceCompilationInTests: Boolean = false
 
   override fun startRunProfile(starter: RunProfileStarter, state: RunProfileState, environment: ExecutionEnvironment) {
     val project = environment.project
index 5ff2f441433dd18ad943204583b8890502398e5b..d824c80e869bff1bbbf3017008746ff91315ff1e 100644 (file)
@@ -73,9 +73,9 @@ private fun getName(userObject: Any): String {
 
 open class RunConfigurable @JvmOverloads constructor(private val project: Project, var runDialog: RunDialogBase? = null) : Configurable, Disposable {
   @Volatile private var isDisposed: Boolean = false
-  val root = DefaultMutableTreeNode("Root")
-  val treeModel = MyTreeModel(root)
-  val tree = Tree(treeModel)
+  val root: DefaultMutableTreeNode = DefaultMutableTreeNode("Root")
+  val treeModel: MyTreeModel = MyTreeModel(root)
+  val tree: Tree = Tree(treeModel)
   private val rightPanel = JPanel(BorderLayout())
   private val splitter = JBSplitter("RunConfigurable.dividerProportion", 0.3f)
   private var wholePanel: JPanel? = null
@@ -1430,7 +1430,7 @@ open class RunConfigurable @JvmOverloads constructor(private val project: Projec
     }
 
     //Legacy, use canDrop() instead
-    override fun canExchangeRows(oldIndex: Int, newIndex: Int) = false
+    override fun canExchangeRows(oldIndex: Int, newIndex: Int): Boolean = false
 
     override fun canDrop(oldIndex: Int, newIndex: Int, position: RowsDnDSupport.RefinedDropSupport.Position): Boolean {
       if (tree.rowCount <= oldIndex || tree.rowCount <= newIndex || oldIndex < 0 || newIndex < 0) {
index 7eaad650fc692748e0eac63c7f3c6a82bf724b6e..5015fc602cb2d079438e476908e0d1e8b4c9e6c5 100644 (file)
@@ -46,13 +46,13 @@ private const val RECENT = "recent_temporary"
 @State(name = "RunManager", storages = [(Storage(value = StoragePathMacros.WORKSPACE_FILE, useSaveThreshold = ThreeState.NO))])
 open class RunManagerImpl(internal val project: Project) : RunManagerEx(), PersistentStateComponent<Element>, Disposable {
   companion object {
-    const val CONFIGURATION = "configuration"
-    const val NAME_ATTR = "name"
+    const val CONFIGURATION: String = "configuration"
+    const val NAME_ATTR: String = "name"
 
     internal val LOG = logger<RunManagerImpl>()
 
     @JvmStatic
-    fun getInstanceImpl(project: Project) = RunManager.getInstance(project) as RunManagerImpl
+    fun getInstanceImpl(project: Project): RunManagerImpl = RunManager.getInstance(project) as RunManagerImpl
 
     @JvmStatic
     fun canRunConfiguration(environment: ExecutionEnvironment): Boolean {
@@ -190,9 +190,9 @@ open class RunManagerImpl(internal val project: Project) : RunManagerEx(), Persi
     lock.write { templateIdToConfiguration.clear() }
   }
 
-  override fun getConfig() = _config
+  override fun getConfig(): RunManagerConfig = _config
 
-  override val configurationFactories by lazy { idToType.values.toTypedArray() }
+  override val configurationFactories: Array<ConfigurationType> by lazy { idToType.values.toTypedArray() }
 
   override val configurationFactoriesWithoutUnknown: List<ConfigurationType>
     get() = idToType.values.filterSmart { it.isManaged }
@@ -217,9 +217,9 @@ open class RunManagerImpl(internal val project: Project) : RunManagerEx(), Persi
   override val allConfigurationsList: List<RunConfiguration>
     get() = allSettings.mapSmart { it.configuration }
 
-  fun getSettings(configuration: RunConfiguration) = allSettings.firstOrNull { it.configuration === configuration } as? RunnerAndConfigurationSettingsImpl
+  fun getSettings(configuration: RunConfiguration): RunnerAndConfigurationSettingsImpl? = allSettings.firstOrNull { it.configuration === configuration } as? RunnerAndConfigurationSettingsImpl
 
-  override fun getConfigurationSettingsList(type: ConfigurationType) = allSettings.filterSmart { it.type.id == type.id }
+  override fun getConfigurationSettingsList(type: ConfigurationType): List<RunnerAndConfigurationSettings> = allSettings.filterSmart { it.type.id == type.id }
 
   override fun getStructure(type: ConfigurationType): Map<String, List<RunnerAndConfigurationSettings>> {
     val result = LinkedHashMap<String?, MutableList<RunnerAndConfigurationSettings>>()
@@ -625,7 +625,7 @@ open class RunManagerImpl(internal val project: Project) : RunManagerEx(), Persi
     eventPublisher.runConfigurationSelected()
   }
 
-  override fun hasSettings(settings: RunnerAndConfigurationSettings) = lock.read { idToSettings.get(settings.uniqueID) == settings }
+  override fun hasSettings(settings: RunnerAndConfigurationSettings): Boolean = lock.read { idToSettings.get(settings.uniqueID) == settings }
 
   private fun findExistingConfigurationId(settings: RunnerAndConfigurationSettings): String? {
     for ((key, value) in idToSettings) {
@@ -749,7 +749,7 @@ open class RunManagerImpl(internal val project: Project) : RunManagerEx(), Persi
     configuration.beforeRunTasks = result ?: emptyList()
   }
 
-  override fun getConfigurationType(typeName: String) = idToType.get(typeName)
+  override fun getConfigurationType(typeName: String): ConfigurationType? = idToType.get(typeName)
 
   @JvmOverloads
   fun getFactory(typeId: String?, factoryId: String?, checkUnknown: Boolean = false): ConfigurationFactory? {
@@ -843,7 +843,7 @@ open class RunManagerImpl(internal val project: Project) : RunManagerEx(), Persi
     return icon
   }
 
-  fun getConfigurationById(id: String) = lock.read { idToSettings.get(id) }
+  fun getConfigurationById(id: String): RunnerAndConfigurationSettings? = lock.read { idToSettings.get(id) }
 
   override fun findConfigurationByName(name: String?): RunnerAndConfigurationSettings? {
     if (name == null) {
index fecb3525a63011df4c96b042a9ce1559e839407b..6382b51f076ed7b75e54ef3dfc26dd0a7684b4c1 100644 (file)
@@ -36,8 +36,8 @@ private const val RUNNER_ID = "RunnerId"
 private const val CONFIGURATION_TYPE_ATTRIBUTE = "type"
 private const val FACTORY_NAME_ATTRIBUTE = "factoryName"
 private const val FOLDER_NAME = "folderName"
-const val NAME_ATTR = "name"
-const val DUMMY_ELEMENT_NAME = "dummy"
+const val NAME_ATTR: String = "name"
+const val DUMMY_ELEMENT_NAME: String = "dummy"
 private const val TEMPORARY_ATTRIBUTE = "temporary"
 private const val EDIT_BEFORE_RUN = "editBeforeRun"
 private const val ACTIVATE_TOOLWINDOW_BEFORE_RUN = "activateToolWindowBeforeRun"
@@ -45,7 +45,7 @@ private const val ACTIVATE_TOOLWINDOW_BEFORE_RUN = "activateToolWindowBeforeRun"
 private const val TEMP_CONFIGURATION = "tempConfiguration"
 internal const val TEMPLATE_FLAG_ATTRIBUTE = "default"
 
-const val SINGLETON = "singleton"
+const val SINGLETON: String = "singleton"
 
 enum class RunConfigurationLevel {
   WORKSPACE, PROJECT, TEMPORARY
@@ -86,15 +86,15 @@ class RunnerAndConfigurationSettingsImpl @JvmOverloads constructor(val manager:
 
   override fun getFactory(): ConfigurationFactory = _configuration?.factory ?: UnknownConfigurationType.getFactory()
 
-  override fun isTemplate() = isTemplate
+  override fun isTemplate(): Boolean = isTemplate
 
-  override fun isTemporary() = level == RunConfigurationLevel.TEMPORARY
+  override fun isTemporary(): Boolean = level == RunConfigurationLevel.TEMPORARY
 
   override fun setTemporary(value: Boolean) {
     level = if (value) RunConfigurationLevel.TEMPORARY else RunConfigurationLevel.WORKSPACE
   }
 
-  override fun isShared() = level == RunConfigurationLevel.PROJECT
+  override fun isShared(): Boolean = level == RunConfigurationLevel.PROJECT
 
   override fun setShared(value: Boolean) {
     if (value) {
@@ -105,7 +105,7 @@ class RunnerAndConfigurationSettingsImpl @JvmOverloads constructor(val manager:
     }
   }
 
-  override fun getConfiguration() = _configuration ?: UnknownConfigurationType.getFactory().createTemplateConfiguration(manager.project)
+  override fun getConfiguration(): RunConfiguration = _configuration ?: UnknownConfigurationType.getFactory().createTemplateConfiguration(manager.project)
 
   override fun createFactory(): Factory<RunnerAndConfigurationSettings> {
     return Factory {
@@ -143,25 +143,25 @@ class RunnerAndConfigurationSettingsImpl @JvmOverloads constructor(val manager:
     isEditBeforeRun = value
   }
 
-  override fun isEditBeforeRun() = isEditBeforeRun
+  override fun isEditBeforeRun(): Boolean = isEditBeforeRun
 
   override fun setActivateToolWindowBeforeRun(value: Boolean) {
     isActivateToolWindowBeforeRun = value
   }
 
-  override fun isActivateToolWindowBeforeRun() = isActivateToolWindowBeforeRun
+  override fun isActivateToolWindowBeforeRun(): Boolean = isActivateToolWindowBeforeRun
 
   override fun setSingleton(value: Boolean) {
     isSingleton = value
   }
 
-  override fun isSingleton() = isSingleton
+  override fun isSingleton(): Boolean = isSingleton
 
   override fun setFolderName(value: String?) {
     folderName = value
   }
 
-  override fun getFolderName() = folderName
+  override fun getFolderName(): String? = folderName
 
   fun readExternal(element: Element, isShared: Boolean) {
     isTemplate = element.getAttributeBooleanValue(TEMPLATE_FLAG_ATTRIBUTE)
@@ -337,9 +337,9 @@ class RunnerAndConfigurationSettingsImpl @JvmOverloads constructor(val manager:
     }
   }
 
-  override fun getRunnerSettings(runner: ProgramRunner<*>) = runnerSettings.getOrCreateSettings(runner)
+  override fun getRunnerSettings(runner: ProgramRunner<*>): RunnerSettings? = runnerSettings.getOrCreateSettings(runner)
 
-  override fun getConfigurationSettings(runner: ProgramRunner<*>) = configurationPerRunnerSettings.getOrCreateSettings(runner)
+  override fun getConfigurationSettings(runner: ProgramRunner<*>): ConfigurationPerRunnerSettings? = configurationPerRunnerSettings.getOrCreateSettings(runner)
 
   override fun getType(): ConfigurationType = factory.type
 
@@ -384,9 +384,9 @@ class RunnerAndConfigurationSettingsImpl @JvmOverloads constructor(val manager:
     }
   }
 
-  override fun compareTo(other: Any) = if (other is RunnerAndConfigurationSettings) name.compareTo(other.name) else 0
+  override fun compareTo(other: Any): Int = if (other is RunnerAndConfigurationSettings) name.compareTo(other.name) else 0
 
-  override fun toString() = "${type.displayName}: ${if (isTemplate) "<template>" else name} (level: $level)"
+  override fun toString(): String = "${type.displayName}: ${if (isTemplate) "<template>" else name} (level: $level)"
 
   private inner class InfoProvider(override val runner: ProgramRunner<*>) : ConfigurationInfoProvider {
     override val configuration: RunConfiguration
index d7d7600c62b4cd4f561f81a854d561b2a92ea31b..8ce6f4180e982310c349e1c6a8929d17bd4a1729 100644 (file)
@@ -26,7 +26,7 @@ internal class RunConfigurationBean {
 enum class RunConfigurableNodeKind {
   CONFIGURATION_TYPE, FOLDER, CONFIGURATION, TEMPORARY_CONFIGURATION, UNKNOWN;
 
-  fun supportsDnD() = this == FOLDER || this == CONFIGURATION || this == TEMPORARY_CONFIGURATION
+  fun supportsDnD(): Boolean = this == FOLDER || this == CONFIGURATION || this == TEMPORARY_CONFIGURATION
 
   val isConfiguration: Boolean
     get() = (this == CONFIGURATION) or (this == TEMPORARY_CONFIGURATION)
index 0557983848264aeea87ca0a31b7adc059175a171..4da4691c12b4ebce232e26ef27f81841a7511ee3 100644 (file)
@@ -26,7 +26,7 @@ import com.intellij.psi.formatter.common.AbstractBlock
 class CStyleCommentBlock(comment: ASTNode, private val indent: Indent?): AbstractBlock(comment, null, null) {
 
   private val lines by lazy { lineBlocks() }
-  val isCommentFormattable by lazy {
+  val isCommentFormattable: Boolean by lazy {
     lines.drop(1).all { it.text.startsWith("*") }
   }
 
@@ -42,7 +42,7 @@ class CStyleCommentBlock(comment: ASTNode, private val indent: Indent?): Abstrac
     return child2.getSpacing(null, this)
   }
 
-  override fun getIndent() = indent
+  override fun getIndent(): Indent? = indent
 
   override fun buildChildren(): List<Block> {
     if (!isCommentFormattable) return emptyList()
@@ -76,7 +76,7 @@ class CStyleCommentBlock(comment: ASTNode, private val indent: Indent?): Abstrac
         }
   }
 
-  override fun isLeaf() = !isCommentFormattable
+  override fun isLeaf(): Boolean = !isCommentFormattable
 
 }
 
index 7ad77b0ce0b72ef403ba938ef9924c0d99d385c2..c1670ffd4fd8623e70aab3b06e4a6b81349f4bcc 100644 (file)
@@ -32,21 +32,21 @@ class TextLineBlock(
 
   override fun getSubBlocks(): List<Block> = emptyList()
 
-  override fun getWrap() = null
+  override fun getWrap(): Nothing? = null
 
-  override fun getIndent() = indent
+  override fun getIndent(): Indent? = indent
 
-  override fun getAlignment() = alignment
+  override fun getAlignment(): Alignment? = alignment
 
-  override fun getSpacing(child1: Block?, child2: Block) = spacing
+  override fun getSpacing(child1: Block?, child2: Block): Spacing? = spacing
 
   override fun getChildAttributes(newChildIndex: Int): ChildAttributes {
     throw UnsupportedOperationException("Should not be called")
   }
 
-  override fun isIncomplete() = false
+  override fun isIncomplete(): Boolean = false
 
-  override fun isLeaf() = true
+  override fun isLeaf(): Boolean = true
 
   override fun toString(): String {
     return "TextLineBlock(text='$text', textRange=$textRange)"
index afb8dc656a4b7b059ea1d1f93aa066489bb14f2e..a9fc96f8cfc8d18c7523d100206697fba44a8451 100644 (file)
@@ -69,7 +69,7 @@ class ModuleGroupingTreeHelper<M: Any, N: MutableTreeNode> private constructor(
     @JvmStatic
     fun <M: Any, N : MutableTreeNode> forEmptyTree(groupingEnabled: Boolean, grouping: ModuleGroupingImplementation<M>,
                                                    moduleGroupNodeFactory: (ModuleGroup) -> N, moduleNodeFactory: (M) -> N,
-                                                   nodeComparator: Comparator<in N>) =
+                                                   nodeComparator: Comparator<in N>): ModuleGroupingTreeHelper<M, N> =
       ModuleGroupingTreeHelper(groupingEnabled, grouping, moduleGroupNodeFactory, moduleNodeFactory, nodeComparator)
 
     @JvmStatic
@@ -111,7 +111,7 @@ class ModuleGroupingTreeHelper<M: Any, N: MutableTreeNode> private constructor(
     }
 
     @JvmStatic
-    fun createDefaultGrouping(grouper: ModuleGrouper) = object : ModuleGroupingImplementation<Module> {
+    fun createDefaultGrouping(grouper: ModuleGrouper): ModuleGroupingImplementation<Module> = object : ModuleGroupingImplementation<Module> {
       override val compactGroupNodes: Boolean
         get() = grouper.compactGroupNodes
 
@@ -364,19 +364,19 @@ class ModuleGroupingTreeHelper<M: Any, N: MutableTreeNode> private constructor(
   }
 
   @TestOnly
-  fun getNodeForGroupMap() = Collections.unmodifiableMap(nodeForGroup)
+  fun getNodeForGroupMap(): MutableMap<ModuleGroup, N>? = Collections.unmodifiableMap(nodeForGroup)
 
   @TestOnly
-  fun getVirtualGroupToChildNodeMap() = Collections.unmodifiableMap(virtualGroupToChildNode)
+  fun getVirtualGroupToChildNodeMap(): MutableMap<ModuleGroup, N>? = Collections.unmodifiableMap(virtualGroupToChildNode)
 
   @TestOnly
-  fun getModuleByNodeMap() = nodeData.mapValues { it.value.module }.filterValues { it != null }
+  fun getModuleByNodeMap(): Map<N, M?> = nodeData.mapValues { it.value.module }.filterValues { it != null }
 
   @TestOnly
-  fun getGroupByNodeMap() = nodeData.mapValues { it.value.group }.filterValues { it != null }
+  fun getGroupByNodeMap(): Map<N, ModuleGroup?> = nodeData.mapValues { it.value.group }.filterValues { it != null }
 
   @TestOnly
-  fun isGroupingEnabled() = groupingEnabled
+  fun isGroupingEnabled(): Boolean = groupingEnabled
 }
 
 private class ModuleTreeNodeData<M>(val module: M?, val group: ModuleGroup?)
index e6eca5a611563f6bdeeddf977ce3a0181d3ac037..7bc889139a6b5a740ce3e036665ebaf5dc51a291 100644 (file)
@@ -12,18 +12,18 @@ import com.intellij.util.xmlb.XmlSerializerUtil
  */
 @State(name = "ProjectViewSharedSettings", storages = [(Storage(value = "projectView.xml"))])
 class ProjectViewSharedSettings : PersistentStateComponent<ProjectViewSharedSettings> {
-  var flattenPackages = false
-  var showMembers = false
-  var sortByType = false
-  var showModules = true
-  var flattenModules = false
-  var showLibraryContents = true
-  var hideEmptyPackages = true
-  var abbreviatePackages = false
-  var autoscrollFromSource = false
-  var autoscrollToSource = false
-  var foldersAlwaysOnTop = true
-  var manualOrder = false
+  var flattenPackages: Boolean = false
+  var showMembers: Boolean = false
+  var sortByType: Boolean = false
+  var showModules: Boolean = true
+  var flattenModules: Boolean = false
+  var showLibraryContents: Boolean = true
+  var hideEmptyPackages: Boolean = true
+  var abbreviatePackages: Boolean = false
+  var autoscrollFromSource: Boolean = false
+  var autoscrollToSource: Boolean = false
+  var foldersAlwaysOnTop: Boolean = true
+  var manualOrder: Boolean = false
 
   override fun getState(): ProjectViewSharedSettings? {
     return this
index 4c87559890dfcabe94ddabbc00688d39c2fb3d72..cd338d784d355a57b3901be415a4a9e0c010b140 100644 (file)
@@ -34,7 +34,7 @@ abstract class PrebuiltIndexProviderBase<Value> : Disposable {
     private val LOG = Logger.getInstance("#com.intellij.index.PrebuiltIndexProviderBase")
 
     @JvmField
-    val DEBUG_PREBUILT_INDICES = SystemProperties.getBooleanProperty("debug.prebuilt.indices", false)
+    val DEBUG_PREBUILT_INDICES: Boolean = SystemProperties.getBooleanProperty("debug.prebuilt.indices", false)
   }
 
   init {
index d4ace5ab61f266914ccd8d26ad066682c30e8726..1158a4f85f4dc1169ebea739ba8b13791d5f0889 100644 (file)
@@ -17,12 +17,12 @@ import java.io.DataOutput
 abstract class PrebuiltIndexAwareIdIndexer : PrebuiltIndexProviderBase<Map<IdIndexEntry, Int>>(), LexingIdIndexer {
   companion object {
     private val LOG = Logger.getInstance("#com.intellij.index.PrebuiltIndexAwareIdIndexer")
-    val ID_INDEX_FILE_NAME = "id-index"
+    val ID_INDEX_FILE_NAME: String = "id-index"
   }
 
-  override val indexName get() = ID_INDEX_FILE_NAME
+  override val indexName: String get() = ID_INDEX_FILE_NAME
 
-  override val indexExternalizer get() = IdIndexMapDataExternalizer()
+  override val indexExternalizer: IdIndexMapDataExternalizer get() = IdIndexMapDataExternalizer()
 
   override fun map(inputData: FileContent): Map<IdIndexEntry, Int> {
     val map = get(inputData)
index b5f80c3846952b2af359251974c6f7fb80f319f3..794d896db14b0030cee6c6dc092957fefc5e2abd 100644 (file)
@@ -120,7 +120,7 @@ class AutomaticModuleUnloader(private val project: Project) : PersistentStateCom
     }
   }
 
-  fun createAction(text: String, action: (MutableList<String>) -> Unit) = object : NotificationAction(text) {
+  fun createAction(text: String, action: (MutableList<String>) -> Unit): NotificationAction = object : NotificationAction(text) {
     override fun actionPerformed(e: AnActionEvent, notification: Notification) {
       val unloaded = ArrayList<String>()
       val moduleManager = ModuleManager.getInstance(project)
@@ -136,7 +136,7 @@ class AutomaticModuleUnloader(private val project: Project) : PersistentStateCom
     loadedModulesListStorage.modules.addAll(modules)
   }
 
-  override fun getState() = loadedModulesListStorage
+  override fun getState(): LoadedModulesListStorage = loadedModulesListStorage
 
   override fun loadState(state: LoadedModulesListStorage) {
     setLoadedModules(state.modules)
@@ -144,7 +144,7 @@ class AutomaticModuleUnloader(private val project: Project) : PersistentStateCom
 
   companion object {
     @JvmStatic
-    fun getInstance(project: Project) = project.service<AutomaticModuleUnloader>()
+    fun getInstance(project: Project): AutomaticModuleUnloader = project.service<AutomaticModuleUnloader>()
 
     private val NOTIFICATION_GROUP = NotificationGroup.balloonGroup("Automatic Module Unloading")
   }
index 18b123b6becab1d3bb04955eea7ba59229e14327..0e3c845478fd0a95edee23969cbb282960adca82 100644 (file)
@@ -55,7 +55,7 @@ class ModuleAttachProcessor : ProjectAttachProcessor() {
     }
 
     @JvmStatic
-    fun getPrimaryModule(project: Project) = if (ProjectAttachProcessor.canAttachToProject()) findModuleInBaseDir(project) else null
+    fun getPrimaryModule(project: Project): Module? = if (ProjectAttachProcessor.canAttachToProject()) findModuleInBaseDir(project) else null
 
     @JvmStatic
     fun getSortedModules(project: Project): List<Module> {
index c6896425516caacc0831124999418453e58b7b79..f4e46195cf5a3c0dd1a7bbda9afd03563d2db88a 100644 (file)
@@ -38,7 +38,7 @@ import java.io.File
  * @author traff
  */
 
-val EP_NAME = "com.intellij.filetype.prebuiltStubsProvider"
+val EP_NAME: String = "com.intellij.filetype.prebuiltStubsProvider"
 
 object PrebuiltStubsProviders : FileTypeExtension<PrebuiltStubsProvider>(EP_NAME)
 
@@ -50,7 +50,7 @@ interface PrebuiltStubsProvider {
 class FileContentHashing {
   private val hashing = Hashing.sha256()
 
-  fun hashString(fileContent: FileContent) = hashing.hashBytes(fileContent.content)!!
+  fun hashString(fileContent: FileContent): HashCode = hashing.hashBytes(fileContent.content)!!
 }
 
 
@@ -60,7 +60,7 @@ class HashCodeDescriptor : HashCodeExternalizers(), KeyDescriptor<HashCode> {
   override fun isEqual(val1: HashCode, val2: HashCode): Boolean = val1 == val2
 
   companion object {
-    val instance = HashCodeDescriptor()
+    val instance: HashCodeDescriptor = HashCodeDescriptor()
   }
 }
 
@@ -84,7 +84,7 @@ class StubTreeExternalizer : DataExternalizer<SerializedStubTree> {
     value.write(out)
   }
 
-  override fun read(`in`: DataInput) = SerializedStubTree(`in`)
+  override fun read(`in`: DataInput): SerializedStubTree = SerializedStubTree(`in`)
 }
 
 abstract class PrebuiltStubsProviderBase : PrebuiltIndexProviderBase<SerializedStubTree>(), PrebuiltStubsProvider {
@@ -93,13 +93,13 @@ abstract class PrebuiltStubsProviderBase : PrebuiltIndexProviderBase<SerializedS
 
   protected abstract val stubVersion: Int
 
-  override val indexName get() = SDK_STUBS_STORAGE_NAME
+  override val indexName: String get() = SDK_STUBS_STORAGE_NAME
 
-  override val indexExternalizer get() = StubTreeExternalizer()
+  override val indexExternalizer: StubTreeExternalizer get() = StubTreeExternalizer()
 
   companion object {
-    val PREBUILT_INDICES_PATH_PROPERTY = "prebuilt_indices_path"
-    val SDK_STUBS_STORAGE_NAME = "sdk-stubs"
+    val PREBUILT_INDICES_PATH_PROPERTY: String = "prebuilt_indices_path"
+    val SDK_STUBS_STORAGE_NAME: String = "sdk-stubs"
     private val LOG = Logger.getInstance("#com.intellij.psi.stubs.PrebuiltStubsProviderBase")
   }
 
index 4ffa22a429a7c53eb1930343ac442a6b20f9d4ec..3527a5a44952ab6fe5748da28f99ac76688e42a4 100644 (file)
@@ -27,7 +27,7 @@ interface MethodMatcher {
 }
 
 object AnyParamMatcher: ParamMatcher {
-  override fun isMatching(paramNames: List<String>) = true
+  override fun isMatching(paramNames: List<String>): Boolean = true
 }
 
 class StringParamMatcher(private val paramMatchers: List<StringMatcher>): ParamMatcher {
index f6e7ad47199e174dfd4c8935b2e09ac989b424d5..dc4bb22cd02d59f1075ac256721b3e824e358e8e 100644 (file)
@@ -20,7 +20,7 @@ interface StringMatcher {
 }
 
 class StringMatcherImpl(private val matcher: (String) -> Boolean) : StringMatcher {
-  override fun isMatching(text: String) = matcher(text)
+  override fun isMatching(text: String): Boolean = matcher(text)
 }
 
 object StringMatcherBuilder {
index f5becf1d3054b56547f31548f66c7dba8895d4ae..ea7450dbbfb297ae77de296901cd167f51a66b7b 100644 (file)
@@ -28,7 +28,7 @@ import java.io.IOException
 import java.nio.file.Path
 import java.util.*
 
-const val NOTIFICATION_GROUP_ID = "Load Error"
+const val NOTIFICATION_GROUP_ID: String = "Load Error"
 
 @TestOnly
 var DEBUG_LOG: String? = null
index 5bb18c3e0d8d7823ae25aa8def7d0f95e308deb1..fbdb061ef06213b51647cb903dc3f9683e37abf7 100644 (file)
@@ -25,7 +25,7 @@ class DebugLogManager(private val properties: PropertiesComponent) : Application
     applyCategories(categories)
   }
 
-  fun getSavedCategories() =
+  fun getSavedCategories(): List<Pair<String, DebugLogLevel>> =
     fromString(properties.getValue(LOG_DEBUG_CATEGORIES), DebugLogLevel.DEBUG) +
     fromString(properties.getValue(LOG_TRACE_CATEGORIES), DebugLogLevel.TRACE)
 
index 2089022346c75009d4f734c89e216fb1de7fd580..478dc93960681246b3f13e78899ccf4e8de040bc 100644 (file)
@@ -47,7 +47,7 @@ open class ITNReporter : ErrorReportSubmitter() {
   /**
    * Used to enable error reporting even in release versions.
    */
-  open fun showErrorInRelease(event: IdeaLoggingEvent) = false
+  open fun showErrorInRelease(event: IdeaLoggingEvent): Boolean = false
 }
 
 fun setPluginInfo(event: IdeaLoggingEvent, errorBean: ErrorBean) {
index e4cdadfea54fa0534da7c1151f66feaa3be59631..2bf95a6b6a2091e9c385e3ceedd484fa5f69511f 100644 (file)
@@ -138,7 +138,7 @@ class EditCustomVmOptionsAction : EditCustomSettingsAction() {
   override fun file(): File? = EditCustomVmOptionsAction.file.value
   override fun template(): String = "# custom ${ApplicationNamesInfo.getInstance().fullProductName} VM options\n\n${VMOptions.read() ?: ""}"
 
-  fun isEnabled() = file() != null
+  fun isEnabled(): Boolean = file() != null
 
   class AccessExtension : NonProjectFileWritingAccessExtension {
     override fun isWritable(file: VirtualFile): Boolean = FileUtil.pathsEqual(file.path, EditCustomVmOptionsAction.file.value?.path)
index ee03176061d80fc6f445f89233c05392bfecf919..0bfdab553fd0aff36a2e204188813325ac52915f 100644 (file)
@@ -41,6 +41,7 @@ import java.util.function.Function
 import java.util.function.Supplier
 import javax.swing.Action
 import javax.swing.JCheckBox
+import javax.swing.JComponent
 import javax.swing.JPanel
 
 class ConvertModuleGroupsToQualifiedNamesDialog(val project: Project) : DialogWrapper(project) {
@@ -105,7 +106,7 @@ class ConvertModuleGroupsToQualifiedNamesDialog(val project: Project) : DialogWr
       .addToBottom(recordPreviousNames)
   }
 
-  override fun getPreferredFocusedComponent() = editorArea.focusTarget
+  override fun getPreferredFocusedComponent(): JComponent = editorArea.focusTarget
 
   private fun generateLineExtension(line: Int): Collection<LineExtensionInfo> {
     val lineText = document.charsSequence.subSequence(document.getLineStartOffset(line), document.getLineEndOffset(line)).toString()
index d2035154b218a3004048c6c53e332de16d58f5ed..84a3b351cfddfb1c45d414035fdda76c8f70bc2c 100644 (file)
@@ -17,17 +17,17 @@ import javax.swing.JComponent
 import javax.swing.border.Border
 
 open class DarculaOptionButtonUI : BasicOptionButtonUI() {
-  protected open val clipXOffset = scale(7)
+  protected open val clipXOffset: Int = scale(7)
 
   private var optionButtonBorder: Border? = null
 
-  override fun configureOptionButton() = super.configureOptionButton().also { optionButtonBorder = optionButton.border }
-  override fun unconfigureOptionButton() = super.unconfigureOptionButton().also {
+  override fun configureOptionButton(): Unit = super.configureOptionButton().also { optionButtonBorder = optionButton.border }
+  override fun unconfigureOptionButton(): Unit = super.unconfigureOptionButton().also {
     optionButton.border = optionButtonBorder
     optionButtonBorder = null
   }
 
-  override fun createMainButton() = object : MainButton() {
+  override fun createMainButton(): MainButton = object : MainButton() {
     override fun paintNotSimple(g: Graphics2D) {
       g.clipRect(0, 0, width - clipXOffset, height)
       paintBackground(g, this)
@@ -36,10 +36,10 @@ open class DarculaOptionButtonUI : BasicOptionButtonUI() {
     }
   }
 
-  override fun configureMainButton() = super.configureMainButton().also { mainButton.isOpaque = false }
-  override fun unconfigureMainButton() = super.unconfigureMainButton().also { mainButton.isOpaque = true }
+  override fun configureMainButton(): Unit = super.configureMainButton().also { mainButton.isOpaque = false }
+  override fun unconfigureMainButton(): Unit = super.unconfigureMainButton().also { mainButton.isOpaque = true }
 
-  override fun createArrowButton() = object : ArrowButton() {
+  override fun createArrowButton(): ArrowButton = object : ArrowButton() {
     override fun paintNotSimple(g: Graphics2D) {
       g.clipRect(clipXOffset, 0, width - clipXOffset, height)
       paintBackground(g, this)
@@ -57,12 +57,12 @@ open class DarculaOptionButtonUI : BasicOptionButtonUI() {
     g.fill(DarculaComboBoxUI.getArrowShape(b))
   }
 
-  override fun configureArrowButton() = super.configureArrowButton().also { arrowButton.isOpaque = false }
-  override fun unconfigureArrowButton() = super.unconfigureArrowButton().also { arrowButton.isOpaque = true }
+  override fun configureArrowButton(): Unit = super.configureArrowButton().also { arrowButton.isOpaque = false }
+  override fun unconfigureArrowButton(): Unit = super.unconfigureArrowButton().also { arrowButton.isOpaque = true }
 
-  override val arrowButtonPreferredSize get() = Dimension(getArrowButtonPreferredSize(null).width, optionButton.preferredSize.height)
+  override val arrowButtonPreferredSize: Dimension get() = Dimension(getArrowButtonPreferredSize(null).width, optionButton.preferredSize.height)
 
-  override val showPopupXOffset get() = JBUI.scale(3)
+  override val showPopupXOffset: Int get() = JBUI.scale(3)
 
   override fun paint(g: Graphics, c: JComponent) {
     if (!isSimpleButton) paintSeparatorArea(g as Graphics2D, c)
@@ -88,13 +88,13 @@ open class DarculaOptionButtonUI : BasicOptionButtonUI() {
     g.fill(Rectangle2D.Float(x, yOffset, LW.getFloat(), mainButton.height - yOffset * 2))
   }
 
-  override fun updateOptions() = super.updateOptions().also {
+  override fun updateOptions(): Unit = super.updateOptions().also {
     optionButton.border = if (isSimpleButton) optionButtonBorder else mainButton.border
   }
 
   companion object {
     @Suppress("UNUSED_PARAMETER")
     @JvmStatic
-    fun createUI(c: JComponent) = DarculaOptionButtonUI()
+    fun createUI(c: JComponent): DarculaOptionButtonUI = DarculaOptionButtonUI()
   }
 }
\ No newline at end of file
index ccdec4f749c85e0f059415e6a55a794d0798a716..fe781c115c0bdf9a0b6aa9613f023227f83b4380 100644 (file)
@@ -11,9 +11,9 @@ import javax.swing.JComponent
 
 // TODO replace arrow with specific icon when it's ready
 class MacIntelliJOptionButtonUI : DarculaOptionButtonUI() {
-  override val arrowButtonPreferredSize get() = Dimension(scale(21), optionButton.preferredSize.height)
+  override val arrowButtonPreferredSize: Dimension get() = Dimension(scale(21), optionButton.preferredSize.height)
 
-  override val showPopupXOffset = scale(3)
+  override val showPopupXOffset: Int = scale(3)
 
   override fun paintSeparator(g: Graphics2D, c: JComponent) {
     val insets = mainButton.insets
@@ -26,6 +26,6 @@ class MacIntelliJOptionButtonUI : DarculaOptionButtonUI() {
   companion object {
     @Suppress("UNUSED_PARAMETER")
     @JvmStatic
-    fun createUI(c: JComponent) = MacIntelliJOptionButtonUI()
+    fun createUI(c: JComponent): MacIntelliJOptionButtonUI = MacIntelliJOptionButtonUI()
   }
 }
\ No newline at end of file
index 13ea9e20365f99ae4ccfb11f2f43f704f970c1c7..a5305e9d7c1425443a858116bfe4672b663af573 100644 (file)
@@ -20,7 +20,7 @@ class WinIntelliJOptionButtonUI : BasicOptionButtonUI() {
   private var mainButtonChangeListener: ChangeListener? = null
   private var arrowButtonChangeListener: ChangeListener? = null
 
-  override fun createMainButton() = object : MainButton() {
+  override fun createMainButton(): MainButton = object : MainButton() {
     override fun paintNotSimple(g: Graphics2D) {
       g.clipRect(0, 0, width - outerInsets.right, height)
 //      background is painted in button ui
@@ -34,14 +34,14 @@ class WinIntelliJOptionButtonUI : BasicOptionButtonUI() {
     }
   }
 
-  override fun configureMainButton() = super.configureMainButton().also {
+  override fun configureMainButton(): Unit = super.configureMainButton().also {
     mainButton.isOpaque = false
     mainButtonBorder = mainButton.border
 
     mainButtonChangeListener = createInnerButtonChangeListener().apply(mainButton::addChangeListener)
   }
 
-  override fun unconfigureMainButton() = super.unconfigureMainButton().also {
+  override fun unconfigureMainButton(): Unit = super.unconfigureMainButton().also {
     mainButton.removeChangeListener(mainButtonChangeListener)
     mainButtonChangeListener = null
 
@@ -50,7 +50,7 @@ class WinIntelliJOptionButtonUI : BasicOptionButtonUI() {
     mainButtonBorder = null
   }
 
-  override fun createArrowButton() = object : ArrowButton() {
+  override fun createArrowButton(): ArrowButton = object : ArrowButton() {
     override fun paintNotSimple(g: Graphics2D) {
       val bw = scale(2)
       g.clipRect(outerInsets.left + bw, 0, width - (outerInsets.left + bw), height)
@@ -70,14 +70,14 @@ class WinIntelliJOptionButtonUI : BasicOptionButtonUI() {
     }
   }
 
-  override fun configureArrowButton() = super.configureArrowButton().also {
+  override fun configureArrowButton(): Unit = super.configureArrowButton().also {
     arrowButton.isOpaque = false
     arrowButtonBorder = arrowButton.border
 
     arrowButtonChangeListener = createInnerButtonChangeListener().apply(arrowButton::addChangeListener)
   }
 
-  override fun unconfigureArrowButton() = super.unconfigureArrowButton().also {
+  override fun unconfigureArrowButton(): Unit = super.unconfigureArrowButton().also {
     arrowButton.removeChangeListener(arrowButtonChangeListener)
     arrowButtonChangeListener = null
 
@@ -86,11 +86,11 @@ class WinIntelliJOptionButtonUI : BasicOptionButtonUI() {
     arrowButtonBorder = null
   }
 
-  override val arrowButtonPreferredSize get() = Dimension(scale(23), optionButton.preferredSize.height)
+  override val arrowButtonPreferredSize: Dimension get() = Dimension(scale(23), optionButton.preferredSize.height)
 
-  override val showPopupXOffset get() = scale(4)
+  override val showPopupXOffset: Int get() = scale(4)
 
-  override fun createLayoutManager() = object : OptionButtonLayout() {
+  override fun createLayoutManager(): OptionButtonLayout = object : OptionButtonLayout() {
     override fun layoutContainer(parent: Container) {
       val mainButtonWidth = optionButton.width - if (arrowButton.isVisible) arrowButton.preferredSize.width else 0
       val offset = if (arrowButton.isVisible) scale(2) else 0
@@ -125,7 +125,7 @@ class WinIntelliJOptionButtonUI : BasicOptionButtonUI() {
     g.fill(Rectangle(x, y, 2 * lw, height))
   }
 
-  override fun updateOptions() = super.updateOptions().also {
+  override fun updateOptions(): Unit = super.updateOptions().also {
     mainButton.border = if (isSimpleButton) mainButtonBorder else ButtonBorder()
     arrowButton.border = if (isSimpleButton) arrowButtonBorder else ButtonBorder()
   }
@@ -142,6 +142,6 @@ class WinIntelliJOptionButtonUI : BasicOptionButtonUI() {
   companion object {
     @Suppress("UNUSED_PARAMETER")
     @JvmStatic
-    fun createUI(c: JComponent) = WinIntelliJOptionButtonUI()
+    fun createUI(c: JComponent): WinIntelliJOptionButtonUI = WinIntelliJOptionButtonUI()
   }
 }
\ No newline at end of file
index 97ba8a7cfe847ee62d3dbd7e571e92b17bedc28e..27aa5947524aef1ce0ab74cd0199fa861fe07903 100644 (file)
@@ -31,9 +31,9 @@ class LatencyRecorderImpl : LatencyRecorder {
 }
 
 class LatencyRecord {
-  var totalKeysTyped = 0
-  var totalLatency = 0L
-  var maxLatency = 0L
+  var totalKeysTyped: Int = 0
+  var totalLatency: Long = 0L
+  var maxLatency: Long = 0L
 
   fun update(latencyInMS: Long) {
     totalKeysTyped++
@@ -43,12 +43,12 @@ class LatencyRecord {
     }
   }
 
-  val averageLatency get() = totalLatency / totalKeysTyped
+  val averageLatency: Long get() = totalLatency / totalKeysTyped
 }
 
 class FileTypeLatencyRecord(val fileType: FileType) {
-  val totalLatency = LatencyRecord()
-  val actionLatencyRecords = mutableMapOf<String, LatencyRecord>()
+  val totalLatency: LatencyRecord = LatencyRecord()
+  val actionLatencyRecords: MutableMap<String, LatencyRecord> = mutableMapOf<String, LatencyRecord>()
 
   fun update(action: String, latencyInMS: Long) {
     totalLatency.update(latencyInMS)
@@ -56,7 +56,7 @@ class FileTypeLatencyRecord(val fileType: FileType) {
   }
 }
 
-val latencyMap = mutableMapOf<FileType, FileTypeLatencyRecord>()
+val latencyMap: MutableMap<FileType, FileTypeLatencyRecord> = mutableMapOf<FileType, FileTypeLatencyRecord>()
 
 fun recordTypingLatency(editor: Editor, action: String, latencyInMS: Long) {
   val fileType = FileDocumentManager.getInstance().getFile(editor.document)?.fileType ?: return
@@ -66,7 +66,7 @@ fun recordTypingLatency(editor: Editor, action: String, latencyInMS: Long) {
   latencyRecord.update(getActionKey(action), latencyInMS)
 }
 
-fun getActionKey(action: String) =
+fun getActionKey(action: String): String =
   if (action.length == 1) {
     when(action[0]) {
       in 'A'..'Z', in 'a'..'z', in '0'..'9' -> "Letter"
index 4eb1eed1b35bdcb025e9bd2d54252d94d85d6ebc..484dba6943dd7a8e3fb1a94bc6078e754eb905b9 100644 (file)
@@ -21,10 +21,10 @@ class JdkSettingsUsageCollector : UsagesCollector() {
       .toSet()
   }
 
-  val prefixes = listOf("-Xms",
-                        "-Xmx",
-                        "-XX:SoftRefLRUPolicyMSPerMB",
-                        "-XX:ReservedCodeCacheSize")
+  val prefixes: List<String> = listOf("-Xms",
+                                      "-Xmx",
+                                      "-XX:SoftRefLRUPolicyMSPerMB",
+                                      "-XX:ReservedCodeCacheSize")
 
   fun accept(key: String?): Boolean {
     if (key == null) return false
index 27756faedb936c0fb2578ecf6d1d8d8fb8d0e397..5871f6ffa2fea90386829ee595c4510e902e9ec4 100644 (file)
@@ -9,9 +9,9 @@ import com.intellij.util.containers.ContainerUtil
 import java.util.*
 
 open class LogEvent(session: String, bucket: String, recorderId: String, recorderVersion: String, type: String) {
-  val session = escape(session)
-  val bucket = escape(bucket)
-  val time = System.currentTimeMillis()
+  val session: String = escape(session)
+  val bucket: String = escape(bucket)
+  val time: Long = System.currentTimeMillis()
   val recorder: LogEventRecorder = LogEventRecorder(escape(recorderId), escape(recorderVersion))
   val action: LogEventAction = LogEventAction(escape(type))
 
index 5cca31240e3dc0001e8d2ed21f71bb1680efc0b3..5d4d834f4a485086f0d13e2f8e38a9a7c9723cc2 100644 (file)
@@ -18,6 +18,7 @@ package com.intellij.openapi.keymap.impl
 import com.intellij.configurationStore.SchemeDataHolder
 import com.intellij.openapi.actionSystem.IdeActions
 import com.intellij.openapi.actionSystem.MouseShortcut
+import com.intellij.openapi.keymap.Keymap
 import com.intellij.openapi.keymap.KeymapManager
 import com.intellij.openapi.options.SchemeState
 import com.intellij.openapi.util.SystemInfo
@@ -31,7 +32,7 @@ open class DefaultKeymapImpl(dataHolder: SchemeDataHolder<KeymapImpl>, private v
       // ignore
     }
 
-  override fun getSchemeState() = SchemeState.NON_PERSISTENT
+  override fun getSchemeState(): SchemeState = SchemeState.NON_PERSISTENT
 
   override fun getPresentableName(): String = DefaultKeymap.instance.getKeymapPresentableName(this)
 
@@ -45,5 +46,5 @@ open class DefaultKeymapImpl(dataHolder: SchemeDataHolder<KeymapImpl>, private v
 
   // default keymap can have parent only in the defaultKeymapManager
   // also, it allows us to avoid dependency on KeymapManager (maybe not initialized yet)
-  override fun findParentScheme(parentSchemeName: String) = defaultKeymapManager.findScheme(parentSchemeName)
+  override fun findParentScheme(parentSchemeName: String): Keymap? = defaultKeymapManager.findScheme(parentSchemeName)
 }
index f7bcc5b7116d66a9f71bda2458d712c58536ab1a..e64169fa696d4a88df0a9b71167e00729bc135c2 100644 (file)
@@ -52,12 +52,12 @@ fun KeymapImpl(name: String, dataHolder: SchemeDataHolder<KeymapImpl>): KeymapIm
 
 open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDataHolder<KeymapImpl>? = null) : ExternalizableSchemeAdapter(), Keymap, SerializableScheme {
   private var parent: KeymapImpl? = null
-  open var canModify = true
+  open var canModify: Boolean = true
 
   @JvmField
   internal var schemeState: SchemeState? = null
 
-  override fun getSchemeState() = schemeState
+  override fun getSchemeState(): SchemeState? = schemeState
 
   private val actionIdToShortcuts = THashMap<String, MutableList<Shortcut>>()
     get() {
@@ -123,7 +123,7 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
       return result
     }
 
-  override fun getPresentableName() = name
+  override fun getPresentableName(): String = name
 
   override fun deriveKeymap(newName: String): KeymapImpl {
     if (canModify()) {
@@ -178,9 +178,9 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     return otherKeymap
   }
 
-  override fun getParent() = parent
+  override fun getParent(): KeymapImpl? = parent
 
-  override final fun canModify() = canModify
+  override final fun canModify(): Boolean = canModify
 
   override fun addShortcut(actionId: String, shortcut: Shortcut) {
     val list = actionIdToShortcuts.getOrPut(actionId) {
@@ -335,7 +335,7 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     return sortInRegistrationOrder(list)
   }
 
-  override fun getActionIds(firstKeyStroke: KeyStroke) = getActionIds(firstKeyStroke, { it.keystrokeToIds }, KeymapImpl::convertKeyStroke)
+  override fun getActionIds(firstKeyStroke: KeyStroke): Array<String> = getActionIds(firstKeyStroke, { it.keystrokeToIds }, KeymapImpl::convertKeyStroke)
 
   override fun getActionIds(firstKeyStroke: KeyStroke, secondKeyStroke: KeyStroke?): Array<String> {
     val ids = getActionIds(firstKeyStroke)
@@ -387,7 +387,7 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     while (true)
   }
 
-  override fun getActionIds(shortcut: MouseShortcut) = getActionIds(shortcut, { it.mouseShortcutToActionIds }, KeymapImpl::convertMouseShortcut)
+  override fun getActionIds(shortcut: MouseShortcut): Array<String> = getActionIds(shortcut, { it.mouseShortcutToActionIds }, KeymapImpl::convertMouseShortcut)
 
   private fun <T> getActionIds(shortcut: T, shortcutToActionIds: (keymap: KeymapImpl) -> Map<T, MutableList<String>>, convertShortcut: (keymap: KeymapImpl, shortcut: T) -> T): Array<String> {
     // first, get keystrokes from own map
@@ -416,7 +416,7 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     return sortInRegistrationOrder(list)
   }
 
-  fun isActionBound(actionId: String) = keymapManager.boundActions.contains(actionId)
+  fun isActionBound(actionId: String): Boolean = keymapManager.boundActions.contains(actionId)
 
   override fun getShortcuts(actionId: String?): Array<Shortcut> = getMutableShortcutList(actionId).let { if (it.isEmpty()) Shortcut.EMPTY_ARRAY else it.toTypedArray() }
 
@@ -534,7 +534,7 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     cleanShortcutsCache()
   }
 
-  protected open fun findParentScheme(parentSchemeName: String) = keymapManager.schemeManager.findSchemeByName(parentSchemeName)
+  protected open fun findParentScheme(parentSchemeName: String): Keymap? = keymapManager.schemeManager.findSchemeByName(parentSchemeName)
 
   override fun writeScheme(): Element {
     dataHolder?.let {
@@ -594,7 +594,7 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     cleanShortcutsCache()
   }
 
-  fun hasOwnActionId(actionId: String) = actionIdToShortcuts.containsKey(actionId)
+  fun hasOwnActionId(actionId: String): Boolean = actionIdToShortcuts.containsKey(actionId)
 
   fun clearOwnActionsId(actionId: String) {
     actionIdToShortcuts.remove(actionId)
@@ -643,11 +643,11 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     return result
   }
 
-  protected open fun convertKeyStroke(keyStroke: KeyStroke) = keyStroke
+  protected open fun convertKeyStroke(keyStroke: KeyStroke): KeyStroke = keyStroke
 
-  protected open fun convertMouseShortcut(shortcut: MouseShortcut) = shortcut
+  protected open fun convertMouseShortcut(shortcut: MouseShortcut): MouseShortcut = shortcut
 
-  protected open fun convertShortcut(shortcut: Shortcut) = shortcut
+  protected open fun convertShortcut(shortcut: Shortcut): Shortcut = shortcut
 
   override fun addShortcutChangeListener(listener: Keymap.Listener) {
     listeners.add(listener)
@@ -664,7 +664,7 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     }
   }
 
-  override fun toString() = presentableName
+  override fun toString(): String = presentableName
 
   override fun equals(other: Any?): Boolean {
     if (other !is KeymapImpl) return false
@@ -675,7 +675,7 @@ open class KeymapImpl @JvmOverloads constructor(private var dataHolder: SchemeDa
     return true
   }
 
-  override fun hashCode() = name.hashCode()
+  override fun hashCode(): Int = name.hashCode()
 }
 
 private fun sortInRegistrationOrder(ids: List<String>?): Array<String> {
index f94f3b3d2b2832d8cd28e38110db80689afaec6e..a94096164d4ea5aa7b3737430d63a758d2c8e53e 100644 (file)
@@ -79,15 +79,15 @@ class KeymapManagerImpl(defaultKeymap: DefaultKeymap, factory: SchemeManagerFact
     var ourKeymapManagerInitialized: Boolean = false
   }
 
-  override fun getAllKeymaps() = getKeymaps(Conditions.alwaysTrue<Keymap>()).toTypedArray()
+  override fun getAllKeymaps(): Array<Keymap> = getKeymaps(Conditions.alwaysTrue<Keymap>()).toTypedArray()
 
-  fun getKeymaps(additionalFilter: Condition<Keymap>) = schemeManager.allSchemes.filter { !it.presentableName.startsWith("$") && additionalFilter.value(it)  }
+  fun getKeymaps(additionalFilter: Condition<Keymap>): List<Keymap> = schemeManager.allSchemes.filter { !it.presentableName.startsWith("$") && additionalFilter.value(it)  }
 
-  override fun getKeymap(name: String) = schemeManager.findSchemeByName(name)
+  override fun getKeymap(name: String): Keymap? = schemeManager.findSchemeByName(name)
 
-  override fun getActiveKeymap() = schemeManager.activeScheme
+  override fun getActiveKeymap(): Keymap? = schemeManager.activeScheme
 
-  override fun setActiveKeymap(keymap: Keymap?) = schemeManager.setCurrent(keymap)
+  override fun setActiveKeymap(keymap: Keymap?): Unit = schemeManager.setCurrent(keymap)
 
   override fun bindShortcuts(sourceActionId: String, targetActionId: String) {
     boundShortcuts.put(targetActionId, sourceActionId)
@@ -97,7 +97,7 @@ class KeymapManagerImpl(defaultKeymap: DefaultKeymap, factory: SchemeManagerFact
     boundShortcuts.remove(targetActionId)
   }
 
-  override fun getBoundActions() = boundShortcuts.keys
+  override fun getBoundActions(): MutableSet<String> = boundShortcuts.keys
 
   override fun getActionBinding(actionId: String): String? {
     var visited: MutableSet<String>? = null
@@ -116,7 +116,7 @@ class KeymapManagerImpl(defaultKeymap: DefaultKeymap, factory: SchemeManagerFact
     return if (id == actionId) null else id
   }
 
-  override fun getSchemeManager() = schemeManager
+  override fun getSchemeManager(): SchemeManager<Keymap> = schemeManager
 
   fun setKeymaps(keymaps: List<Keymap>, active: Keymap?, removeCondition: Condition<Keymap>?) {
     schemeManager.setSchemes(keymaps, active, removeCondition)
index 0317a7725186a9def57ad6219eec1df4c503b269..a0d3686d66541ab9ebcb05efae3adeb961271904 100644 (file)
@@ -37,11 +37,11 @@ class MacOSDefaultKeymap(dataHolder: SchemeDataHolder<KeymapImpl>, defaultKeymap
     }
   }
 
-  override fun convertKeyStroke(keyStroke: KeyStroke) = _convertKeyStroke(keyStroke)
+  override fun convertKeyStroke(keyStroke: KeyStroke): KeyStroke = _convertKeyStroke(keyStroke)
 
-  override fun convertMouseShortcut(shortcut: MouseShortcut) = _convertMouseShortcut(shortcut)
+  override fun convertMouseShortcut(shortcut: MouseShortcut): MouseShortcut = _convertMouseShortcut(shortcut)
 
-  override fun convertShortcut(shortcut: Shortcut) = convertShortcutFromParent(shortcut)
+  override fun convertShortcut(shortcut: Shortcut): Shortcut = convertShortcutFromParent(shortcut)
 }
 
 private fun _convertKeyStroke(parentKeyStroke: KeyStroke): KeyStroke = KeyStroke.getKeyStroke(parentKeyStroke.keyCode, mapModifiers(parentKeyStroke.modifiers), parentKeyStroke.isOnKeyRelease)
index c08a47cc8ead79c2205c11075f15f191a6f2e33b..6c2b26d9decd3f0ae88974e57f12af592b866fc4 100644 (file)
@@ -42,6 +42,7 @@ import java.io.File
 import java.io.IOException
 import java.lang.IllegalStateException
 import java.util.*
+import kotlin.collections.HashSet
 import kotlin.collections.set
 
 /**
@@ -50,7 +51,7 @@ import kotlin.collections.set
 object UpdateChecker {
   private val LOG = Logger.getInstance("#com.intellij.openapi.updateSettings.impl.UpdateChecker")
 
-  @JvmField val NOTIFICATIONS = NotificationGroup(IdeBundle.message("update.notifications.title"), NotificationDisplayType.STICKY_BALLOON, true)
+  @JvmField val NOTIFICATIONS: NotificationGroup = NotificationGroup(IdeBundle.message("update.notifications.title"), NotificationDisplayType.STICKY_BALLOON, true)
 
   private const val DISABLED_UPDATE = "disabled_update.txt"
 
@@ -66,7 +67,7 @@ object UpdateChecker {
    * Has no effect on non-bundled or "essential" (i.e. required for one of open projects) plugins.
    */
   @Suppress("MemberVisibilityCanBePrivate")
-  val excludedFromUpdateCheckPlugins = hashSetOf<String>()
+  val excludedFromUpdateCheckPlugins: HashSet<String> = hashSetOf<String>()
 
   private val updateUrl: String
     get() = System.getProperty("idea.updates.url") ?: ApplicationInfoEx.getInstanceEx().updateUrls.checkingUrl
@@ -103,7 +104,7 @@ object UpdateChecker {
    * An immediate check for plugin updates for use from a command line (read "Toolbox").
    */
   @JvmStatic
-  fun getPluginUpdates() =
+  fun getPluginUpdates(): Collection<PluginDownloader>? =
     checkPluginsUpdate(UpdateSettings.getInstance(), EmptyProgressIndicator(), null, BuildNumber.currentVersion())
 
   private fun doUpdateAndShowResult(project: Project?,
@@ -461,7 +462,7 @@ object UpdateChecker {
   @Deprecated("Replaced", ReplaceWith("PermanentInstallationID.get()", "com.intellij.openapi.application.PermanentInstallationID"))
   @JvmStatic
   @Suppress("unused", "UNUSED_PARAMETER")
-  fun getInstallationUID(c: PropertiesComponent) = PermanentInstallationID.get()
+  fun getInstallationUID(c: PropertiesComponent): String = PermanentInstallationID.get()
 
   @JvmStatic
   val disabledToUpdatePlugins: Set<String>
index 877ed2e1afce59af2dda22fd8de33622e61afff6..bc5048f17d39edc22bdb8cf36fe8741d5541779e 100644 (file)
@@ -36,13 +36,13 @@ class Product(node: Element) {
   val codes: Set<String> = node.getChildren("code").map { it.value.trim() }.toSet()
   val channels: List<UpdateChannel> = node.getChildren("channel").map(::UpdateChannel)
 
-  override fun toString() = codes.firstOrNull() ?: "-"
+  override fun toString(): String = codes.firstOrNull() ?: "-"
 }
 
 class UpdateChannel(node: Element) {
   companion object {
-    const val LICENSING_EAP = "eap"
-    const val LICENSING_RELEASE = "release"
+    const val LICENSING_EAP: String = "eap"
+    const val LICENSING_RELEASE: String = "release"
   }
 
   val id: String = node.getAttributeValue("id") ?: throw JDOMException("channel@id missing")
@@ -51,7 +51,7 @@ class UpdateChannel(node: Element) {
   val evalDays: Int = node.getAttributeValue("evalDays")?.toInt() ?: 30
   val builds: List<BuildInfo> = node.getChildren("build").map(::BuildInfo)
 
-  override fun toString() = id
+  override fun toString(): String = id
 }
 
 class BuildInfo(node: Element) {
@@ -83,7 +83,7 @@ class BuildInfo(node: Element) {
   val downloadUrl: String?
     get() = buttons.find(ButtonInfo::isDownload)?.url
 
-  override fun toString() = "${number}/${version}"
+  override fun toString(): String = "${number}/${version}"
 }
 
 class ButtonInfo(node: Element) {
@@ -91,7 +91,7 @@ class ButtonInfo(node: Element) {
   val url: String = node.getAttributeValue("url") ?: throw JDOMException("button@url missing")
   val isDownload: Boolean = node.getAttributeValue("download") != null  // a button marked with this attribute is hidden when a patch is available
 
-  override fun toString() = name
+  override fun toString(): String = name
 }
 
 class PatchInfo(node: Element) {
index 90c432ce0b107fd3997f96516751bbf7e3093b22..ddfcf058912f94ea06f5c6a25a6f6ef1d4f72715 100644 (file)
@@ -7,35 +7,35 @@ import com.intellij.util.xmlb.annotations.OptionTag
 
 class UpdateOptions : BaseState() {
   @get:CollectionBean
-  val pluginHosts by list<String>()
+  val pluginHosts: MutableList<String> by list()
 
   @get:CollectionBean
-  val ignoredBuildNumbers by list<String>()
+  val ignoredBuildNumbers: MutableList<String> by list()
 
   @get:CollectionBean
-  val enabledExternalComponentSources by list<String>()
+  val enabledExternalComponentSources: MutableList<String> by list()
 
   @get:CollectionBean
-  val knownExternalComponentSources by list<String>()
+  val knownExternalComponentSources: MutableList<String> by list()
 
   @get:CollectionBean
-  val externalUpdateChannels by map<String, String>()
+  val externalUpdateChannels: MutableMap<String, String> by map()
 
   @get:OptionTag("CHECK_NEEDED")
-  var isCheckNeeded by property(true)
+  var isCheckNeeded: Boolean by property(true)
 
   @get:OptionTag("LAST_TIME_CHECKED")
-  var lastTimeChecked by property(0L)
+  var lastTimeChecked: Long by property(0L)
 
   @get:OptionTag("LAST_BUILD_CHECKED")
-  var lastBuildChecked by string()
+  var lastBuildChecked: String? by string()
 
   @get:OptionTag("UPDATE_CHANNEL_TYPE")
-  var updateChannelType by string(ChannelStatus.RELEASE.code)
+  var updateChannelType: String? by string(ChannelStatus.RELEASE.code)
 
   @get:OptionTag("SECURE_CONNECTION")
-  var isUseSecureConnection by property(true)
+  var isUseSecureConnection: Boolean by property(true)
 
   @get:OptionTag("THIRD_PARTY_PLUGINS_ALLOWED")
-  var isThirdPartyPluginsAllowed by property(false)
+  var isThirdPartyPluginsAllowed: Boolean by property(false)
 }
\ No newline at end of file
index fa134d53ac987bff4ecf66cf13d5c6751bca1d48..c92b3de96c16d53d8cc8db48ae59b23977864b02 100644 (file)
@@ -34,5 +34,5 @@ package com.intellij.openapi.util
  * Put here extension functions and variables for classes from util/util.rt modules
  */
 
-operator fun <A, B> Pair<A, B>.component1() = this.first
-operator fun <A, B> Pair<A, B>.component2() = this.second
+operator fun <A, B> Pair<A, B>.component1(): A = this.first
+operator fun <A, B> Pair<A, B>.component2(): B = this.second
index fc752370b3c087862b28a924031ca18f9c907c7b..b7d5775e0ef30789ac7cf24dcc960723f0136be3 100644 (file)
@@ -34,11 +34,11 @@ val File.parentSystemIndependentPath: String
   get() = parent.replace(File.separatorChar, '/')
 
 // PathUtilRt.getParentPath returns empty string if no parent path, but in Kotlin "null" is better because elvis operator could be used
-fun getParentPath(path: String) = StringUtil.nullize(PathUtilRt.getParentPath(path))
+fun getParentPath(path: String): String? = StringUtil.nullize(PathUtilRt.getParentPath(path))
 
-fun endsWithSlash(path: String) = path.getOrNull(path.length - 1) == '/'
+fun endsWithSlash(path: String): Boolean = path.getOrNull(path.length - 1) == '/'
 
-fun endsWithName(path: String, name: String) = path.endsWith(name) && (path.length == name.length || path.getOrNull(path.length - name.length - 1) == '/')
+fun endsWithName(path: String, name: String): Boolean = path.endsWith(name) && (path.length == name.length || path.getOrNull(path.length - name.length - 1) == '/')
 
 fun Path.setOwnerPermissions() {
   Files.getFileAttributeView(this, PosixFileAttributeView::class.java)?.let {
index fb6c1e9643cc160e793318d9c16d96c6671ff056..edb90335a6abac396214bc9bb77cf8214bfa511b 100644 (file)
@@ -16,7 +16,7 @@ import java.awt.Rectangle
 class ProjectFrameBounds(private val project: Project) : PersistentStateComponent<FrameInfo>, ModificationTracker {
   companion object {
     @JvmStatic
-    fun getInstance(project: Project) = project.service<ProjectFrameBounds>()
+    fun getInstance(project: Project): ProjectFrameBounds = project.service<ProjectFrameBounds>()
   }
 
   // in device space
@@ -26,7 +26,7 @@ class ProjectFrameBounds(private val project: Project) : PersistentStateComponen
   val isInFullScreen: Boolean
     get() = rawFrameInfo?.fullScreen ?: false
 
-  override fun getState() = rawFrameInfo
+  override fun getState(): FrameInfo? = rawFrameInfo
 
   override fun loadState(state: FrameInfo) {
     rawFrameInfo = state
@@ -49,10 +49,10 @@ class ProjectFrameBounds(private val project: Project) : PersistentStateComponen
 
 class FrameInfo : BaseState() {
   // flat is used due to backward compatibility
-  @get:Property(flat = true) var bounds by property<Rectangle>()
-  @get:Attribute var extendedState by property(Frame.NORMAL)
+  @get:Property(flat = true) var bounds: Rectangle? by property<Rectangle>()
+  @get:Attribute var extendedState: Int by property(Frame.NORMAL)
 
-  @get:Attribute var fullScreen by property(false)
+  @get:Attribute var fullScreen: Boolean by property(false)
 }
 
 fun WindowManagerImpl.getFrameInfoInDeviceSpace(project: Project): FrameInfo? {
index 94a60bdc563e35c9fb08c71861aeb8cdda5f95b9..baedc292a3b53143b1a41b2454ee237231a3e02f 100644 (file)
@@ -29,30 +29,30 @@ private fun canActivateOnStart(id: String?): Boolean {
 class WindowInfoImpl : Cloneable, WindowInfo, BaseState() {
   companion object {
     internal const val TAG = "window_info"
-    const val DEFAULT_WEIGHT = 0.33f
+    const val DEFAULT_WEIGHT: Float = 0.33f
   }
 
   @get:Transient
-  var isRegistered = false
+  var isRegistered: Boolean = false
 
-  override var isActive by property(false)
+  override var isActive: Boolean by property(false)
 
   @get:Attribute(converter = ToolWindowAnchorConverter::class)
   override var anchor: ToolWindowAnchor by property(ToolWindowAnchor.LEFT) { it == ToolWindowAnchor.LEFT }
 
   @get:Attribute("auto_hide")
-  override var isAutoHide by property(false)
+  override var isAutoHide: Boolean by property(false)
 
   /**
    * Bounds of window in "floating" mode. It equals to `null` if floating bounds are undefined.
    */
   @get:Property(flat = true, style = Property.Style.ATTRIBUTE)
-  override var floatingBounds by property<Rectangle?>()
+  override var floatingBounds: Rectangle? by property<Rectangle?>()
 
   /**
    * ID of the tool window
    */
-  var id by string()
+  var id: String? by string()
 
   /**
    * @return type of the tool window in internal (docked or sliding) mode. Actually the tool
@@ -60,27 +60,27 @@ class WindowInfoImpl : Cloneable, WindowInfo, BaseState() {
    * tool window had when it was internal one.
    */
   @get:Attribute("internal_type")
-  var internalType by property(ToolWindowType.DOCKED)
+  var internalType: ToolWindowType by property(ToolWindowType.DOCKED)
 
-  override var type by property(ToolWindowType.DOCKED)
+  override var type: ToolWindowType by property(ToolWindowType.DOCKED)
 
   @get:Attribute("visible")
-  var isVisible by property(false)
+  var isVisible: Boolean by property(false)
 
   @get:Attribute("show_stripe_button")
-  override var isShowStripeButton by property(true)
+  override var isShowStripeButton: Boolean by property(true)
 
   /**
    * Internal weight of tool window. "weight" means how much of internal desktop
    * area the tool window is occupied. The weight has sense if the tool window is docked or
    * sliding.
    */
-  var weight by property(DEFAULT_WEIGHT) { Math.max(0f, Math.min(1f, it)) }
+  var weight: Float by property(DEFAULT_WEIGHT) { Math.max(0f, Math.min(1f, it)) }
 
-  var sideWeight by property(0.5f) { Math.max(0f, Math.min(1f, it)) }
+  var sideWeight: Float by property(0.5f) { Math.max(0f, Math.min(1f, it)) }
 
   @get:Attribute("side_tool")
-  override var isSplit by property(false)
+  override var isSplit: Boolean by property(false)
 
   @get:Attribute("content_ui", converter = ContentUiTypeConverter::class)
   override var contentUiType: ToolWindowContentUiType by property(ToolWindowContentUiType.TABBED) { it == ToolWindowContentUiType.TABBED }
@@ -88,7 +88,7 @@ class WindowInfoImpl : Cloneable, WindowInfo, BaseState() {
   /**
    * Defines order of tool window button inside the stripe.
    */
-  var order by property(-1)
+  var order: Int by property(-1)
 
   @get:Transient
   var isWasRead: Boolean = false
@@ -138,7 +138,7 @@ class WindowInfoImpl : Cloneable, WindowInfo, BaseState() {
     return anchor.hashCode() + id!!.hashCode() + type.hashCode() + order
   }
 
-  override fun toString() = "id: $id, ${super.toString()}"
+  override fun toString(): String = "id: $id, ${super.toString()}"
 }
 
 private class ContentUiTypeConverter : Converter<ToolWindowContentUiType>() {
index 99528e9eb2c5958d1bc43f40a7464561c8083fe7..98f5879ec2002437e67844636fbee11542274e34 100644 (file)
@@ -6,15 +6,12 @@ import java.awt.Component
 import java.awt.Graphics
 import java.awt.Point
 import java.awt.Rectangle
-import javax.swing.Icon
 
 /**
  * @author graann on 08/02/2018
  */
 
 abstract class AdditionalIcon(val myIcon: ActiveIcon) {
-  constructor(icon: Icon) : this(ActiveIcon(icon))
-
   open val tooltip: String? = null
   var x: Int = 0
   val centerPoint: Point
@@ -38,7 +35,7 @@ abstract class AdditionalIcon(val myIcon: ActiveIcon) {
   abstract val active: Boolean
   abstract val available: Boolean
   abstract val action: Runnable
-  open val afterText = true
+  open val afterText: Boolean = true
 
   private fun getIconY(): Int {
     return rectangle.y + rectangle.height / 2 - getIconHeight() / 2 + 1
index dc5f65e79866a3a2efc7c5990138e1b5f6dbbfb7..694eb277c6fbdfcec15e6a648925cdee6a91338b 100644 (file)
@@ -47,7 +47,7 @@ open class BasicOptionButtonUI : OptionButtonUI() {
 
   protected var popup: ListPopup? = null
   protected var showPopupAction: AnAction? = null
-  protected var isPopupShowing = false
+  protected var isPopupShowing: Boolean = false
 
   protected var propertyChangeListener: PropertyChangeListener? = null
   protected var changeListener: ChangeListener? = null
@@ -55,7 +55,7 @@ open class BasicOptionButtonUI : OptionButtonUI() {
   protected var arrowButtonActionListener: ActionListener? = null
   protected var arrowButtonMouseListener: MouseListener? = null
 
-  protected val isSimpleButton get() = optionButton.isSimpleButton
+  protected val isSimpleButton: Boolean get() = optionButton.isSimpleButton
 
   override fun installUI(c: JComponent) {
     _optionButton = c as JBOptionButton
@@ -75,8 +75,8 @@ open class BasicOptionButtonUI : OptionButtonUI() {
     _optionButton = null
   }
 
-  override fun getPreferredSize(c: JComponent) = Dimension(mainButton.preferredSize.width + arrowButton.preferredSize.width,
-                                                           maxOf(mainButton.preferredSize.height, arrowButton.preferredSize.height))
+  override fun getPreferredSize(c: JComponent): Dimension = Dimension(mainButton.preferredSize.width + arrowButton.preferredSize.width,
+                                                                      maxOf(mainButton.preferredSize.height, arrowButton.preferredSize.height))
 
   protected open fun installPopup() {
     showPopupAction = DumbAwareAction.create { _ -> showPopup() }
@@ -265,9 +265,9 @@ open class BasicOptionButtonUI : OptionButtonUI() {
     }
   }
 
-  protected open val showPopupXOffset get() = 0
-  protected open val showPopupBelowLocation get() = RelativePoint(optionButton, Point(showPopupXOffset, optionButton.height + scale(6)))
-  protected open val showPopupAboveLocation get() = RelativePoint(optionButton, Point(showPopupXOffset, -scale(6)))
+  protected open val showPopupXOffset: Int get() = 0
+  protected open val showPopupBelowLocation: RelativePoint get() = RelativePoint(optionButton, Point(showPopupXOffset, optionButton.height + scale(6)))
+  protected open val showPopupAboveLocation: RelativePoint get() = RelativePoint(optionButton, Point(showPopupXOffset, -scale(6)))
 
   protected open fun createPopup(toSelect: Action?, ensureSelection: Boolean): ListPopup {
     val (actionGroup, mapping) = createActionMapping()
@@ -292,7 +292,7 @@ open class BasicOptionButtonUI : OptionButtonUI() {
     return Pair(actionGroup, mapping)
   }
 
-  protected open fun createAnAction(action: Action) = action.getValue(OptionAction.AN_ACTION) as? AnAction ?: ActionDelegate(action)
+  protected open fun createAnAction(action: Action): AnAction = action.getValue(OptionAction.AN_ACTION) as? AnAction ?: ActionDelegate(action)
 
   private fun updateTooltip() {
     val toolTip = if (!isSimpleButton) optionButton.optionTooltipText else optionButton.toolTipText
@@ -306,14 +306,14 @@ open class BasicOptionButtonUI : OptionButtonUI() {
   }
 
   open inner class BaseButton : JButton() {
-    override fun hasFocus() = optionButton.hasFocus()
-    override fun isDefaultButton() = optionButton.isDefaultButton
+    override fun hasFocus(): Boolean = optionButton.hasFocus()
+    override fun isDefaultButton(): Boolean = optionButton.isDefaultButton
 
-    override fun paint(g: Graphics) = if (isSimpleButton) super.paint(g) else cloneAndPaint(g) { paintNotSimple(it) }
-    open fun paintNotSimple(g: Graphics2D) = super.paint(g)
+    override fun paint(g: Graphics): Unit = if (isSimpleButton) super.paint(g) else cloneAndPaint(g) { paintNotSimple(it) }
+    open fun paintNotSimple(g: Graphics2D): Unit = super.paint(g)
 
-    override fun paintBorder(g: Graphics) = if (isSimpleButton) super.paintBorder(g) else cloneAndPaint(g) { paintBorderNotSimple(it) }
-    open fun paintBorderNotSimple(g: Graphics2D) = super.paintBorder(g)
+    override fun paintBorder(g: Graphics): Unit = if (isSimpleButton) super.paintBorder(g) else cloneAndPaint(g) { paintBorderNotSimple(it) }
+    open fun paintBorderNotSimple(g: Graphics2D): Unit = super.paintBorder(g)
   }
 
   open inner class MainButton : BaseButton()
@@ -328,8 +328,8 @@ open class BasicOptionButtonUI : OptionButtonUI() {
       arrowButton.bounds = Rectangle(mainButtonWidth, 0, arrowButton.preferredSize.width, optionButton.height)
     }
 
-    override fun preferredLayoutSize(parent: Container) = parent.preferredSize
-    override fun minimumLayoutSize(parent: Container) = parent.minimumSize
+    override fun preferredLayoutSize(parent: Container): Dimension = parent.preferredSize
+    override fun minimumLayoutSize(parent: Container): Dimension = parent.minimumSize
   }
 
   open inner class OptionButtonPopup(step: ActionPopupStep, dataContext: DataContext, private val ensureSelection: Boolean)
@@ -342,14 +342,14 @@ open class BasicOptionButtonUI : OptionButtonUI() {
       if (ensureSelection) super.afterShow()
     }
 
-    protected val background get() = mainButton.background
+    protected val background: Color? get() = mainButton.background
 
-    override fun createContent() = super.createContent().also {
+    override fun createContent(): JComponent = super.createContent().also {
       list.clearSelection() // prevents first action selection if all actions are disabled
       list.border = JBUI.Borders.empty(2, 0)
     }
 
-    override fun getListElementRenderer() = object : PopupListElementRenderer<Any>(this) {
+    override fun getListElementRenderer(): PopupListElementRenderer<Any> = object : PopupListElementRenderer<Any>(this) {
       override fun getBackground() = this@OptionButtonPopup.background
       override fun createSeparator() = super.createSeparator().apply { border = JBUI.Borders.empty(2, 6) }
       override fun getDefaultItemComponentBorder() = JBUI.Borders.empty(6, 8)
@@ -362,8 +362,8 @@ open class BasicOptionButtonUI : OptionButtonUI() {
                       null, true, defaultSelection, false, true) {
     // if there is no default selection condition - -1 should be returned, this way first enabled action should be selected by
     // OptionButtonPopup.afterShow() (if corresponding ensureSelection parameter is true)
-    override fun getDefaultOptionIndex() = defaultSelection?.let { super.getDefaultOptionIndex() } ?: -1
-    override fun isSpeedSearchEnabled() = false
+    override fun getDefaultOptionIndex(): Int = defaultSelection?.let { super.getDefaultOptionIndex() } ?: -1
+    override fun isSpeedSearchEnabled(): Boolean = false
   }
 
   open inner class ActionDelegate(val action: Action) : DumbAwareAction() {
@@ -384,7 +384,7 @@ open class BasicOptionButtonUI : OptionButtonUI() {
   companion object {
     @Suppress("UNUSED_PARAMETER")
     @JvmStatic
-    fun createUI(c: JComponent) = BasicOptionButtonUI()
+    fun createUI(c: JComponent): BasicOptionButtonUI = BasicOptionButtonUI()
 
     fun paintBackground(g: Graphics, c: JComponent) {
       g.color = c.background
index b538fea8cb3273195a3987601caaff20473d2ae6..551fedfbaa3759adf81ad96658e636c38773999e 100644 (file)
@@ -104,7 +104,7 @@ fun htmlComponent(text: String = "", font: Font = UIUtil.getLabelFont(), backgro
   return pane
 }
 
-fun RadioButton(text: String) = JRadioButton(BundleBase.replaceMnemonicAmpersand(text))
+fun RadioButton(text: String): JRadioButton = JRadioButton(BundleBase.replaceMnemonicAmpersand(text))
 
 fun CheckBox(text: String, selected: Boolean = false, toolTip: String? = null): JCheckBox {
   val component = JCheckBox(BundleBase.replaceMnemonicAmpersand(text), selected)
index 30b15ecd9e52cf4f49d20c9beac85efe12ae4758..0c257f08cb7a3e2285f61cc4d144374f467b817d 100644 (file)
@@ -39,22 +39,22 @@ abstract class Cell {
    * If this constraint is not set the grow weight is set to 0 and the component will not grow (unless some automatic rule is not applied (see [com.intellij.ui.layout.panel])).
    * Grow weight will only be compared against the weights for the same cell.
    */
-  val growX = CCFlags.growX
+  val growX: CCFlags = CCFlags.growX
   @Suppress("unused")
-  val growY = CCFlags.growY
-  val grow = CCFlags.grow
+  val growY: CCFlags = CCFlags.growY
+  val grow: CCFlags = CCFlags.grow
 
   /**
    * Makes the column that the component is residing in grow with `weight`.
    */
-  val pushX = CCFlags.pushX
+  val pushX: CCFlags = CCFlags.pushX
 
   /**
    * Makes the row that the component is residing in grow with `weight`.
    */
   @Suppress("unused")
-  val pushY = CCFlags.pushY
-  val push = CCFlags.push
+  val pushY: CCFlags = CCFlags.pushY
+  val push: CCFlags = CCFlags.push
 
   fun link(text: String, style: UIUtil.ComponentStyle? = null, action: () -> Unit) {
     val result = Link(text, action = action)
index 57016f34588d28d156ed9698dde483f1cd843c23..54ceec37a7e7c500ddfc5028e1aeb5ee481eac6a 100644 (file)
@@ -12,7 +12,7 @@ import javax.swing.ButtonGroup
 import javax.swing.JLabel
 
 class LayoutBuilder @PublishedApi internal constructor(@PublishedApi internal val builder: LayoutBuilderImpl, val buttonGroup: ButtonGroup? = null) {
-  inline fun row(label: String, init: Row.() -> Unit) = row(label = Label(label), init = init)
+  inline fun row(label: String, init: Row.() -> Unit): Row = row(label = Label(label), init = init)
 
   inline fun row(label: JLabel? = null, separated: Boolean = false, init: Row.() -> Unit): Row {
     val row = builder.newRow(label, buttonGroup, separated)
@@ -28,7 +28,7 @@ class LayoutBuilder @PublishedApi internal constructor(@PublishedApi internal va
     builder.noteRow(text, linkHandler)
   }
 
-  fun noteRow(text: String) = noteRow(text, null)
+  fun noteRow(text: String): Unit = noteRow(text, null)
 
   inline fun buttonGroup(init: LayoutBuilder.() -> Unit) {
     LayoutBuilder(builder, ButtonGroup()).init()
index cdd3466253b4ba9b1a74ee1cd2b88ac082ed677c..bd573f683f45143aefed47884119c1175c85b0aa 100644 (file)
@@ -17,7 +17,7 @@ interface LayoutBuilderImpl {
 
   // backward compatibility
   @Deprecated(level = DeprecationLevel.HIDDEN, message = "deprecated")
-  fun newRow(label: JLabel? = null, buttonGroup: ButtonGroup? = null, separated: Boolean = false, indented: Boolean = false) = newRow(label, buttonGroup, separated)
+  fun newRow(label: JLabel? = null, buttonGroup: ButtonGroup? = null, separated: Boolean = false, indented: Boolean = false): Row = newRow(label, buttonGroup, separated)
 
   fun build(container: Container, layoutConstraints: Array<out LCFlags>)
 
index c92fca266119c6387bfa7a48348dcc865977b69a..b1360d43ea179e596fb41c418471d5d1780a6801 100755 (executable)
@@ -67,7 +67,7 @@ open class MigLayout @JvmOverloads constructor(val layoutConstraints: LC = LC(),
   @Transient
   private var dirty = true
 
-  var isDebugEnabled = false
+  var isDebugEnabled: Boolean = false
 
   private val debugMillis: Int
     get() {
@@ -369,7 +369,7 @@ open class MigLayout @JvmOverloads constructor(val layoutConstraints: LC = LC(),
     }
   }
 
-  override fun maximumLayoutSize(parent: Container) = Dimension(Integer.MAX_VALUE, Integer.MAX_VALUE)
+  override fun maximumLayoutSize(parent: Container): Dimension = Dimension(Integer.MAX_VALUE, Integer.MAX_VALUE)
 
   private fun getSizeImpl(parent: Container, sizeType: Int): Dimension {
     checkCache(parent)
index cc3db0dddea25d4cb496b9b124dc94ef44f1053b..cf8c9b540ec6fdbeafaa378d9f3bba2b276583d4 100644 (file)
@@ -24,7 +24,7 @@ object Urls {
   val caseInsensitiveUrlHashingStrategy: TObjectHashingStrategy<Url> by lazy { CaseInsensitiveUrlHashingStrategy() }
 
   @JvmStatic
-  fun newUri(scheme: String?, path: String) = UrlImpl(scheme, null, path)
+  fun newUri(scheme: String?, path: String): UrlImpl = UrlImpl(scheme, null, path)
 
   @JvmStatic
   fun newUrl(scheme: String, authority: String, path: String, rawParameters: String?): UrlImpl {
@@ -73,7 +73,7 @@ object Urls {
   }
 
   @JvmStatic
-  fun parseEncoded(url: String) = parse(url, false)
+  fun parseEncoded(url: String): Url? = parse(url, false)
 
   @JvmStatic
   fun newHttpUrl(authority: String, path: String?): Url {
@@ -198,7 +198,7 @@ object Urls {
   }
 
   @JvmStatic
-  fun newUnparsable(string: String) = UrlImpl(null, null, string, null)
+  fun newUnparsable(string: String): UrlImpl = UrlImpl(null, null, string, null)
 
   @JvmOverloads
   @JvmStatic
index 3670d1d0d30cd6a52a95ef7009bbe72bd14bbcec..42ad6d84063a31381211a3d9659e8595572a7a73 100644 (file)
@@ -285,7 +285,7 @@ fun parseAndCheckIsLocalHost(uri: String?, onlyAnyOrLoopback: Boolean = true, ho
   return false
 }
 
-fun HttpRequest.isRegularBrowser() = userAgent?.startsWith("Mozilla/5.0") ?: false
+fun HttpRequest.isRegularBrowser(): Boolean = userAgent?.startsWith("Mozilla/5.0") ?: false
 
 // forbid POST requests from browser without Origin
 fun HttpRequest.isWriteFromBrowserWithoutOrigin(): Boolean {
@@ -295,7 +295,7 @@ fun HttpRequest.isWriteFromBrowserWithoutOrigin(): Boolean {
 
 fun ByteBuf.readUtf8(): String = toString(Charsets.UTF_8)
 
-fun ByteBuf.writeUtf8(data: CharSequence) = writeCharSequence(data, Charsets.UTF_8)
+fun ByteBuf.writeUtf8(data: CharSequence): Int = writeCharSequence(data, Charsets.UTF_8)
 
 @Suppress("FunctionName")
 fun MultiThreadEventLoopGroup(workerCount: Int, threadFactory: ThreadFactory): MultithreadEventLoopGroup {
index 72590470d0449f78d8cd07ea18c118cf076c3a7a..e533113caf19a080817da9854334d5dfc332d4ae 100644 (file)
@@ -19,4 +19,4 @@ import java.net.InetAddress
 import java.net.InetSocketAddress
 
 @JvmOverloads
-fun loopbackSocketAddress(port: Int = -1) = InetSocketAddress(InetAddress.getLoopbackAddress(), if (port == -1) NetUtils.findAvailableSocketPort() else port)
\ No newline at end of file
+fun loopbackSocketAddress(port: Int = -1): InetSocketAddress = InetSocketAddress(InetAddress.getLoopbackAddress(), if (port == -1) NetUtils.findAvailableSocketPort() else port)
\ No newline at end of file
index f13cbe4c9971d6b279e0e8285bc5b1b04cac2466..db6bdbdaf7fe79b12661b706f92ad57c0ea1db7a 100644 (file)
@@ -29,7 +29,7 @@ import java.io.IOException
 abstract class HttpRequestHandler {
   companion object {
     // Your handler will be instantiated on first user request
-    val EP_NAME = ExtensionPointName.create<HttpRequestHandler>("com.intellij.httpRequestHandler")
+    val EP_NAME: ExtensionPointName<HttpRequestHandler> = ExtensionPointName.create<HttpRequestHandler>("com.intellij.httpRequestHandler")
 
     @JvmStatic
     fun checkPrefix(uri: String, prefix: String): Boolean {
index 352995aa9cc93a2995ea136945e6b5b04ee2b32d..7f7a5a95109a51eda6daf6cfc2d90d466d8c820c 100644 (file)
@@ -15,5 +15,7 @@
  */
 package org.jetbrains.io
 
+import io.netty.bootstrap.Bootstrap
+
 // used in Go
-fun oioClientBootstrap() = com.intellij.util.io.oioClientBootstrap()
\ No newline at end of file
+fun oioClientBootstrap(): Bootstrap = com.intellij.util.io.oioClientBootstrap()
\ No newline at end of file
index d38c0f03dad5d9dcb2e1392a3f421b83ad7241ea..631377bd947b4d703b96e3b3993f18a5d924eac3 100644 (file)
@@ -15,7 +15,7 @@ interface StreamProvider {
   /**
    * Called only on `write`
    */
-  fun isApplicable(fileSpec: String, roamingType: RoamingType = RoamingType.DEFAULT) = true
+  fun isApplicable(fileSpec: String, roamingType: RoamingType = RoamingType.DEFAULT): Boolean = true
 
   /**
    * @param fileSpec
@@ -24,7 +24,7 @@ interface StreamProvider {
    */
   fun write(fileSpec: String, content: ByteArray, size: Int = content.size, roamingType: RoamingType = RoamingType.DEFAULT)
 
-  fun write(path: String, content: BufferExposingByteArrayOutputStream, roamingType: RoamingType = RoamingType.DEFAULT) = write(path, content.internalBuffer, content.size(), roamingType)
+  fun write(path: String, content: BufferExposingByteArrayOutputStream, roamingType: RoamingType = RoamingType.DEFAULT): Unit = write(path, content.internalBuffer, content.size(), roamingType)
 
   /**
    * `true` if provider is applicable for file.
index b67ed24ddc107a88448350d41bf9226442a1840d..47aa268970925518031f23bbcd36cf554d36b4f9 100644 (file)
@@ -105,7 +105,7 @@ abstract class BaseState : SerializationFilter, ModificationTracker {
   /**
    * Empty string is always normalized to null.
    */
-  fun property(defaultValue: String?) = string(defaultValue)
+  fun property(defaultValue: String?): StoredPropertyBase<String?> = string(defaultValue)
 
   /**
    * Empty string is always normalized to null.
@@ -167,7 +167,7 @@ abstract class BaseState : SerializationFilter, ModificationTracker {
     return true
   }
 
-  fun isEqualToDefault() = properties.all { it.isEqualToDefault() }
+  fun isEqualToDefault(): Boolean = properties.all { it.isEqualToDefault() }
 
   @Transient
   override fun getModificationCount(): Long {
@@ -178,9 +178,9 @@ abstract class BaseState : SerializationFilter, ModificationTracker {
     return result
   }
 
-  override fun equals(other: Any?) = this === other || (other is BaseState && properties == other.properties)
+  override fun equals(other: Any?): Boolean = this === other || (other is BaseState && properties == other.properties)
 
-  override fun hashCode() = properties.hashCode()
+  override fun hashCode(): Int = properties.hashCode()
 
   override fun toString(): String {
     if (properties.isEmpty()) {
index cbf7cdb956d175370c4786a4ba6aaea915413d0a..112936228eb708a2696a04c0a3864059b2793c2a 100644 (file)
@@ -21,7 +21,7 @@ import kotlin.reflect.jvm.javaGetter
 
 inline fun <reified T : Any> logger(): Logger = Logger.getInstance(T::class.java)
 
-fun logger(category: String) = Logger.getInstance(category)
+fun logger(category: String): Logger = Logger.getInstance(category)
 
 /**
  * Get logger instance to be used in Kotlin package methods. Usage:
@@ -34,7 +34,7 @@ fun logger(category: String) = Logger.getInstance(category)
  * ```
  * Notice explicit type declaration which can't be skipped in this case.
  */
-fun logger(field: KProperty<Logger>) = Logger.getInstance(field.javaGetter!!.declaringClass)
+fun logger(field: KProperty<Logger>): Logger = Logger.getInstance(field.javaGetter!!.declaringClass)
 
 inline fun Logger.debug(e: Exception? = null, lazyMessage: () -> String) {
   if (isDebugEnabled) {
index e37c0a57f8d8a819d8aa2ad439a964c3f5368af5..c8cd30345206c509178c95208f8a38b30fd3f9f5 100644 (file)
@@ -100,5 +100,5 @@ abstract class ModuleGrouper {
   }
 }
 
-fun isQualifiedModuleNamesEnabled(project: Project) = Registry.`is`("project.qualified.module.names") &&
-                                                      !ModuleManager.getInstance(project).hasModuleGroups()
+fun isQualifiedModuleNamesEnabled(project: Project): Boolean = Registry.`is`("project.qualified.module.names") &&
+                                                               !ModuleManager.getInstance(project).hasModuleGroups()
index e87620e6a6a379d813c6c7261550e7cb3fa944e9..2a1e0362c0adecdb75cd36a6a0fba5e175bf32f1 100644 (file)
@@ -10,7 +10,7 @@ import com.intellij.util.xmlb.annotations.Property
 
 @Property(style = Property.Style.ATTRIBUTE)
 class ExternalStorageConfiguration : BaseState() {
-  var enabled by property(false)
+  var enabled: Boolean by property(false)
 }
 
 @State(name = "ExternalStorageConfigurationManager")
@@ -22,7 +22,7 @@ class ExternalStorageConfigurationManager : PersistentStateComponent<ExternalSto
 
   private var state = ExternalStorageConfiguration()
 
-  override fun getModificationCount() = state.modificationCount
+  override fun getModificationCount(): Long = state.modificationCount
 
   override fun getState(): ExternalStorageConfiguration {
     return state
@@ -32,7 +32,7 @@ class ExternalStorageConfigurationManager : PersistentStateComponent<ExternalSto
     this.state = state
   }
 
-  fun isEnabled() = state.enabled
+  fun isEnabled(): Boolean = state.enabled
 
   /**
    * Internal use only. Call ExternalProjectsManagerImpl.setStoreExternally instead.
index 42886cdeb955567949f604a7907fdb138f2add70..f513838a01462c68c4348d6898956fc8eb4af18f 100644 (file)
@@ -26,7 +26,7 @@ fun <K, V> MutableMap<K, MutableList<V>>.putValue(key: K, value: V) {
   }
 }
 
-fun Collection<*>?.isNullOrEmpty() = this == null || isEmpty()
+fun Collection<*>?.isNullOrEmpty(): Boolean = this == null || isEmpty()
 
 inline fun <T, R> Iterator<T>.computeIfAny(processor: (T) -> R): R? {
   for (item in this) {
@@ -58,7 +58,7 @@ inline fun <T, R> List<T>.computeIfAny(processor: (T) -> R): R? {
   return null
 }
 
-fun <T> List<T>?.nullize() = if (isNullOrEmpty()) null else this
+fun <T> List<T>?.nullize(): List<T>? = if (isNullOrEmpty()) null else this
 
 inline fun <T> Array<out T>.forEachGuaranteed(operation: (T) -> Unit): Unit {
   var errors: MutableList<Throwable>? = null
index bec734408225148f427c8f840222d23f5d798068..d4953e1d729ba6c62690463d01bea70e32c1f6f6 100644 (file)
@@ -94,6 +94,6 @@ fun ByteBuffer.toByteArray(): ByteArray {
   return bytes
 }
 
-fun String.encodeUrlQueryParameter() = URLEncoder.encode(this, Charsets.UTF_8.name())!!
+fun String.encodeUrlQueryParameter(): String = URLEncoder.encode(this, Charsets.UTF_8.name())!!
 
 fun String.decodeBase64(): ByteArray = Base64.getDecoder().decode(this)
\ No newline at end of file
index 2dcf1364a95a5776d7f0b10674a5df7263b80bdd..dc2bdd8d4eb78948636e15ce6b258af9ce731d6c 100644 (file)
@@ -12,7 +12,7 @@ import java.nio.file.attribute.BasicFileAttributes
 import java.nio.file.attribute.FileTime
 import java.util.*
 
-fun Path.exists() = Files.exists(this)
+fun Path.exists(): Boolean = Files.exists(this)
 
 fun Path.createDirectories(): Path {
   // symlink or existing regular file - Java SDK do this check, but with as `isDirectory(dir, LinkOption.NOFOLLOW_LINKS)`, i.e. links are not checked
@@ -144,11 +144,11 @@ fun Path.readBytes(): ByteArray = Files.readAllBytes(this)
 
 fun Path.readText(): String = readBytes().toString(Charsets.UTF_8)
 
-fun Path.readChars() = inputStream().reader().readCharSequence(size().toInt())
+fun Path.readChars(): CharSequence = inputStream().reader().readCharSequence(size().toInt())
 
-fun Path.writeChild(relativePath: String, data: ByteArray) = resolve(relativePath).write(data)
+fun Path.writeChild(relativePath: String, data: ByteArray): Path = resolve(relativePath).write(data)
 
-fun Path.writeChild(relativePath: String, data: String) = writeChild(relativePath, data.toByteArray())
+fun Path.writeChild(relativePath: String, data: String): Path = writeChild(relativePath, data.toByteArray())
 
 @JvmOverloads
 fun Path.write(data: ByteArray, offset: Int = 0, size: Int = data.size): Path {
@@ -191,7 +191,7 @@ fun Path.write(data: String): Path {
   return this
 }
 
-fun Path.size() = Files.size(this)
+fun Path.size(): Long = Files.size(this)
 
 fun Path.basicAttributesIfExists(): BasicFileAttributes? {
   try {
@@ -202,13 +202,13 @@ fun Path.basicAttributesIfExists(): BasicFileAttributes? {
   }
 }
 
-fun Path.sizeOrNull() = basicAttributesIfExists()?.size() ?: -1
+fun Path.sizeOrNull(): Long = basicAttributesIfExists()?.size() ?: -1
 
-fun Path.isHidden() = Files.isHidden(this)
+fun Path.isHidden(): Boolean = Files.isHidden(this)
 
-fun Path.isDirectory() = Files.isDirectory(this)
+fun Path.isDirectory(): Boolean = Files.isDirectory(this)
 
-fun Path.isFile() = Files.isRegularFile(this)
+fun Path.isFile(): Boolean = Files.isRegularFile(this)
 
 fun Path.move(target: Path): Path = Files.move(this, target, StandardCopyOption.REPLACE_EXISTING)
 
index ab965e697b494bd30a5d42f84b85e6923921c175..fe19ebd8199886707f52c08dd6ef888fed7f9973 100644 (file)
@@ -64,7 +64,7 @@ fun Parent.write(output: OutputStream, lineSeparator: String = "\n", filter: JDO
 }
 
 @Throws(IOException::class, JDOMException::class)
-fun loadElement(chars: CharSequence) = loadElement(CharSequenceReader(chars))
+fun loadElement(chars: CharSequence): Element = loadElement(CharSequenceReader(chars))
 
 @Throws(IOException::class, JDOMException::class)
 fun loadElement(reader: Reader): Element = loadDocument(reader).detachRootElement()
@@ -77,7 +77,7 @@ fun loadElement(path: Path): Element = loadDocument(path.inputStream().bufferedR
 
 fun loadDocument(reader: Reader): Document = reader.use { getSaxBuilder().build(it) }
 
-fun Element?.isEmpty() = this == null || JDOMUtil.isEmpty(this)
+fun Element?.isEmpty(): Boolean = this == null || JDOMUtil.isEmpty(this)
 
 fun Element.getOrCreate(name: String): Element {
   var element = getChild(name)
index f5ec6d9f2b75a16c3c36dfadffb21cbb60d78b27..db52d65fb8e811735569ef4d9ad63f73a230a4fb 100644 (file)
@@ -120,7 +120,7 @@ open class AsyncPromise<T : Any?> : InternalPromiseUtil.BasePromise<T>(), Cancel
     }
   }
 
-  fun setError(error: String) = setError(createError(error))
+  fun setError(error: String): Boolean = setError(createError(error))
 
   override fun cancel() {
     setError(InternalPromiseUtil.OBSOLETE_ERROR)
@@ -240,7 +240,7 @@ open class AsyncPromise<T : Any?> : InternalPromiseUtil.BasePromise<T>(), Cancel
     }
   }
 
-  override fun getValue() = valueRef.get()
+  override fun getValue(): PromiseValue<T>? = valueRef.get()
 }
 
 private class CompoundConsumer<T>(c1: Consumer<in T>, c2: Consumer<in T>) : Consumer<T> {
index a92008528deed1c186e1e747b9d7997dda3f501a..89ed3e97502067b1a6f64f61507d99a9d8d391b7 100644 (file)
@@ -57,21 +57,21 @@ fun <T> cancelledPromise(): Promise<T> = InternalPromiseUtil.CANCELLED_PROMISE.v
 interface ObsolescentFunction<Param, Result> : Function<Param, Result>, Obsolescent
 
 abstract class ValueNodeAsyncFunction<PARAM, RESULT>(private val node: Obsolescent) : Function<PARAM, Promise<RESULT>>, Obsolescent {
-  override fun isObsolete() = node.isObsolete
+  override fun isObsolete(): Boolean = node.isObsolete
 }
 
 abstract class ObsolescentConsumer<T>(private val obsolescent: Obsolescent) : Obsolescent, Consumer<T> {
-  override fun isObsolete() = obsolescent.isObsolete
+  override fun isObsolete(): Boolean = obsolescent.isObsolete
 }
 
-inline fun <T, SUB_RESULT> Promise<T>.then(obsolescent: Obsolescent, crossinline handler: (T) -> SUB_RESULT) = then(object : ObsolescentFunction<T, SUB_RESULT> {
+inline fun <T, SUB_RESULT> Promise<T>.then(obsolescent: Obsolescent, crossinline handler: (T) -> SUB_RESULT): Promise<SUB_RESULT> = then(object : ObsolescentFunction<T, SUB_RESULT> {
   override fun `fun`(param: T) = handler(param)
 
   override fun isObsolete() = obsolescent.isObsolete
 })
 
 
-inline fun <T> Promise<T>.onSuccess(node: Obsolescent, crossinline handler: (T) -> Unit) = onSuccess(object : ObsolescentConsumer<T>(node) {
+inline fun <T> Promise<T>.onSuccess(node: Obsolescent, crossinline handler: (T) -> Unit): Promise<T> = onSuccess(object : ObsolescentConsumer<T>(node) {
   override fun accept(param: T) = handler(param)
 })
 
@@ -84,7 +84,7 @@ inline fun Promise<*>.processed(node: Obsolescent, crossinline handler: () -> Un
 }
 
 @Suppress("UNCHECKED_CAST")
-inline fun Promise<*>.doneRun(crossinline handler: () -> Unit) = onSuccess { handler() }
+inline fun Promise<*>.doneRun(crossinline handler: () -> Unit): Promise<out Any> = onSuccess { handler() }
 
 @Suppress("UNCHECKED_CAST")
 inline fun <T> Promise<*>.thenRun(crossinline handler: () -> T): Promise<T> = (this as Promise<Any?>).then { handler() }
@@ -95,7 +95,7 @@ inline fun Promise<*>.processedRun(crossinline handler: () -> Unit): Promise<*>
 }
 
 
-inline fun <T, SUB_RESULT> Promise<T>.thenAsync(node: Obsolescent, crossinline handler: (T) -> Promise<SUB_RESULT>) = thenAsync(object : ValueNodeAsyncFunction<T, SUB_RESULT>(node) {
+inline fun <T, SUB_RESULT> Promise<T>.thenAsync(node: Obsolescent, crossinline handler: (T) -> Promise<SUB_RESULT>): Promise<SUB_RESULT> = thenAsync(object : ValueNodeAsyncFunction<T, SUB_RESULT>(node) {
   override fun `fun`(param: T) = handler(param)
 })
 
@@ -106,13 +106,13 @@ inline fun <T> Promise<T>.thenAsyncAccept(node: Obsolescent, crossinline handler
   })
 }
 
-inline fun <T> Promise<T>.thenAsyncAccept(crossinline handler: (T) -> Promise<*>) = thenAsync(Function<T, Promise<Any?>> { param ->
+inline fun <T> Promise<T>.thenAsyncAccept(crossinline handler: (T) -> Promise<*>): Promise<Any?> = thenAsync(Function<T, Promise<Any?>> { param ->
   @Suppress("UNCHECKED_CAST")
   (return@Function handler(param) as Promise<Any?>)
 })
 
 
-inline fun Promise<*>.onError(node: Obsolescent, crossinline handler: (Throwable) -> Unit) = onError(object : ObsolescentConsumer<Throwable>(node) {
+inline fun Promise<*>.onError(node: Obsolescent, crossinline handler: (Throwable) -> Unit): Promise<out Any> = onError(object : ObsolescentConsumer<Throwable>(node) {
   override fun accept(param: Throwable) = handler(param)
 })
 
index 95ddc67ddd55391087a1602be89e9f0ebb5b3c84..e5ee6580f3218b17c43d1c2c2a120d6cb30ef99e 100644 (file)
@@ -34,7 +34,7 @@ fun serializeElementToBinary(element: Element, out: OutputStream) {
   BinaryXmlWriter(DataOutputStream(out)).write(element)
 }
 
-fun deserializeElementFromBinary(input: InputStream) = BinaryXmlReader(DataInputStream(input)).read()
+fun deserializeElementFromBinary(input: InputStream): Element = BinaryXmlReader(DataInputStream(input)).read()
 
 private class BinaryXmlReader(private val input: DataInputStream) {
   private val strings = ArrayList<String>()
index 675046ccb9e14015ddf43f0daac1e5eb739f16f8..bf2b4d08aa664be1bdd0f313b5e51eee84d164ab 100644 (file)
@@ -4,7 +4,7 @@ package com.intellij.configurationStore
 import com.intellij.openapi.components.*
 import com.intellij.util.messages.Topic
 
-val STORAGE_TOPIC = Topic("STORAGE_LISTENER", StorageManagerListener::class.java, Topic.BroadcastDirection.TO_PARENT)
+val STORAGE_TOPIC: Topic<StorageManagerListener> = Topic("STORAGE_LISTENER", StorageManagerListener::class.java, Topic.BroadcastDirection.TO_PARENT)
 
 interface StateStorageManager {
   val macroSubstitutor: TrackingPathMacroSubstitutor?
index 9a2f9df7fb3067745392f44cc3875852d57d9778..ef4b4c6b110b84f2880969830902e92b7059e8cd 100644 (file)
@@ -12,7 +12,7 @@ import com.intellij.openapi.extensions.ExtensionPointName
  */
 interface StreamProviderFactory {
   companion object {
-    val EP_NAME = ExtensionPointName.create<StreamProviderFactory>("com.intellij.streamProviderFactory")
+    val EP_NAME: ExtensionPointName<StreamProviderFactory> = ExtensionPointName.create<StreamProviderFactory>("com.intellij.streamProviderFactory")
   }
 
   fun createProvider(componentManager: ComponentManager, storageManager: StateStorageManager): StreamProvider? = null
index e890a82d568026def82f899cabbf5f7fcc8e8f5b..caee60e4288ec527e65cfb4a528f031a227c25d9 100644 (file)
@@ -21,13 +21,13 @@ interface SchemeNameToFileName {
   fun schemeNameToFileName(name: String): String
 }
 
-val OLD_NAME_CONVERTER = object : SchemeNameToFileName {
+val OLD_NAME_CONVERTER: SchemeNameToFileName = object : SchemeNameToFileName {
   override fun schemeNameToFileName(name: String) = FileUtil.sanitizeFileName(name, true)
 }
-val CURRENT_NAME_CONVERTER = object : SchemeNameToFileName {
+val CURRENT_NAME_CONVERTER: SchemeNameToFileName = object : SchemeNameToFileName {
   override fun schemeNameToFileName(name: String) = FileUtil.sanitizeFileName(name, false)
 }
-val MODERN_NAME_CONVERTER = object : SchemeNameToFileName {
+val MODERN_NAME_CONVERTER: SchemeNameToFileName = object : SchemeNameToFileName {
   override fun schemeNameToFileName(name: String) = sanitizeFileName(name)
 }
 
@@ -69,11 +69,11 @@ abstract class LazySchemeProcessor<SCHEME, MUTABLE_SCHEME : SCHEME>(private val
                             isBundled: Boolean = false): MUTABLE_SCHEME
   override fun writeScheme(scheme: MUTABLE_SCHEME): Element? = (scheme as SerializableScheme).writeScheme()
 
-  open fun isSchemeFile(name: CharSequence) = true
+  open fun isSchemeFile(name: CharSequence): Boolean = true
 
-  open fun isSchemeDefault(scheme: MUTABLE_SCHEME, digest: ByteArray) = false
+  open fun isSchemeDefault(scheme: MUTABLE_SCHEME, digest: ByteArray): Boolean = false
 
-  open fun isSchemeEqualToBundled(scheme: MUTABLE_SCHEME) = false
+  open fun isSchemeEqualToBundled(scheme: MUTABLE_SCHEME): Boolean = false
 }
 
 class DigestOutputStream(val digest: MessageDigest) : OutputStream() {
@@ -85,7 +85,7 @@ class DigestOutputStream(val digest: MessageDigest) : OutputStream() {
     digest.update(b, off, len)
   }
 
-  override fun toString() = "[Digest Output Stream] $digest"
+  override fun toString(): String = "[Digest Output Stream] $digest"
 }
 
 fun Element.digest(): ByteArray {
@@ -101,7 +101,7 @@ abstract class SchemeWrapper<out T>(name: String) : ExternalizableSchemeAdapter(
   val scheme: T
     get() = lazyScheme.value
 
-  override fun getSchemeState() = if (lazyScheme.isInitialized()) SchemeState.POSSIBLY_CHANGED else SchemeState.UNCHANGED
+  override fun getSchemeState(): SchemeState = if (lazyScheme.isInitialized()) SchemeState.POSSIBLY_CHANGED else SchemeState.UNCHANGED
 
   init {
     this.name = name
@@ -109,7 +109,7 @@ abstract class SchemeWrapper<out T>(name: String) : ExternalizableSchemeAdapter(
 }
 
 abstract class LazySchemeWrapper<T>(name: String, dataHolder: SchemeDataHolder<SchemeWrapper<T>>, protected val writer: (scheme: T) -> Element) : SchemeWrapper<T>(name) {
-  protected val dataHolder = AtomicReference(dataHolder)
+  protected val dataHolder: AtomicReference<SchemeDataHolder<SchemeWrapper<T>>> = AtomicReference(dataHolder)
 
   override final fun writeScheme(): Element {
     val dataHolder = dataHolder.get()
@@ -119,9 +119,9 @@ abstract class LazySchemeWrapper<T>(name: String, dataHolder: SchemeDataHolder<S
 }
 
 class InitializedSchemeWrapper<out T : Scheme>(scheme: T, private val writer: (scheme: T) -> Element) : SchemeWrapper<T>(scheme.name) {
-  override val lazyScheme = lazyOf(scheme)
+  override val lazyScheme: Lazy<T> = lazyOf(scheme)
 
-  override fun writeScheme() = writer(scheme)
+  override fun writeScheme(): Element = writer(scheme)
 }
 
 fun unwrapState(element: Element, project: Project, iprAdapter: SchemeManagerIprProvider?, schemeManager: SchemeManager<*>): Element? {
index 18ed41bf8833b440dbb90cc86015863c82b8f689..447dad253c02185b26507dabd8bec3501582d891 100644 (file)
@@ -21,34 +21,34 @@ import com.intellij.util.messages.MessageBus
 import org.jetbrains.annotations.NotNull
 
 class EmptyModuleManager(private val project: Project, messageBus: MessageBus) : ModuleManager() {
-  override fun hasModuleGroups() = false
+  override fun hasModuleGroups(): Boolean = false
 
-  override fun newModule(filePath: String, moduleTypeId: String) = throw UnsupportedOperationException()
+  override fun newModule(filePath: String, moduleTypeId: String): Nothing = throw UnsupportedOperationException()
 
-  override fun loadModule(filePath: String) = throw UnsupportedOperationException()
+  override fun loadModule(filePath: String): Nothing = throw UnsupportedOperationException()
 
   override fun disposeModule(module: Module) {
   }
 
-  override fun getModules() = emptyArray<Module>()
+  override fun getModules(): Array<Module> = emptyArray<Module>()
 
-  override fun findModuleByName(name: String) = null
+  override fun findModuleByName(name: String): Nothing? = null
 
-  override fun getSortedModules() = emptyArray<Module>()
+  override fun getSortedModules(): Array<Module> = emptyArray<Module>()
 
-  override fun moduleDependencyComparator() = throw UnsupportedOperationException()
+  override fun moduleDependencyComparator(): Nothing = throw UnsupportedOperationException()
 
-  override fun getModuleDependentModules(module: Module) = emptyList<Module>()
+  override fun getModuleDependentModules(module: Module): List<Module> = emptyList<Module>()
 
-  override fun isModuleDependent(@NotNull module: Module, @NotNull onModule: Module) = false
+  override fun isModuleDependent(@NotNull module: Module, @NotNull onModule: Module): Boolean = false
 
-  override fun moduleGraph() = moduleGraph(true)
+  override fun moduleGraph(): Nothing = moduleGraph(true)
 
-  override fun moduleGraph(includeTests: Boolean) = throw UnsupportedOperationException()
+  override fun moduleGraph(includeTests: Boolean): Nothing = throw UnsupportedOperationException()
 
-  override fun getModifiableModel() = throw UnsupportedOperationException()
+  override fun getModifiableModel(): Nothing = throw UnsupportedOperationException()
 
-  override fun getModuleGroupPath(module: Module) = emptyArray<String>()
+  override fun getModuleGroupPath(module: Module): Array<String> = emptyArray<String>()
 
   override fun setUnloadedModules(unloadedModuleNames: MutableList<String>) {
   }
@@ -57,11 +57,11 @@ class EmptyModuleManager(private val project: Project, messageBus: MessageBus) :
     return createGrouper(project, model)
   }
 
-  override fun getAllModuleDescriptions() = emptyList<ModuleDescription>()
+  override fun getAllModuleDescriptions(): List<ModuleDescription> = emptyList<ModuleDescription>()
 
-  override fun getUnloadedModuleDescriptions() = emptyList<UnloadedModuleDescription>()
+  override fun getUnloadedModuleDescriptions(): List<UnloadedModuleDescription> = emptyList<UnloadedModuleDescription>()
 
-  override fun getUnloadedModuleDescription(moduleName: String) = null
+  override fun getUnloadedModuleDescription(moduleName: String): Nothing? = null
 
   override fun removeUnloadedModules(unloadedModules: MutableCollection<UnloadedModuleDescription>) {
   }
index 8f1f2ee3231765a165b7776e1b79609fd8b08d73..c7988f61fbeb400108a875e26f2685aefca1bbce 100644 (file)
@@ -23,13 +23,13 @@ import com.intellij.openapi.roots.ModuleRootManager
  * @author nik
  */
 class LoadedModuleDescriptionImpl(private val module: Module): LoadedModuleDescription {
-  override fun getModule() = module
+  override fun getModule(): Module = module
 
-  override fun getName() = module.name
+  override fun getName(): String = module.name
 
-  override fun getDependencyModuleNames() = ModuleRootManager.getInstance(module).dependencyModuleNames.asList()
+  override fun getDependencyModuleNames(): List<String> = ModuleRootManager.getInstance(module).dependencyModuleNames.asList()
 
-  override fun equals(other: Any?) = other is LoadedModuleDescriptionImpl && module == other.module
+  override fun equals(other: Any?): Boolean = other is LoadedModuleDescriptionImpl && module == other.module
 
-  override fun hashCode() = module.hashCode()
+  override fun hashCode(): Int = module.hashCode()
 }
\ No newline at end of file
index 0b0f269a128498b0caeba9ab151061ef7d93830f..7f5395c3f8e032d5a6adab7f9235eb09ab62d07a 100644 (file)
@@ -38,7 +38,7 @@ class ModulePointerImpl : ModulePointer {
     this.lock = lock
   }
 
-  override fun getModule() = lock.read { module }
+  override fun getModule(): Module? = lock.read { module }
 
   override fun getModuleName(): String = lock.read { module?.name ?: moduleName!! }
 
@@ -62,5 +62,5 @@ class ModulePointerImpl : ModulePointer {
     moduleName = newName
   }
 
-  override fun toString() = "moduleName: $moduleName, module: $module"
+  override fun toString(): String = "moduleName: $moduleName, module: $module"
 }
index bddfbcce04076f580193fd2e40ead6f88002a699..01cb24d6833c4c810c6b7aad2a43a3db5f303d4e 100644 (file)
@@ -50,7 +50,7 @@ class ModulePointerManagerImpl(private val project: Project) : ModulePointerMana
     })
   }
 
-  override fun getState() = ModuleRenamingHistoryState().apply {
+  override fun getState(): ModuleRenamingHistoryState = ModuleRenamingHistoryState().apply {
     lock.read {
       oldToNewName.putAll(this@ModulePointerManagerImpl.oldToNewName)
     }
index 86454252b3fbda6f9d574c3493a3faeece0f08c1..024e82f58fb6c93efd70a550022b619d1681a4fb 100644 (file)
@@ -10,5 +10,5 @@ class ModuleRenamingHistoryState {
   @Property(surroundWithTag = false)
   @XMap(entryTagName = "module", keyAttributeName = "old-name", valueAttributeName = "new-name")
   @JvmField
-  var oldToNewName = HashMap<String, String>()
+  var oldToNewName: HashMap<String, String> = HashMap<String, String>()
 }
\ No newline at end of file
index 3dca6d491e009bab53a0979157545ad33517379d..4ce190ec5108313a54f5f0bfd59995b5f991384d 100644 (file)
@@ -32,17 +32,17 @@ import java.nio.file.Paths
 class UnloadedModuleDescriptionImpl(val modulePath: ModulePath,
                                     private val dependencyModuleNames: List<String>,
                                     private val contentRoots: List<VirtualFilePointer>) : UnloadedModuleDescription {
-  override fun getGroupPath() = modulePath.group?.split(ModuleManagerImpl.MODULE_GROUP_SEPARATOR) ?: emptyList()
+  override fun getGroupPath(): List<String> = modulePath.group?.split(ModuleManagerImpl.MODULE_GROUP_SEPARATOR) ?: emptyList()
 
-  override fun getName() = modulePath.moduleName
+  override fun getName(): String = modulePath.moduleName
 
-  override fun getContentRoots() = contentRoots
+  override fun getContentRoots(): List<VirtualFilePointer> = contentRoots
 
-  override fun getDependencyModuleNames() = dependencyModuleNames
+  override fun getDependencyModuleNames(): List<String> = dependencyModuleNames
 
-  override fun equals(other: Any?) = other is UnloadedModuleDescriptionImpl && name == other.name
+  override fun equals(other: Any?): Boolean = other is UnloadedModuleDescriptionImpl && name == other.name
 
-  override fun hashCode() = name.hashCode()
+  override fun hashCode(): Int = name.hashCode()
 
   companion object {
     @JvmStatic
index 2906e2fa5907b9bdd790aab93e376eae0fce761f..bc5b8cce0e03c70f7d5e4ce65608823522286451 100644 (file)
@@ -7,13 +7,13 @@ import kotlin.properties.Delegates
 abstract class ExternalizableSchemeAdapter : ExternalizableScheme {
   private var myName: String by Delegates.notNull()
 
-  override fun getName() = myName
+  override fun getName(): String = myName
 
   override fun setName(value: String) {
     myName = value
   }
 
-  override fun toString() = name
+  override fun toString(): String = name
 }
 
 abstract class BaseSchemeProcessor<SCHEME, MUTABLE_SCHEME : SCHEME> : NonLazySchemeProcessor<SCHEME, MUTABLE_SCHEME>()
index fafdc7764baa626cfbeab323206e42eb411a73be..b7e95c43c07b1a364466c8c0db536bef64ebda66 100644 (file)
@@ -7,7 +7,7 @@ import java.io.File
 
 abstract class SchemeManager<T> {
   companion object {
-    const val EDITABLE_COPY_PREFIX = "_@user_"
+    const val EDITABLE_COPY_PREFIX: String = "_@user_"
 
     @JvmStatic
     fun getDisplayName(scheme: Scheme): String {
@@ -48,7 +48,7 @@ abstract class SchemeManager<T> {
     addScheme(scheme as T, replaceExisting)
   }
 
-  fun addScheme(scheme: T) = addScheme(scheme, true)
+  fun addScheme(scheme: T): Unit = addScheme(scheme, true)
 
   abstract fun addScheme(scheme: T, replaceExisting: Boolean)
 
index 29777b5096cc13ed8444e6ef4cfb2a798386a633..0c3bbaf28550b850654389584eba45f252f94c7d 100644 (file)
@@ -20,10 +20,10 @@ interface ExternalizableScheme : Scheme {
 abstract class SchemeManagerFactory {
   companion object {
     @JvmStatic
-    fun getInstance() = ServiceManager.getService(SchemeManagerFactory::class.java)!!
+    fun getInstance(): SchemeManagerFactory = ServiceManager.getService(SchemeManagerFactory::class.java)!!
 
     @JvmStatic
-    fun getInstance(project: Project) = ServiceManager.getService(project, SchemeManagerFactory::class.java)!!
+    fun getInstance(project: Project): SchemeManagerFactory = ServiceManager.getService(project, SchemeManagerFactory::class.java)!!
   }
 
   /**
@@ -56,7 +56,7 @@ abstract class SchemeProcessor<SCHEME, in MUTABLE_SCHEME: SCHEME> {
     return (scheme as Scheme).name
   }
 
-  open fun isExternalizable(scheme: SCHEME) = scheme is ExternalizableScheme
+  open fun isExternalizable(scheme: SCHEME): Boolean = scheme is ExternalizableScheme
 
   /**
    * Element will not be modified, it is safe to return non-cloned instance.
@@ -82,7 +82,7 @@ abstract class SchemeProcessor<SCHEME, in MUTABLE_SCHEME: SCHEME> {
    * If scheme implements [com.intellij.configurationStore.SerializableScheme], this method will be called only if [com.intellij.configurationStore.SerializableScheme.getSchemeState] returns `null`
    */
   @Suppress("KDocUnresolvedReference")
-  open fun getState(scheme: SCHEME) = SchemeState.POSSIBLY_CHANGED
+  open fun getState(scheme: SCHEME): SchemeState = SchemeState.POSSIBLY_CHANGED
 
   /**
    * May be called from any thread - EDT is not guaranteed.
index f97c4dc55992e71aea65f698efb792afe7bdaa02..c6ba4c7d62b89583b94a97ed31df897e81d5d113 100644 (file)
@@ -20,9 +20,9 @@ open class ProjectLibraryTable(val project: Project) : LibraryTableBase() {
     fun getInstance(project: Project): LibraryTable = project.service<ProjectLibraryTable>()
   }
 
-  override fun getTableLevel() = LibraryTablesRegistrar.PROJECT_LEVEL
+  override fun getTableLevel(): String = LibraryTablesRegistrar.PROJECT_LEVEL
 
-  override fun getPresentation() = PROJECT_LIBRARY_TABLE_PRESENTATION
+  override fun getPresentation(): LibraryTablePresentation = PROJECT_LIBRARY_TABLE_PRESENTATION
 
   class LibraryStateSplitter : StateSplitterEx() {
     override fun splitState(state: Element): MutableList<Pair<Element, String>> = StateSplitterEx.splitState(state, LibraryImpl.LIBRARY_NAME_ATTR)
index 66be616093ab8bbbcaa19c54377b0e75728b3b4b..24de5d2b5f736725adae13825964abe03339114b 100755 (executable)
@@ -25,13 +25,13 @@ interface Breakpoint {
     /**
      * This value is used when the corresponding parameter is absent
      */
-    const val EMPTY_VALUE = -1
+    const val EMPTY_VALUE: Int = -1
 
     /**
      * A breakpoint has this ID if it does not reflect an actual breakpoint in a
      * JavaScript VM debugger.
      */
-    const val INVALID_ID = -1
+    const val INVALID_ID: Int = -1
   }
 
   val target: BreakpointTarget
@@ -61,7 +61,7 @@ interface Breakpoint {
    * Be aware! V8 doesn't provide reliable debugger API, so, sometimes actual locations is empty - in this case this methods return "true".
    * V8 debugger doesn't report about resolved breakpoint if it is happened after initial breakpoint set. So, you cannot trust "actual locations".
    */
-  fun isActualLineCorrect() = true
+  fun isActualLineCorrect(): Boolean = true
 }
 
 /**
index 06cee00048dc2698ae694ca5d77471265cd6e8f3..2f1e355ed815653675434d95152cf4575aab70ad 100644 (file)
@@ -62,7 +62,7 @@ interface BreakpointManager {
 
   fun removeAll(): Promise<*>
 
-  fun getMuteMode() = BreakpointManager.MUTE_MODE.ONE
+  fun getMuteMode(): MUTE_MODE = BreakpointManager.MUTE_MODE.ONE
 
   /**
    * Flushes the breakpoint parameter changes (set* methods) into the browser
index 5b7a5e63fe69029b0bfda3b33e93720c7666d48a..91175cf310731679a9d7547a002e06b5b12878e8 100644 (file)
@@ -15,7 +15,7 @@ import org.jetbrains.concurrency.rejectedPromise
 import java.util.concurrent.ConcurrentMap
 
 abstract class BreakpointManagerBase<T : BreakpointBase<*>> : BreakpointManager {
-  override val breakpoints = ContainerUtil.newConcurrentSet<T>()
+  override val breakpoints: MutableSet<T> = ContainerUtil.newConcurrentSet<T>()
 
   protected val breakpointDuplicationByTarget: ConcurrentMap<T, T> = ConcurrentCollectionFactory.createMap<T, T>(object : TObjectHashingStrategy<T> {
     override fun computeHashCode(b: T): Int {
@@ -96,7 +96,7 @@ abstract class BreakpointManagerBase<T : BreakpointBase<*>> : BreakpointManager
   }
 
   @Suppress("UNCHECKED_CAST")
-  override fun flush(breakpoint: Breakpoint) = (breakpoint as T).flush(this)
+  override fun flush(breakpoint: Breakpoint): Promise<*> = (breakpoint as T).flush(this)
 
   override fun enableBreakpoints(enabled: Boolean): Promise<*> = rejectedPromise<Any?>("Unsupported")
 
@@ -116,16 +116,16 @@ class DummyBreakpointManager : BreakpointManager {
     throw UnsupportedOperationException()
   }
 
-  override fun remove(breakpoint: Breakpoint) = nullPromise()
+  override fun remove(breakpoint: Breakpoint): Promise<*> = nullPromise()
 
   override fun addBreakpointListener(listener: BreakpointListener) {
   }
 
-  override fun removeAll() = nullPromise()
+  override fun removeAll(): Promise<*> = nullPromise()
 
-  override fun flush(breakpoint: Breakpoint) = nullPromise()
+  override fun flush(breakpoint: Breakpoint): Promise<*> = nullPromise()
 
-  override fun enableBreakpoints(enabled: Boolean) = nullPromise()
+  override fun enableBreakpoints(enabled: Boolean): Promise<*> = nullPromise()
 
   override fun setBreakOnFirstStatement() {
   }
index 7405906ff01fd40a720e177223a337ebaf993a37..46d7348601790484d7092f5fb8e55364effc7070 100644 (file)
@@ -17,7 +17,7 @@ package org.jetbrains.debugger
 
 import com.intellij.openapi.util.NotNullLazyValue
 
-const val RECEIVER_NAME = "this"
+const val RECEIVER_NAME: String = "this"
 
 @Deprecated("")
 /**
index 27396dc46b67c4c90506d2c3c461c4872cf39e43..cc622e9a930df614f9c15d97a9dcf77eb49ad563 100644 (file)
@@ -20,7 +20,7 @@ import org.jetbrains.concurrency.thenRun
 import org.jetbrains.debugger.values.ValueManager
 
 abstract class EvaluateContextBase<VALUE_MANAGER : ValueManager>(val valueManager: VALUE_MANAGER) : EvaluateContext {
-  override fun withValueManager(objectGroup: String) = this
+  override fun withValueManager(objectGroup: String): EvaluateContextBase<VALUE_MANAGER> = this
 
-  override fun refreshOnDone(promise: Promise<*>) = promise.thenRun { valueManager.clearCaches() }
+  override fun refreshOnDone(promise: Promise<*>): Promise<Unit> = promise.thenRun { valueManager.clearCaches() }
 }
\ No newline at end of file
index 93e303c22e97dca4a45c0eb3a857a13d94fce528..2fd3f0aa8d7de5f54bcf64a9cb15c25aab4f1a1b 100644 (file)
@@ -26,9 +26,9 @@ class ObjectPropertyImpl(name: String,
                          valueModifier: ValueModifier? = null,
                          private val flags: Int = 0) : VariableImpl(name, value, valueModifier), ObjectProperty {
   companion object {
-    val WRITABLE = 0x01
-    val CONFIGURABLE = 0x02
-    val ENUMERABLE = 0x04
+    val WRITABLE: Int = 0x01
+    val CONFIGURABLE: Int = 0x02
+    val ENUMERABLE: Int = 0x04
   }
 
   override val isWritable: Boolean
index 812fbc4f1921694a0951c52fcea3955f439bb0db..2dff49250bb69b5934505c7308ac92eb92b5631d 100755 (executable)
@@ -25,14 +25,14 @@ abstract class ScriptBase(override val type: Script.Type,
                           line: Int,
                           override val column: Int,
                           override val endLine: Int) : UserDataHolderBase(), Script {
-  override val line = Math.max(line, 0)
+  override val line: Int = Math.max(line, 0)
 
   @SuppressWarnings("UnusedDeclaration")
   private @Volatile var source: Promise<String>? = null
 
   override var sourceMap: SourceMap? = null
 
-  override fun toString() = "[url=$url, lineRange=[$line;$endLine]]"
+  override fun toString(): String = "[url=$url, lineRange=[$line;$endLine]]"
 
   override val isWorker: Boolean = false
 }
\ No newline at end of file
index 4ee4157001252acbbee245bf0fa7710af2ffe0da..a8cf29a657119c65db7a67a056825055c5756824 100644 (file)
@@ -20,7 +20,7 @@ import com.intellij.util.Url
 import org.jetbrains.concurrency.Promise
 import org.jetbrains.debugger.values.FunctionValue
 
-const val VM_SCHEME = "vm"
+const val VM_SCHEME: String = "vm"
 
 interface ScriptManager {
   fun getSource(script: Script): Promise<String>
@@ -31,7 +31,7 @@ interface ScriptManager {
 
   fun forEachScript(scriptProcessor: (Script) -> Boolean)
 
-  fun forEachScript(scriptProcessor: Processor<Script>) = forEachScript { scriptProcessor.process(it)}
+  fun forEachScript(scriptProcessor: Processor<Script>): Unit = forEachScript { scriptProcessor.process(it)}
 
   fun getScript(function: FunctionValue): Promise<Script>
 
index fb99c7dff0c291c7e2af2e069b3eb2c0c8a7e10b..53da11b9c32c3483f459d9b3e43b8375cd7a22ba 100644 (file)
@@ -18,9 +18,10 @@ package org.jetbrains.debugger
 import com.intellij.util.Url
 import com.intellij.util.Urls
 import com.intellij.util.containers.ContainerUtil
+import java.util.concurrent.ConcurrentMap
 
 abstract class ScriptManagerBaseEx<SCRIPT : ScriptBase> : ScriptManagerBase<SCRIPT>() {
-  protected val idToScript = ContainerUtil.newConcurrentMap<String, SCRIPT>()
+  protected val idToScript: ConcurrentMap<String, SCRIPT> = ContainerUtil.newConcurrentMap<String, SCRIPT>()
 
   override final fun forEachScript(scriptProcessor: (Script) -> Boolean) {
     for (script in idToScript.values) {
@@ -30,14 +31,14 @@ abstract class ScriptManagerBaseEx<SCRIPT : ScriptBase> : ScriptManagerBase<SCRI
     }
   }
 
-  override final fun findScriptById(id: String) = idToScript[id]
+  override final fun findScriptById(id: String): SCRIPT? = idToScript[id]
 
   fun clear(listener: DebugEventListener) {
     idToScript.clear()
     listener.scriptsCleared()
   }
 
-  override final fun findScriptByUrl(rawUrl: String) = findScriptByUrl(rawUrlToOurUrl(rawUrl))
+  override final fun findScriptByUrl(rawUrl: String): SCRIPT? = findScriptByUrl(rawUrlToOurUrl(rawUrl))
 
   override final fun findScriptByUrl(url: Url): SCRIPT? {
     for (script in idToScript.values) {
@@ -48,5 +49,5 @@ abstract class ScriptManagerBaseEx<SCRIPT : ScriptBase> : ScriptManagerBase<SCRI
     return null
   }
 
-  open fun rawUrlToOurUrl(rawUrl: String) = Urls.parseEncoded(rawUrl)!!
+  open fun rawUrlToOurUrl(rawUrl: String): Url = Urls.parseEncoded(rawUrl)!!
 }
\ No newline at end of file
index 69386e9c4dc587f8c2cfd47a2bfb900c4c3b8540..bfd9d98e65e847680a41adea0ea019da1a87fea8 100644 (file)
@@ -25,7 +25,7 @@ class ScriptRegExpBreakpointTarget(private val regExp: String, val language: Str
     }
   }
 
-  override fun toString() = regExp
+  override fun toString(): String = regExp
 
   override fun equals(other: Any?): Boolean {
     if (this === other) {
@@ -37,5 +37,5 @@ class ScriptRegExpBreakpointTarget(private val regExp: String, val language: Str
     return regExp == (other as ScriptRegExpBreakpointTarget).regExp
   }
 
-  override fun hashCode() = regExp.hashCode()
+  override fun hashCode(): Int = regExp.hashCode()
 }
\ No newline at end of file
index f24864cb4f1a656ab87bf4908828c24a9d05eac4..275ba088215e75519134b28249c1dff5987bf77c 100644 (file)
@@ -27,7 +27,7 @@ interface SuspendContextManager<CALL_FRAME : CallFrame> {
 
   val contextOrFail: SuspendContext<CALL_FRAME>
 
-  fun isContextObsolete(context: SuspendContext<*>) = this.context !== context
+  fun isContextObsolete(context: SuspendContext<*>): Boolean = this.context !== context
 
   fun setOverlayMessage(message: String?)
 
index 4f780e09c32aecbf1840ae2f5a441c71bf87b985..5e2d7f4d9400df80887c04b678e5fc1ea2d75abc 100644 (file)
@@ -8,9 +8,9 @@ import org.jetbrains.concurrency.resolvedPromise
 import java.util.concurrent.atomic.AtomicReference
 
 abstract class SuspendContextManagerBase<T : SuspendContextBase<CALL_FRAME>, CALL_FRAME : CallFrame> : SuspendContextManager<CALL_FRAME> {
-  val contextRef = AtomicReference<T>()
+  val contextRef: AtomicReference<T> = AtomicReference<T>()
 
-  protected val suspendCallback = AtomicReference<AsyncPromise<Void?>>()
+  protected val suspendCallback: AtomicReference<AsyncPromise<Void?>> = AtomicReference<AsyncPromise<Void?>>()
 
   protected abstract val debugListener: DebugEventListener
 
@@ -50,7 +50,7 @@ abstract class SuspendContextManagerBase<T : SuspendContextBase<CALL_FRAME>, CAL
   override val contextOrFail: T
     get() = contextRef.get() ?: throw IllegalStateException("No current suspend context")
 
-  override fun suspend() = suspendCallback.get() ?: if (context == null) doSuspend() else resolvedPromise()
+  override fun suspend(): Promise<out Any?> = suspendCallback.get() ?: if (context == null) doSuspend() else resolvedPromise()
 
   protected abstract fun doSuspend(): Promise<*>
 
@@ -59,9 +59,9 @@ abstract class SuspendContextManagerBase<T : SuspendContextBase<CALL_FRAME>, CAL
 
   override fun restartFrame(callFrame: CALL_FRAME): Promise<Boolean> = restartFrame(callFrame, contextOrFail)
 
-  protected open fun restartFrame(callFrame: CALL_FRAME, currentContext: T) = rejectedPromise<Boolean>("Unsupported")
+  protected open fun restartFrame(callFrame: CALL_FRAME, currentContext: T): Promise<Boolean> = rejectedPromise<Boolean>("Unsupported")
 
-  override fun canRestartFrame(callFrame: CallFrame) = false
+  override fun canRestartFrame(callFrame: CallFrame): Boolean = false
 
-  override val isRestartFrameSupported = false
+  override val isRestartFrameSupported: Boolean = false
 }
\ No newline at end of file
index 79760f417625f2ab821ff2f025ad25bbdea1957f..bdd75c63283066222e00ae08a842fad0f6dd0169 100644 (file)
@@ -28,7 +28,7 @@ object ValueModifierUtil {
   fun setValue(variable: Variable,
                newValue: String,
                evaluateContext: EvaluateContext,
-               modifier: ValueModifier) = evaluateContext.evaluate(newValue)
+               modifier: ValueModifier): Promise<Any?> = evaluateContext.evaluate(newValue)
     .thenAsyncAccept { modifier.setValue(variable, it.value, evaluateContext) }
 
   fun evaluateGet(variable: Variable,
index 71a9b07453859c535c6ce5691337acc140e365b9..6af35c761238796d9cbbe94ba62f47c0e8e14cb9 100644 (file)
@@ -19,7 +19,7 @@ import com.intellij.openapi.util.UserDataHolderBase
 import com.intellij.util.containers.ContainerUtil
 
 abstract class VmBase(override val debugListener: DebugEventListener) : Vm, AttachStateManager, UserDataHolderBase() {
-  override val evaluateContext by lazy(LazyThreadSafetyMode.NONE) { computeEvaluateContext() }
+  override val evaluateContext: EvaluateContext? by lazy(LazyThreadSafetyMode.NONE) { computeEvaluateContext() }
 
   override val attachStateManager: AttachStateManager = this
 
index 039edb5f092704d47aff897ca02b5e7d02faaee5..62713ca9b962da6950949c753cc8585bdb63f5e0 100644 (file)
@@ -116,7 +116,7 @@ abstract class MappingList(private val mappings: List<MappingEntry>) : Mappings
   }
 
   // todo honor Google Chrome bug related to paused location
-  override fun get(line: Int, column: Int) = mappings.getOrNull(indexOf(line, column))
+  override fun get(line: Int, column: Int): MappingEntry? = mappings.getOrNull(indexOf(line, column))
 
   private fun getNext(index: Int) = mappings.getOrNull(index + 1)
 
@@ -163,7 +163,7 @@ abstract class MappingList(private val mappings: List<MappingEntry>) : Mappings
     return if (nextMapping == null) document.getLineEndOffset(getLine(mapping)) else lineStartOffset + getColumn(nextMapping)
   }
 
-  override fun getByIndex(index: Int) = mappings.get(index)
+  override fun getByIndex(index: Int): MappingEntry = mappings.get(index)
 
   // entries will be processed in this list order
   fun processMappingsInLine(line: Int, entryProcessor: MappingsProcessorInLine): Boolean {
index 605130d4d6fe14c4fd352b957cbff4e2662d942a..c4a5ebfc9335da03c050dad92a57a242dfa86607 100644 (file)
@@ -49,12 +49,12 @@ class NestedSourceMap(private val childMap: SourceMap, private val parentMap: So
     return result
   }
 
-  override fun findSourceIndex(sourceFile: VirtualFile, localFileUrlOnly: Boolean) = parentMap.findSourceIndex(sourceFile, localFileUrlOnly)
+  override fun findSourceIndex(sourceFile: VirtualFile, localFileUrlOnly: Boolean): Int = parentMap.findSourceIndex(sourceFile, localFileUrlOnly)
 
   override fun findSourceIndex(sourceUrls: List<Url>,
                                sourceFile: VirtualFile?,
                                resolver: Lazy<SourceFileResolver?>?,
-                               localFileUrlOnly: Boolean) = parentMap.findSourceIndex(sourceUrls, sourceFile, resolver, localFileUrlOnly)
+                               localFileUrlOnly: Boolean): Int = parentMap.findSourceIndex(sourceUrls, sourceFile, resolver, localFileUrlOnly)
 
   override fun processSourceMappingsInLine(sourceIndex: Int, sourceLine: Int, mappingProcessor: MappingsProcessorInLine): Boolean {
     val childSourceMappings = childMap.findSourceMappings(sourceIndex)
index 0a4e7749746d4df683cc33ad56a8841750b57794..283a5b6cbd632996eb1e0e8723802467490f2e36 100644 (file)
@@ -41,7 +41,7 @@ interface SourceMap {
     return if (sourceIndex >= 0) findSourceMappings(sourceIndex) else null
   }
 
-  fun getSourceLineByRawLocation(rawLine: Int, rawColumn: Int) = generatedMappings.get(rawLine, rawColumn)?.sourceLine ?: -1
+  fun getSourceLineByRawLocation(rawLine: Int, rawColumn: Int): Int = generatedMappings.get(rawLine, rawColumn)?.sourceLine ?: -1
 
   fun findSourceIndex(sourceFile: VirtualFile, localFileUrlOnly: Boolean): Int
 
@@ -71,9 +71,9 @@ class OneLevelSourceMap(override val outFile: String?,
   }
 
   // returns SourceMappingList
-  override fun findSourceMappings(sourceIndex: Int) = sourceIndexToMappings.get(sourceIndex)!!
+  override fun findSourceMappings(sourceIndex: Int): MappingList = sourceIndexToMappings.get(sourceIndex)!!
 
-  override fun findSourceIndex(sourceFile: VirtualFile, localFileUrlOnly: Boolean) = sourceResolver.findSourceIndexByFile(sourceFile, localFileUrlOnly)
+  override fun findSourceIndex(sourceFile: VirtualFile, localFileUrlOnly: Boolean): Int = sourceResolver.findSourceIndexByFile(sourceFile, localFileUrlOnly)
 
   override fun processSourceMappingsInLine(sourceIndex: Int, sourceLine: Int, mappingProcessor: MappingsProcessorInLine): Boolean {
     return findSourceMappings(sourceIndex).processMappingsInLine(sourceLine, mappingProcessor)
index 4580046644cedfb7cd07f6be3f114a64cb74bb4d..f9a1b240f12414cf5301276b909c7a8f9041f3bc 100644 (file)
@@ -27,7 +27,7 @@ class SourceResolver(private val rawSources: List<String>,
                      private val sourceContents: List<String?>?,
                      baseUrlIsFile: Boolean = true) {
   companion object {
-    fun isAbsolute(path: String) = path.startsWith('/') || (SystemInfo.isWindows && (path.length > 2 && path[1] == ':'))
+    fun isAbsolute(path: String): Boolean = path.startsWith('/') || (SystemInfo.isWindows && (path.length > 2 && path[1] == ':'))
   }
 
   val canonicalizedUrls: Array<Url> by lazy {
@@ -66,7 +66,7 @@ class SourceResolver(private val rawSources: List<String>,
     return if (sourceIndex < 0 || sourceIndex >= sourceContents!!.size) null else sourceContents[sourceIndex]
   }
 
-  fun getSourceIndex(url: Url) = canonicalizedUrlToSourceIndex[url]
+  fun getSourceIndex(url: Url): Int = canonicalizedUrlToSourceIndex[url]
 
   fun getRawSource(entry: MappingEntry): String? {
     val index = entry.source
@@ -123,7 +123,7 @@ class SourceResolver(private val rawSources: List<String>,
     return -1
   }
 
-  fun getUrlIfLocalFile(entry: MappingEntry) = canonicalizedUrls.getOrNull(entry.source)?.let { if (it.isInLocalFileSystem) it else null }
+  fun getUrlIfLocalFile(entry: MappingEntry): Url? = canonicalizedUrls.getOrNull(entry.source)?.let { if (it.isInLocalFileSystem) it else null }
 }
 
 fun canonicalizePath(url: String, baseUrl: Url, baseUrlIsFile: Boolean): String {
index 45f53de9ffffce36567be7735e94d42d4a28784b..2825bc1e52ac277ec18135ea37cb6d7c8cde2425 100644 (file)
@@ -41,5 +41,5 @@ interface FunctionValue : ObjectValue {
    * Method could be called (it is normal and expected) for unresolved function.
    * It must return quickly. Return [com.intellij.util.ThreeState.UNSURE] otherwise.
    */
-  fun hasScopes() = ThreeState.UNSURE
+  fun hasScopes(): ThreeState = ThreeState.UNSURE
 }
index 3bcf71215ff9fac6ca529ee5b02eb1322d10ab8e..55917c42369eb517cafb8929f1167200b5ff0a92 100755 (executable)
@@ -44,10 +44,10 @@ interface ObjectValue : Value {
   /**
    * It must return quickly. Return [com.intellij.util.ThreeState.UNSURE] otherwise.
    */
-  fun hasProperties() = ThreeState.UNSURE
+  fun hasProperties(): ThreeState = ThreeState.UNSURE
 
   /**
    * It must return quickly. Return [com.intellij.util.ThreeState.UNSURE] otherwise.
    */
-  fun hasIndexedProperties() = ThreeState.NO
+  fun hasIndexedProperties(): ThreeState = ThreeState.NO
 }
\ No newline at end of file
index 5985ae92a7d2a92413b43d2c93e9d867cd638ded..f56adc43d73b46fe3050875acf059da22785ed6a 100644 (file)
@@ -18,7 +18,7 @@ abstract class ObjectValueBase<VALUE_LOADER : ValueManager>(type: ValueType) : V
     override fun isObsolete() = obsolescent.isObsolete || childrenManager.valueManager.isObsolete
   }
 
-  override fun getProperties(names: List<String>, evaluateContext: EvaluateContext, obsolescent: Obsolescent) = properties
+  override fun getProperties(names: List<String>, evaluateContext: EvaluateContext, obsolescent: Obsolescent): Promise<List<Variable>> = properties
     .thenAsync(object : MyObsolescentAsyncFunction<List<Variable>, List<Variable>>(obsolescent) {
       override fun `fun`(variables: List<Variable>) = getSpecifiedProperties(variables, names, evaluateContext)
     })
index 3bf59f7d2138a3c4264b346ee58374d539698763..995e905c6799bae69af9b11ecdb98772eae8bd2a 100644 (file)
@@ -24,16 +24,16 @@ open class PrimitiveValue(type: ValueType, override val valueString: String) : V
   }
 
   companion object {
-    val NA_N_VALUE = "NaN"
-    val INFINITY_VALUE = "Infinity"
+    val NA_N_VALUE: String = "NaN"
+    val INFINITY_VALUE: String = "Infinity"
 
     @JvmField
-    val NULL = PrimitiveValue(ValueType.NULL, "null")
+    val NULL: PrimitiveValue = PrimitiveValue(ValueType.NULL, "null")
     @JvmField
-    val UNDEFINED = PrimitiveValue(ValueType.UNDEFINED, "undefined")
+    val UNDEFINED: PrimitiveValue = PrimitiveValue(ValueType.UNDEFINED, "undefined")
 
-    val NAN = PrimitiveValue(ValueType.NUMBER, NA_N_VALUE)
-    val INFINITY = PrimitiveValue(ValueType.NUMBER, INFINITY_VALUE)
+    val NAN: PrimitiveValue = PrimitiveValue(ValueType.NUMBER, NA_N_VALUE)
+    val INFINITY: PrimitiveValue = PrimitiveValue(ValueType.NUMBER, INFINITY_VALUE)
 
     private val TRUE = PrimitiveValue(ValueType.BOOLEAN, "true")
     private val FALSE = PrimitiveValue(ValueType.BOOLEAN, "false")
index 6e4d68ee3b41d4ca1c677adc20b7b405f73478f1..2b6fbc557433133169646a917b77a9412dd657b4 100644 (file)
@@ -33,9 +33,9 @@ abstract class ValueManager() : Obsolescent {
     cacheStamp.incrementAndGet()
   }
 
-  fun getCacheStamp() = cacheStamp.get()
+  fun getCacheStamp(): Int = cacheStamp.get()
 
-  override final fun isObsolete() = obsolete
+  override final fun isObsolete(): Boolean = obsolete
 
   fun markObsolete() {
     obsolete = true
index 6f816e913783ef4c3c82299ec0fcc2ff72b4068b..7aca0c95b22797e5e2756ff26ea72577d09ee03f 100644 (file)
@@ -43,6 +43,6 @@ enum class ValueType {
     get() = this == OBJECT || this == ARRAY || this == FUNCTION || this == NODE
 
   companion object {
-    fun fromIndex(index: Int) = VALUE_TYPES.get(index)
+    fun fromIndex(index: Int): ValueType = VALUE_TYPES.get(index)
   }
 }
\ No newline at end of file
index 211fa682a5f01fe128f1467fe8ef23071cc9340e..029fc92140d671c6812bdd65b77722cf4207bc0a 100644 (file)
@@ -7,7 +7,7 @@ import org.jetbrains.concurrency.createError
 import org.jetbrains.jsonProtocol.Request
 import java.util.concurrent.atomic.AtomicInteger
 
-val LOG = Logger.getInstance(CommandProcessor::class.java)
+val LOG: Logger = Logger.getInstance(CommandProcessor::class.java)
 
 abstract class CommandProcessor<INCOMING, INCOMING_WITH_SEQ : Any, SUCCESS_RESPONSE : Any?> : CommandSenderBase<SUCCESS_RESPONSE>(),
                                                                                               MessageManager.Handler<Request<*>, INCOMING, INCOMING_WITH_SEQ, SUCCESS_RESPONSE>,
@@ -15,7 +15,7 @@ abstract class CommandProcessor<INCOMING, INCOMING_WITH_SEQ : Any, SUCCESS_RESPO
                                                                                               MessageProcessor {
   private val currentSequence = AtomicInteger()
   @Suppress("LeakingThis")
-  protected val messageManager = MessageManager(this)
+  protected val messageManager: MessageManager<Request<*>, INCOMING, INCOMING_WITH_SEQ, SUCCESS_RESPONSE> = MessageManager(this)
 
   override fun cancelWaitingRequests() {
     messageManager.cancelWaitingRequests()
@@ -53,5 +53,5 @@ interface RequestCallback<SUCCESS_RESPONSE> {
 
   fun onError(error: Throwable)
 
-  fun onError(error: String) = onError(createError(error))
+  fun onError(error: String): Unit = onError(createError(error))
 }
\ No newline at end of file
index 2a398afc1d7a699a3029a98fc3500fd347aad74b..325ab3317202660662e59c8f9947bcbd8c6c5e8b 100644 (file)
@@ -42,7 +42,7 @@ class MessageManager<REQUEST, INCOMING, INCOMING_WITH_SEQ : Any, SUCCESS>(privat
 
     fun getSequence(incomingWithSeq: INCOMING_WITH_SEQ): Int = throw AbstractMethodError()
 
-    fun getSequence(incomingWithSeq: INCOMING_WITH_SEQ, incoming: INCOMING) = getSequence(incomingWithSeq)
+    fun getSequence(incomingWithSeq: INCOMING_WITH_SEQ, incoming: INCOMING): Int = getSequence(incomingWithSeq)
 
     fun acceptNonSequence(incoming: INCOMING)
 
@@ -107,7 +107,7 @@ class MessageManager<REQUEST, INCOMING, INCOMING_WITH_SEQ : Any, SUCCESS>(privat
     }
   }
 
-  fun getCallbackAndRemove(id: Int) = callbackMap.remove(id) ?: throw IllegalArgumentException("Cannot find callback with id $id")
+  fun getCallbackAndRemove(id: Int): RequestCallback<SUCCESS> = callbackMap.remove(id) ?: throw IllegalArgumentException("Cannot find callback with id $id")
 
   fun cancelWaitingRequests() {
     // we should call them in the order they have been submitted
index c4e3189ad64f2d086ade087cc72bba127e42c977..b92be849755a625785c02695ebd5c095c32379ca 100644 (file)
  */
 package org.jetbrains.rpc
 
-const val CONNECTION_CLOSED_MESSAGE = "Connection closed"
+const val CONNECTION_CLOSED_MESSAGE: String = "Connection closed"
 
 abstract class MessageManagerBase {
-  @Volatile protected var closed = false
+  @Volatile protected var closed: Boolean = false
 
   protected fun rejectIfClosed(callback: RequestCallback<*>): Boolean {
     if (closed) {
index 73703b9f5543185e6199d60fe71595c7871da756..d5a63deb54c242626f8903bc142ea88b143c2315 100644 (file)
@@ -13,17 +13,17 @@ import org.jetbrains.debugger.values.Value
 import javax.swing.Icon
 
 open class BasicDebuggerViewSupport : MemberFilter, DebuggerViewSupport {
-  protected val defaultMemberFilterPromise = resolvedPromise<MemberFilter>(this)
+  protected val defaultMemberFilterPromise: Promise<MemberFilter> = resolvedPromise<MemberFilter>(this)
 
-  override fun propertyNamesToString(list: List<String>, quotedAware: Boolean) = ValueModifierUtil.propertyNamesToString(list, quotedAware)
+  override fun propertyNamesToString(list: List<String>, quotedAware: Boolean): String = ValueModifierUtil.propertyNamesToString(list, quotedAware)
 
-  override fun computeObjectPresentation(value: ObjectValue, variable: Variable, context: VariableContext, node: XValueNode, icon: Icon) = VariableView.setObjectPresentation(value, icon, node)
+  override fun computeObjectPresentation(value: ObjectValue, variable: Variable, context: VariableContext, node: XValueNode, icon: Icon): Unit = VariableView.setObjectPresentation(value, icon, node)
 
   override fun computeArrayPresentation(value: Value, variable: Variable, context: VariableContext, node: XValueNode, icon: Icon) {
     VariableView.setArrayPresentation(value, context, icon, node)
   }
 
-  override fun getMemberFilter(context: VariableContext) = defaultMemberFilterPromise
+  override fun getMemberFilter(context: VariableContext): Promise<MemberFilter> = defaultMemberFilterPromise
 
   override fun computeReceiverVariable(context: VariableContext, callFrame: CallFrame, node: XCompositeNode): Promise<*> {
     return callFrame.receiverVariable
index 7e9685704e7d037468f604ef560f1cd13b734270..7b6bb7f4ce0a6f873b00a5092608d49134a60d3c 100644 (file)
@@ -36,7 +36,7 @@ class CallFrameView @JvmOverloads constructor(val callFrame: CallFrame,
 
   private var evaluator: XDebuggerEvaluator? = null
 
-  override fun getEqualityObject() = callFrame.equalityObject
+  override fun getEqualityObject(): Any = callFrame.equalityObject
 
   override fun computeChildren(node: XCompositeNode) {
     node.setAlreadySorted(true)
@@ -46,12 +46,12 @@ class CallFrameView @JvmOverloads constructor(val callFrame: CallFrame,
   override val evaluateContext: EvaluateContext
     get() = callFrame.evaluateContext
 
-  override fun watchableAsEvaluationExpression() = true
+  override fun watchableAsEvaluationExpression(): Boolean = true
 
   override val memberFilter: Promise<MemberFilter>
     get() = viewSupport.getMemberFilter(this)
 
-  fun getMemberFilter(scope: Scope) = createVariableContext(scope, this, callFrame).memberFilter
+  fun getMemberFilter(scope: Scope): Promise<MemberFilter> = createVariableContext(scope, this, callFrame).memberFilter
 
   override fun getEvaluator(): XDebuggerEvaluator? {
     if (evaluator == null) {
@@ -60,7 +60,7 @@ class CallFrameView @JvmOverloads constructor(val callFrame: CallFrame,
     return evaluator
   }
 
-  override fun getSourcePosition() = sourceInfo
+  override fun getSourcePosition(): SourceInfo? = sourceInfo
 
   override fun customizePresentation(component: ColoredTextContainer) {
     if (sourceInfo == null) {
index d2faf47996dd5eeeacdd5e916032a7e0e1120dc6..efeadea0dadb7b23b27ac6189d80053b8a5bb8ad 100644 (file)
@@ -21,6 +21,7 @@ import org.jetbrains.debugger.connection.RemoteVmConnection
 import org.jetbrains.debugger.connection.VmConnection
 import java.util.concurrent.ConcurrentMap
 import java.util.concurrent.atomic.AtomicBoolean
+import javax.swing.event.HyperlinkListener
 
 interface MultiVmDebugProcess {
   val mainVm: Vm?
@@ -48,8 +49,8 @@ abstract class DebugProcessImpl<out C : VmConnection<*>>(session: XDebugSession,
   protected val repeatStepInto: AtomicBoolean = AtomicBoolean()
   @Volatile var lastStep: StepAction? = null
   @Volatile protected var lastCallFrame: CallFrame? = null
-  @Volatile protected var isForceStep = false
-  @Volatile protected var disableDoNotStepIntoLibraries = false
+  @Volatile protected var isForceStep: Boolean = false
+  @Volatile protected var disableDoNotStepIntoLibraries: Boolean = false
 
   protected val urlToFileCache: ConcurrentMap<Url, VirtualFile> = ContainerUtil.newConcurrentMap<Url, VirtualFile>()
 
@@ -63,14 +64,14 @@ abstract class DebugProcessImpl<out C : VmConnection<*>>(session: XDebugSession,
   protected val realProcessHandler: ProcessHandler?
     get() = executionResult?.processHandler
 
-  override final fun getSmartStepIntoHandler() = smartStepIntoHandler
+  override final fun getSmartStepIntoHandler(): XSmartStepIntoHandler<*>? = smartStepIntoHandler
 
-  override final fun getBreakpointHandlers() = when (connection.state.status) {
+  override final fun getBreakpointHandlers(): Array<out XBreakpointHandler<*>> = when (connection.state.status) {
     ConnectionStatus.DISCONNECTED, ConnectionStatus.DETACHED, ConnectionStatus.CONNECTION_FAILED -> XBreakpointHandler.EMPTY_ARRAY
     else -> _breakpointHandlers
   }
 
-  override final fun getEditorsProvider() = editorsProvider
+  override final fun getEditorsProvider(): XDebuggerEditorsProvider = editorsProvider
 
   val vm: Vm?
     get() = connection.vm
@@ -110,9 +111,9 @@ abstract class DebugProcessImpl<out C : VmConnection<*>>(session: XDebugSession,
     lastCallFrame = vm.suspendContextManager.context?.topFrame
   }
 
-  override final fun checkCanPerformCommands() = activeOrMainVm != null
+  override final fun checkCanPerformCommands(): Boolean = activeOrMainVm != null
 
-  override final fun isValuesCustomSorted() = true
+  override final fun isValuesCustomSorted(): Boolean = true
 
   override final fun startStepOver(context: XSuspendContext?) {
     val vm = context.vm
@@ -146,7 +147,7 @@ abstract class DebugProcessImpl<out C : VmConnection<*>>(session: XDebugSession,
   }
 
   // some VM (firefox for example) doesn't implement step out correctly, so, we need to fix it
-  protected open fun isVmStepOutCorrect() = true
+  protected open fun isVmStepOutCorrect(): Boolean = true
 
   override fun resume(context: XSuspendContext?) {
     continueVm(context.vm, StepAction.CONTINUE)
@@ -158,7 +159,7 @@ abstract class DebugProcessImpl<out C : VmConnection<*>>(session: XDebugSession,
 
   @Suppress("unused")
   @Deprecated("Pass vm explicitly", ReplaceWith("continueVm(vm!!, stepAction)"))
-  protected open fun continueVm(stepAction: StepAction) = continueVm(activeOrMainVm!!, stepAction)
+  protected open fun continueVm(stepAction: StepAction): Promise<*>? = continueVm(activeOrMainVm!!, stepAction)
 
   /**
    * You can override this method to avoid SuspendContextManager implementation, but it is not recommended.
@@ -194,11 +195,11 @@ abstract class DebugProcessImpl<out C : VmConnection<*>>(session: XDebugSession,
       .onError(RejectErrorReporter(session, "Cannot pause"))
   }
 
-  override final fun getCurrentStateMessage() = connection.state.message
+  override final fun getCurrentStateMessage(): String = connection.state.message
 
-  override final fun getCurrentStateHyperlinkListener() = connection.state.messageLinkListener
+  override final fun getCurrentStateHyperlinkListener(): HyperlinkListener? = connection.state.messageLinkListener
 
-  override fun doGetProcessHandler() = executionResult?.processHandler ?: object : DefaultDebugProcessHandler() { override fun isSilentlyDestroyOnClose() = true }
+  override fun doGetProcessHandler(): ProcessHandler = executionResult?.processHandler ?: object : DefaultDebugProcessHandler() { override fun isSilentlyDestroyOnClose() = true }
 
   fun saveResolvedFile(url: Url, file: VirtualFile) {
     urlToFileCache.putIfAbsent(url, file)
@@ -210,7 +211,7 @@ abstract class DebugProcessImpl<out C : VmConnection<*>>(session: XDebugSession,
 
   open fun getLocationsForBreakpoint(vm: Vm, breakpoint: XLineBreakpoint<*>): List<Location> = throw UnsupportedOperationException()
 
-  override fun isLibraryFrameFilterSupported() = true
+  override fun isLibraryFrameFilterSupported(): Boolean = true
 
   // todo make final (go plugin compatibility)
   override fun checkCanInitBreakpoints(): Boolean {
index 8fc6d70e033995658ce8e7da97e384e320955d16..874e13dce88d7e4363cf61d3b0d0dbb83232adab 100644 (file)
@@ -37,21 +37,21 @@ interface DebuggerViewSupport {
   /**
    * [org.jetbrains.debugger.values.FunctionValue] is special case and handled by SDK
    */
-  fun canNavigateToSource(variable: Variable, context: VariableContext) = false
+  fun canNavigateToSource(variable: Variable, context: VariableContext): Boolean = false
 
   fun computeSourcePosition(name: String, value: Value?, variable: Variable, context: VariableContext, navigatable: XNavigatable) {
   }
 
-  fun computeInlineDebuggerData(name: String, variable: Variable, context: VariableContext, callback: XInlineDebuggerDataCallback) = ThreeState.UNSURE
+  fun computeInlineDebuggerData(name: String, variable: Variable, context: VariableContext, callback: XInlineDebuggerDataCallback): ThreeState = ThreeState.UNSURE
 
   // return null if you don't need to add additional properties
   fun computeAdditionalObjectProperties(value: ObjectValue, variable: Variable, context: VariableContext, node: XCompositeNode): Promise<Any?>? = null
 
   fun getMemberFilter(context: VariableContext): Promise<MemberFilter>
 
-  fun transformErrorOnGetUsedReferenceValue(value: Value?, error: String?) = value
+  fun transformErrorOnGetUsedReferenceValue(value: Value?, error: String?): Value? = value
 
-  fun isInLibraryContent(sourceInfo: SourceInfo, script: Script?) = false
+  fun isInLibraryContent(sourceInfo: SourceInfo, script: Script?): Boolean = false
 
   fun computeReceiverVariable(context: VariableContext, callFrame: CallFrame, node: XCompositeNode): Promise<*>
 }
index dce7014fd982e505c77f5c456718f691298bffb2..dda5f961f5062e613544f6ee8d3c57a89775f3bb 100644 (file)
 package org.jetbrains.debugger
 
 interface MemberFilter {
-  fun isMemberVisible(variable: Variable) = variable.isReadable
+  fun isMemberVisible(variable: Variable): Boolean = variable.isReadable
 
   val additionalVariables: Collection<Variable>
     get() = emptyList()
 
-  fun rawNameToSource(variable: Variable) = variable.name
+  fun rawNameToSource(variable: Variable): String = variable.name
 
   fun sourceNameToRaw(name: String): String? = null
 
-  fun hasNameMappings() = false
+  fun hasNameMappings(): Boolean = false
 }
\ No newline at end of file
index 00fd494c0c1ee9f8f9e8d5bd441044f66fb5d2c4..9d2998ba5445471422e3580f5a89f962ead94209 100644 (file)
@@ -7,6 +7,7 @@ import com.intellij.execution.process.ProcessEvent
 import com.intellij.execution.process.ProcessHandler
 import com.intellij.xdebugger.XDebugProcess
 import org.jetbrains.rpc.LOG
+import java.io.OutputStream
 
 class ProcessHandlerWrapper(private val debugProcess: XDebugProcess, private val handler: ProcessHandler) : ProcessHandler(), KillableProcess {
   init {
@@ -25,7 +26,7 @@ class ProcessHandlerWrapper(private val debugProcess: XDebugProcess, private val
     })
   }
 
-  override fun isSilentlyDestroyOnClose() = handler.isSilentlyDestroyOnClose
+  override fun isSilentlyDestroyOnClose(): Boolean = handler.isSilentlyDestroyOnClose
 
   override fun startNotify() {
     handler.startNotify()
@@ -61,11 +62,11 @@ class ProcessHandlerWrapper(private val debugProcess: XDebugProcess, private val
       }
   }
 
-  override fun detachIsDefault() = handler.detachIsDefault()
+  override fun detachIsDefault(): Boolean = handler.detachIsDefault()
 
-  override fun getProcessInput() = handler.processInput
+  override fun getProcessInput(): OutputStream? = handler.processInput
 
-  override fun canKillProcess() = handler is KillableProcess && handler.canKillProcess()
+  override fun canKillProcess(): Boolean = handler is KillableProcess && handler.canKillProcess()
 
   override fun killProcess() {
     if (handler is KillableProcess) {
index ab54d54a186d0cb7f78d16384bcab4eaa9f183ec..4defe6b2ec3493272f55b2c2aed0eced05acf496 100644 (file)
@@ -118,7 +118,7 @@ abstract class RemoteVmConnection<VmT : Vm> : VmConnection<VmT>() {
   }
 }
 
-fun RemoteVmConnection<*>.open(address: InetSocketAddress, processHandler: ProcessHandler) = open(address, Condition<java.lang.Void> { processHandler.isProcessTerminating || processHandler.isProcessTerminated })
+fun RemoteVmConnection<*>.open(address: InetSocketAddress, processHandler: ProcessHandler): Promise<out Vm> = open(address, Condition<java.lang.Void> { processHandler.isProcessTerminating || processHandler.isProcessTerminated })
 
 fun <T> chooseDebuggee(targets: Collection<T>, selectedIndex: Int, renderer: (T, ColoredListCellRenderer<*>) -> Unit): Promise<T> {
   if (targets.size == 1) {
index af853b25b1c41f934ac5a285097ebf32f0a47d4f..762ed6aeb0bfe5ba00f291c11eb6c4fb8f33a73f 100644 (file)
@@ -14,7 +14,7 @@ class ScopeVariablesGroup(val scope: Scope, parentContext: VariableContext, call
 
   private val callFrame = if (scope.type == ScopeType.LOCAL) callFrame else null
 
-  override fun isAutoExpand() = scope.type == ScopeType.LOCAL || scope.type == ScopeType.CATCH
+  override fun isAutoExpand(): Boolean = scope.type == ScopeType.LOCAL || scope.type == ScopeType.CATCH
 
   override fun getComment(): String? {
     val className = scope.description
index e61b33c4b24ab599a42c9bba2db95d083bc77749..823ff20c5322319e6a9538d3246e19fe978c9ee9 100644 (file)
@@ -27,7 +27,7 @@ import com.intellij.xdebugger.XSourcePosition
 class SourceInfo @JvmOverloads constructor(private val file: VirtualFile, private val line: Int, val column: Int = -1, private var offset: Int = -1, val functionName: String? = null, url: Url? = null) : XSourcePosition {
   private var _url = url
 
-  override fun getFile() = file
+  override fun getFile(): VirtualFile = file
 
   val url: Url
     get() {
@@ -39,7 +39,7 @@ class SourceInfo @JvmOverloads constructor(private val file: VirtualFile, privat
       return result
     }
 
-  override fun getLine() = line
+  override fun getLine(): Int = line
 
   override fun getOffset(): Int {
     if (offset == -1) {
@@ -49,7 +49,7 @@ class SourceInfo @JvmOverloads constructor(private val file: VirtualFile, privat
     return offset
   }
 
-  override fun createNavigatable(project: Project) = OpenFileDescriptor(project, file, line, column)
+  override fun createNavigatable(project: Project): OpenFileDescriptor = OpenFileDescriptor(project, file, line, column)
 
-  override fun toString() = file.path + ":" + line + if (column == -1) "" else ":" + column
+  override fun toString(): String = file.path + ":" + line + if (column == -1) "" else ":" + column
 }
\ No newline at end of file
index 3972407c87955ee9b8aa2bcf9fea694717d03181..274320b41a9a632ce91eff0f8127512972625fe1 100644 (file)
@@ -62,7 +62,7 @@ abstract class SuspendContextView(protected val debugProcess: MultiVmDebugProces
     }
   }
 
-  override fun getActiveExecutionStack() = stacks[activeVm]
+  override fun getActiveExecutionStack(): ScriptExecutionStack? = stacks[activeVm]
 
   override fun getExecutionStacks(): Array<out XExecutionStack> = stacks.values.toTypedArray()
 
index 131f0972a6f2f471046758a020c90aa483d576ea..69d3060f6ccc783a4461faa60d14aa1067436385 100644 (file)
@@ -24,7 +24,7 @@ import java.util.concurrent.atomic.AtomicBoolean
 import java.util.regex.Pattern
 import javax.swing.Icon
 
-fun VariableView(variable: Variable, context: VariableContext) = VariableView(variable.name, variable, context)
+fun VariableView(variable: Variable, context: VariableContext): VariableView = VariableView(variable.name, variable, context)
 
 class VariableView(override val variableName: String, private val variable: Variable, private val context: VariableContext) : XNamedValue(variableName), VariableContext {
   @Volatile private var value: Value? = null
@@ -34,12 +34,12 @@ class VariableView(override val variableName: String, private val variable: Vari
   @Volatile private var remainingChildren: List<Variable>? = null
   @Volatile private var remainingChildrenOffset: Int = 0
 
-  override fun watchableAsEvaluationExpression() = context.watchableAsEvaluationExpression()
+  override fun watchableAsEvaluationExpression(): Boolean = context.watchableAsEvaluationExpression()
 
   override val viewSupport: DebuggerViewSupport
     get() = context.viewSupport
 
-  override val parent = context
+  override val parent: VariableContext = context
 
   override val memberFilter: Promise<MemberFilter>
     get() = context.viewSupport.getMemberFilter(this)
@@ -311,9 +311,9 @@ class VariableView(override val variableName: String, private val variable: Vari
     }
   }
 
-  fun getValue() = variable.value
+  fun getValue(): Value? = variable.value
 
-  override fun canNavigateToSource() = value is FunctionValue || viewSupport.canNavigateToSource(variable, context)
+  override fun canNavigateToSource(): Boolean = value is FunctionValue || viewSupport.canNavigateToSource(variable, context)
 
   override fun computeSourcePosition(navigatable: XNavigatable) {
     if (value is FunctionValue) {
@@ -361,7 +361,7 @@ class VariableView(override val variableName: String, private val variable: Vari
     }
   }
 
-  override fun computeInlineDebuggerData(callback: XInlineDebuggerDataCallback) = viewSupport.computeInlineDebuggerData(variableName, variable, context, callback)
+  override fun computeInlineDebuggerData(callback: XInlineDebuggerDataCallback): ThreeState = viewSupport.computeInlineDebuggerData(variableName, variable, context, callback)
 
   override fun getEvaluationExpression(): String? {
     if (!watchableAsEvaluationExpression()) {
index 0afcc0407db0753d785eec4c716b77bdbd3c38fa..9ad02b0d9c1a75600d26a66012f6e4e91ccd8602 100644 (file)
@@ -31,7 +31,7 @@ private val S1 = ",()[]{}="
 // don't trim trailing .&: - could be part of expression
 private val OPERATOR_TRIMMER = CharMatcher.INVISIBLE.or(CharMatcher.anyOf(S1))
 
-val NAME_TRIMMER = CharMatcher.INVISIBLE.or(CharMatcher.anyOf(S1 + ".&:"))
+val NAME_TRIMMER: CharMatcher = CharMatcher.INVISIBLE.or(CharMatcher.anyOf(S1 + ".&:"))
 
 // generateVirtualFile only for debug purposes
 open class NameMapper(private val document: Document, private val transpiledDocument: Document, private val sourceMappings: Mappings, protected val sourceMap: SourceMap, private val transpiledFile: VirtualFile? = null) {
@@ -88,7 +88,7 @@ open class NameMapper(private val document: Document, private val transpiledDocu
   protected open fun extractName(rawGeneratedName: CharSequence):String? = NAME_TRIMMER.trimFrom(rawGeneratedName)
 
   companion object {
-    fun trimName(rawGeneratedName: CharSequence, isLastToken: Boolean) = (if (isLastToken) NAME_TRIMMER else OPERATOR_TRIMMER).trimFrom(rawGeneratedName)
+    fun trimName(rawGeneratedName: CharSequence, isLastToken: Boolean): String? = (if (isLastToken) NAME_TRIMMER else OPERATOR_TRIMMER).trimFrom(rawGeneratedName)
   }
 }
 
index b349d1b1a76aa643cccc7fdac70b9517dc302c74..cc190be292b25afcbc6f628f9c1c412614573fdf 100644 (file)
@@ -50,7 +50,7 @@ open class DebuggableProgramRunner : AsyncProgramRunner<RunnerSettings>() {
     }
   }
 
-  override fun getRunnerId() = "debuggableProgram"
+  override fun getRunnerId(): String = "debuggableProgram"
 
   override fun canRun(executorId: String, profile: RunProfile): Boolean {
     return DefaultDebugExecutor.EXECUTOR_ID == executorId && profile is DebuggableRunConfiguration && profile.canRun(executorId, profile)
@@ -61,6 +61,6 @@ inline fun startSession(environment: ExecutionEnvironment, crossinline starter:
   return XDebuggerManager.getInstance(environment.project).startSession(environment, xDebugProcessStarter(starter))
 }
 
-inline fun xDebugProcessStarter(crossinline starter: (session: XDebugSession) -> XDebugProcess) = object : XDebugProcessStarter() {
+inline fun xDebugProcessStarter(crossinline starter: (session: XDebugSession) -> XDebugProcess): XDebugProcessStarter = object : XDebugProcessStarter() {
   override fun start(session: XDebugSession) = starter(session)
 }
\ No newline at end of file
index 1e6da92cd036629ceabd7e718d3319ec3c28eba5..861a3b88e2a1d86cfb7c2a19d40c2b61e1b79e75 100644 (file)
@@ -12,7 +12,7 @@ import org.jetbrains.debugger.sourcemap.SourceFileResolver
 abstract class FileUrlMapper {
   companion object {
     @JvmField
-    val EP_NAME = ExtensionPointName.create<FileUrlMapper>("com.jetbrains.fileUrlMapper")
+    val EP_NAME: ExtensionPointName<FileUrlMapper> = ExtensionPointName.create<FileUrlMapper>("com.jetbrains.fileUrlMapper")
   }
 
   open fun getUrls(file: VirtualFile, project: Project, currentAuthority: String?): List<Url> = emptyList()
index ecd26e86050061c7840d2e2d365c74f3cf7b8c9d..007cd6dc6663737ece9e21354cbd2f099dae54a2 100644 (file)
@@ -33,7 +33,7 @@ import org.jetbrains.debugger.MemberFilter
  */
 abstract class JavaScriptDebugAware {
   companion object {
-    val EP_NAME = ExtensionPointName.create<JavaScriptDebugAware>("com.jetbrains.javaScriptDebugAware")
+    val EP_NAME: ExtensionPointName<JavaScriptDebugAware> = ExtensionPointName.create<JavaScriptDebugAware>("com.jetbrains.javaScriptDebugAware")
 
     @JvmStatic
     fun isBreakpointAware(fileType: FileType): Boolean {
@@ -64,7 +64,7 @@ abstract class JavaScriptDebugAware {
   open val isOnlySourceMappedBreakpoints: Boolean
     get() = true
 
-  fun canGetEvaluationInfo(file: PsiFile) = file.fileType == fileType
+  fun canGetEvaluationInfo(file: PsiFile): Boolean = file.fileType == fileType
 
   open fun getEvaluationInfo(element: PsiElement, document: Document, expressionInfoFactory: ExpressionInfoFactory): Promise<ExpressionInfo?>? = null
 
index cf456d1f588ac81a5d1b24c2d4c47e2f5bef0886..c30d7c87213e88d5f17fbfb63c8c39a0d01f1048 100644 (file)
@@ -16,7 +16,7 @@
 package org.jetbrains.debugger
 
 open class MemberFilterWithNameMappings(private val rawNameToSource: Map<String, String> = emptyMap()) : MemberFilter {
-  override fun hasNameMappings() = !rawNameToSource.isEmpty()
+  override fun hasNameMappings(): Boolean = !rawNameToSource.isEmpty()
 
   override fun rawNameToSource(variable: Variable): String {
     val name = variable.name
@@ -24,7 +24,7 @@ open class MemberFilterWithNameMappings(private val rawNameToSource: Map<String,
     return sourceName ?: normalizeMemberName(name)
   }
 
-  protected open fun normalizeMemberName(name: String) = name
+  protected open fun normalizeMemberName(name: String): String = name
 
   override fun sourceNameToRaw(name: String): String? {
     if (!hasNameMappings()) {
index 5f9c615d76f7a6ac2c6001550d65f1b93e59c19b..d975b4d7be847383e7620bfe48b159e18c19be7a 100644 (file)
@@ -10,12 +10,12 @@ interface BoxableType {
   val writeMethodName: String
 
   companion object {
-    val STRING = StandaloneType(NamePath("String"), "writeString", "null")
-    val ANY_STRING = StandaloneType(NamePath("String"), "writeString", "null")
-    val INT = StandaloneType(NamePath("Int"), "writeInt", "-1")
-    val LONG = StandaloneType(NamePath("Long"), "writeLong", "-1")
-    val NUMBER = StandaloneType(NamePath("Double"), "writeDouble", "Double.NaN")
-    val BOOLEAN = StandaloneType(NamePath("Boolean"), "writeBoolean", "false")
-    val MAP = StandaloneType(NamePath("Map<String, String>"), "writeMap", "null")
+    val STRING: StandaloneType = StandaloneType(NamePath("String"), "writeString", "null")
+    val ANY_STRING: StandaloneType = StandaloneType(NamePath("String"), "writeString", "null")
+    val INT: StandaloneType = StandaloneType(NamePath("Int"), "writeInt", "-1")
+    val LONG: StandaloneType = StandaloneType(NamePath("Long"), "writeLong", "-1")
+    val NUMBER: StandaloneType = StandaloneType(NamePath("Double"), "writeDouble", "Double.NaN")
+    val BOOLEAN: StandaloneType = StandaloneType(NamePath("Boolean"), "writeBoolean", "false")
+    val MAP: StandaloneType = StandaloneType(NamePath("Map<String, String>"), "writeMap", "null")
   }
 }
\ No newline at end of file
index 845b9d6b904879c29f38437e7fb200a569f0e48c..64571610a0e70a9a2010894863da9e0522a6e78e 100644 (file)
@@ -34,7 +34,7 @@ abstract class ClassNameScheme(private val suffix: String, private val rootPacka
     return name
   }
 
-  fun getPackageNameVirtual(domainName: String) = getPackageName(rootPackage, domainName)
+  fun getPackageNameVirtual(domainName: String): String = getPackageName(rootPackage, domainName)
 
   class Input(suffix: String, rootPackage: String) : ClassNameScheme(suffix, rootPackage) {
     fun getParseMethodName(domain: String, name: String): String {
index 372607fdc2f70b80f5372fe37f1c5d4cf40404a1..74c89b5f52a53b6221abfd703635b685820e7209 100644 (file)
@@ -25,4 +25,4 @@ internal abstract class ClassScope(val generator: DomainGenerator, val classCont
   abstract val typeDirection: TypeData.Direction
 }
 
-fun ItemDescriptor.Named.getName() = shortName ?: name()
\ No newline at end of file
+fun ItemDescriptor.Named.getName(): String = shortName ?: name()
\ No newline at end of file
index 279a954af042e717b59f602bc935bafa770f8bbf..2f6564d2e37c69e29ece42239ec0afb4daf3bd4b 100644 (file)
@@ -284,4 +284,4 @@ internal class DomainGenerator(val generator: Generator, val domain: ProtocolMet
   }
 }
 
-fun subMessageType(namePath: NamePath) = StandaloneType(namePath, "writeMessage", null)
\ No newline at end of file
+fun subMessageType(namePath: NamePath): StandaloneType = StandaloneType(namePath, "writeMessage", null)
\ No newline at end of file
index fbb3b537d7c14b7bf55aecb2683a0b639cdcd21f..f93a9acaa6847ab60d5db29f38f1bb86439ab4d1 100644 (file)
@@ -228,7 +228,7 @@ internal class Generator(outputDir: String, private val rootPackage: String, req
   }
 }
 
-val READER_INTERFACE_NAME = "ProtocolResponseReader"
+val READER_INTERFACE_NAME: String = "ProtocolResponseReader"
 
 private val INCLUDED_DOMAINS = arrayOf("CSS", "Debugger", "DOM", "Inspector", "Log", "Network", "Page", "Runtime", "ServiceWorker",
                                        "Tracing", "Target", "Overlay", "Console", "DOMDebugger", "Profiler", "HeapProfiler")
index b7e2b1ccf4a8b661b1d01a1905b2d652f1db8ef2..24c3d3d6a70e74a40b50036ce247651af0adc068 100644 (file)
@@ -1,7 +1,7 @@
 package org.jetbrains.protocolModelGenerator
 
 open class ListType(private val itemType: BoxableType) : BoxableType {
-  override val defaultValue = null
+  override val defaultValue: Nothing? = null
 
   override val writeMethodName: String
     get() = when {
index 8cc00c1a490131dbe43fe28aecd956eabac19a96..76964aab04c0d7f8e2b264abb5af3f8198ac42f8 100644 (file)
@@ -1,7 +1,7 @@
 package org.jetbrains.protocolModelGenerator
 
 class StandaloneType(private val namePath: NamePath, override val writeMethodName: String, override val defaultValue: String? = "null") : BoxableType {
-  override val fullText = namePath.getFullText()
+  override val fullText: String = namePath.getFullText()
 
   override fun getShortText(contextNamespace: NamePath): String {
     val nameLength = namePath.getLength()
index a766440b6e79bdeb9fcad0739360524c4df08e19..1f288f04ab85dae69e98a798ff6f255d8cfd2658 100644 (file)
@@ -28,7 +28,7 @@ import org.jetbrains.io.JsonUtil
 
 open class OutMessage() {
   val buffer: ByteBuf = ByteBufAllocator.DEFAULT.heapBuffer()
-  val writer = JsonWriter(ByteBufUtf8Writer(buffer))
+  val writer: JsonWriter = JsonWriter(ByteBufUtf8Writer(buffer))
 
   private var finalized: Boolean = false
 
index 370c78f364a239735a1d5534d0bf157466c0f369..d088a16a0c7be239dfd5bee647f35408f9907949 100644 (file)
@@ -11,8 +11,8 @@ import java.util.*
  * calls {@link #update()}.
  */
 class FileUpdater(private val file: Path) {
-  val builder = StringBuilder()
-  val out = TextOutput(builder)
+  val builder: StringBuilder = StringBuilder()
+  val out: TextOutput = TextOutput(builder)
 
   fun update() {
     if (builder.length == 0) {
index b6bb0de23df734ef7f2150e26171f897bb6a4517..5007fd49d6cb4dc98551a584c3eb134fc8be6771 100644 (file)
@@ -3,6 +3,6 @@ package org.jetbrains.protocolReader
 /**
  * Signals that JSON model has some problem in it.
  */
-fun JsonProtocolModelParseException(message: String) = JsonProtocolModelParseException(message, null)
+fun JsonProtocolModelParseException(message: String): JsonProtocolModelParseException = JsonProtocolModelParseException(message, null)
 
 class JsonProtocolModelParseException(message: String, cause: Throwable?) : RuntimeException(message, cause)
index c6e33edee2a3b8434131e4276ecc8273c5bee4d9..7397d6fa61a0d2f17eafd8b93696e71bcae6355c 100644 (file)
@@ -5,7 +5,7 @@ import java.nio.file.Paths
 import java.util.*
 
 class GenerateConfiguration<ROOT>(val packageName: String, val className: String, readerRootClass: Class<ROOT>, protocolInterfaces: List<Class<*>>, basePackagesMap: Map<Class<*>, String>? = null) {
-  val basePackagesMap = if (basePackagesMap == null) listOf<Map<Class<*>, String>>() else listOf(basePackagesMap)
+  val basePackagesMap: List<Map<Class<*>, String>> = if (basePackagesMap == null) listOf<Map<Class<*>, String>>() else listOf(basePackagesMap)
 
   internal val typeToTypeHandler = InterfaceReader(protocolInterfaces).go()
   internal val root = ReaderRoot(readerRootClass, typeToTypeHandler)
index 23b1ee8b63dbb5e8def552cb62e9f03d8e692274..fc7e8277e34709ffcd70c8a23df261f611b67544 100644 (file)
@@ -101,9 +101,9 @@ class TextOutput(val out: StringBuilder) {
     return this
   }
 
-  fun comma() = append(',').space()
+  fun comma(): TextOutput = append(',').space()
 
-  fun space() = append(' ')
+  fun space(): TextOutput = append(' ')
 
   fun doc(description: String?): TextOutput {
     if (description == null) {
@@ -112,7 +112,7 @@ class TextOutput(val out: StringBuilder) {
     return append("/**").newLine().append(" * ").append(description).newLine().append(" */").newLine()
   }
 
-  fun quote(s: CharSequence) = append('"').append(s).append('"')
+  fun quote(s: CharSequence): TextOutput = append('"').append(s).append('"')
 
   fun maybeIndent() {
     if (justNewLined) {
@@ -133,5 +133,5 @@ class TextOutput(val out: StringBuilder) {
     return this
   }
 
-  operator fun plus(value: String) = append(value)
+  operator fun plus(value: String): TextOutput = append(value)
 }
index 2f46fb112afcb282a2025a3a6b5e08ecb5881a9d..994ae5e168570bf89c0ea614b16c5e151738b846 100644 (file)
@@ -4,13 +4,13 @@ import java.lang.reflect.ParameterizedType
 import java.lang.reflect.Type
 import java.lang.reflect.WildcardType
 
-val TYPE_FACTORY_NAME_PREFIX = 'F'
+val TYPE_FACTORY_NAME_PREFIX: Char = 'F'
 
-val READER_NAME = "reader"
-val PENDING_INPUT_READER_NAME = "inputReader"
+val READER_NAME: String = "reader"
+val PENDING_INPUT_READER_NAME: String = "inputReader"
 
-val JSON_READER_CLASS_NAME = "JsonReaderEx"
-val JSON_READER_PARAMETER_DEF = "$READER_NAME: $JSON_READER_CLASS_NAME"
+val JSON_READER_CLASS_NAME: String = "JsonReaderEx"
+val JSON_READER_PARAMETER_DEF: String = "$READER_NAME: $JSON_READER_CLASS_NAME"
 
 /**
  * Generate Java type name of the passed type. Type may be parameterized.
index b2eab3e3511f5ed800d1ea5dc0c2344b801a5ff3..9195888b4451a21f3ef4d1451ea8b46ce250e46c 100644 (file)
@@ -15,11 +15,11 @@ import javax.swing.Icon
 class MockRunManager : RunManagerEx() {
   override fun findSettings(configuration: RunConfiguration): RunnerAndConfigurationSettings? = null
 
-  override fun getConfigurationType(typeName: String) = TODO("not implemented")
+  override fun getConfigurationType(typeName: String): Nothing = TODO("not implemented")
 
-  override fun hasSettings(settings: RunnerAndConfigurationSettings) = false
+  override fun hasSettings(settings: RunnerAndConfigurationSettings): Boolean = false
 
-  override fun getConfigurationsList(type: ConfigurationType) = emptyList<RunConfiguration>()
+  override fun getConfigurationsList(type: ConfigurationType): List<RunConfiguration> = emptyList<RunConfiguration>()
 
   override fun makeStable(settings: RunnerAndConfigurationSettings) {}
 
index e998d06359f54b54de98f2d1eddd92139b1a2089..68904b023ebd784f81f6236ebd9d47e76020edb9 100644 (file)
@@ -96,13 +96,13 @@ class IntelliJProjectConfiguration {
   }
 
   class LibraryRoots(val classes: List<File>, val sources: List<File>) {
-    val classesPaths
+    val classesPaths: List<String>
       get() = classes.map { FileUtil.toSystemIndependentName(it.absolutePath) }
 
-    val classesUrls
+    val classesUrls: List<String>
       get() = classes.map { JpsPathUtil.getLibraryRootUrl(it) }
 
-    val sourcesUrls
+    val sourcesUrls: List<String>
       get() = sources.map { JpsPathUtil.getLibraryRootUrl(it) }
   }
 }
index 26ea93d276b5a8dc29fb5035868d7f6771133d52..556adcd939db2d65165759cd664c1af89b5b26bf 100644 (file)
@@ -174,10 +174,10 @@ class EdtRule : TestRule {
 }
 
 class InitInspectionRule : TestRule {
-  override fun apply(base: Statement, description: Description) = statement { runInInitMode { base.evaluate() } }
+  override fun apply(base: Statement, description: Description): Statement = statement { runInInitMode { base.evaluate() } }
 }
 
-inline fun statement(crossinline runnable: () -> Unit) = object : Statement() {
+inline fun statement(crossinline runnable: () -> Unit): Statement = object : Statement() {
   override fun evaluate() {
     runnable()
   }
@@ -221,7 +221,7 @@ inline fun <T> Project.runInLoadComponentStateMode(task: () -> T): T {
   }
 }
 
-fun createHeavyProject(path: String, useDefaultProjectSettings: Boolean = false) = ProjectManagerEx.getInstanceEx().newProject(null, path, useDefaultProjectSettings, false)!!
+fun createHeavyProject(path: String, useDefaultProjectSettings: Boolean = false): Project = ProjectManagerEx.getInstanceEx().newProject(null, path, useDefaultProjectSettings, false)!!
 
 fun Project.use(task: (Project) -> Unit) {
   val projectManager = ProjectManagerEx.getInstanceEx() as ProjectManagerImpl
@@ -265,7 +265,7 @@ class DisposeModulesRule(private val projectRule: ProjectRule) : ExternalResourc
  * So, should be one task per rule.
  */
 class WrapRule(private val before: () -> () -> Unit) : TestRule {
-  override fun apply(base: Statement, description: Description) = statement {
+  override fun apply(base: Statement, description: Description): Statement = statement {
     val after = before()
     try {
       base.evaluate()
index 2a333db1b8dab1757697fd3d5dff6730db6c43d0..66a1d11740ba1a50372df0323c612a463ffd253d 100644 (file)
@@ -98,4 +98,4 @@ fun generateTemporaryPath(fileName: String?): Path {
   return path
 }
 
-fun VirtualFile.writeChild(relativePath: String, data: String) = VfsTestUtil.createFile(this, relativePath, data)
\ No newline at end of file
+fun VirtualFile.writeChild(relativePath: String, data: String): VirtualFile = VfsTestUtil.createFile(this, relativePath, data)
\ No newline at end of file
index fb8377fa6cbd7e7a70e0214d005ab6b3685d6068..0e50d1e2598accff22fbabf1b0ab3864bb06ba28 100644 (file)
@@ -23,4 +23,4 @@ fun FileSystem.file(path: String, data: ByteArray): FileSystem {
   return this
 }
 
-fun FileSystem.file(path: String, data: String) = file(path, data.toByteArray())
\ No newline at end of file
+fun FileSystem.file(path: String, data: String): FileSystem = file(path, data.toByteArray())
\ No newline at end of file
index d2ba43eda567d5495fe8daf8cd2fb9cd604227b7..fc8ba84624bf77ad295741b0b226d001e824dd0f 100644 (file)
@@ -79,7 +79,7 @@ fun ProjectInspectionProfileManager.createProfile(localInspectionTool: LocalInsp
   return configureInspections(arrayOf(localInspectionTool), project, disposable)
 }
 
-fun enableInspectionTool(project: Project, tool: InspectionProfileEntry, disposable: Disposable) = enableInspectionTool(project, InspectionToolRegistrar.wrapTool(tool), disposable)
+fun enableInspectionTool(project: Project, tool: InspectionProfileEntry, disposable: Disposable): Unit = enableInspectionTool(project, InspectionToolRegistrar.wrapTool(tool), disposable)
 
 fun enableInspectionTools(project: Project, disposable: Disposable, vararg tools: InspectionProfileEntry) {
   for (tool in tools) {
index 00b60d198ac2c44f18ac82780892153fe7a275bb..88b21e024ad2016853c1743839aff71f940600a9 100644 (file)
@@ -34,7 +34,7 @@ sealed class DirectoryContentSpecImpl : DirectoryContentSpec {
 }
 
 abstract class DirectorySpecBase : DirectoryContentSpecImpl() {
-  protected val children = LinkedHashMap<String, DirectoryContentSpecImpl>()
+  protected val children: LinkedHashMap<String, DirectoryContentSpecImpl> = LinkedHashMap<String, DirectoryContentSpecImpl>()
 
   fun addChild(name: String, spec: DirectoryContentSpecImpl) {
     if (name in children) {