Pull request comments. Formatting and simplification.

Author: themkat

shared/src/main/kotlin/org/javacs/kt/classpath/MavenClassPathResolver.kt

@@ -130,10 +130,10 @@ private val mvnCommandFromPath: Path? by lazy {
     findCommandOnPath("mvn")
 }
 
-private fun mvnCommand(pom : Path) : Path {
-	return requireNotNull(mvnCommandFromPath ?: findProjectCommandWithName("mvnw", pom)?.also {
-		LOG.info("Using mvn wrapper (mvnw) in place of mvn command")
-	}) { "Unable to find the 'mvn' command or suitable wrapper" }
+private fun mvnCommand(pom: Path): Path {
+    return requireNotNull(mvnCommandFromPath ?: findProjectCommandWithName("mvnw", pom)?.also {
+        LOG.info("Using mvn wrapper (mvnw) in place of mvn command")
+    }) { "Unable to find the 'mvn' command or suitable wrapper" }
 }
 
 fun parseMavenArtifact(rawArtifact: String, version: String? = null): Artifact {

shared/src/main/kotlin/org/javacs/kt/util/ShellPathUtils.kt

@@ -34,18 +34,16 @@ private fun findExecutableOnPath(fileName: String): Path? {
     return null
 }
 
-fun findProjectCommandWithName(name : String, projectFile : Path) : Path? {
-	if(isOSWindows()) {
-		return findFileRelativeToProjectFile("$name.cmd", projectFile)
-	} else {
-		return findFileRelativeToProjectFile(name, projectFile)
-	}
-}
+fun findProjectCommandWithName(name: String, projectFile: Path): Path? =
+    if (isOSWindows()) {
+        findFileRelativeToProjectFile("$name.cmd", projectFile)
+    } else {
+        findFileRelativeToProjectFile(name, projectFile)
+    }
 
-private fun findFileRelativeToProjectFile(name : String, projectFile : Path) : Path? {
-	return projectFile.resolveSibling(name).toFile().takeIf { file ->
-		file.isFile && file.canExecute()
-	}?.let { file ->
-		Paths.get(file.absolutePath)
-	}
-}
+private fun findFileRelativeToProjectFile(name: String, projectFile: Path): Path? =
+    projectFile.resolveSibling(name).toFile().takeIf { file ->
+        file.isFile && file.canExecute()
+    }?.let { file ->
+        Paths.get(file.absolutePath)
+    }