From 1e3c01dd07a7322b9483be134dc1b57bc9fb6475 Mon Sep 17 00:00:00 2001 From: "Paulo \"JCranky\" Siqueira" Date: Thu, 22 Dec 2016 12:26:04 +0100 Subject: [PATCH] Remove null usage from scalamd/Main (#32) --- README.md | 4 +- .../src/org/fusesource/scalamd/Main.scala | 46 +++++++++---------- 2 files changed, 24 insertions(+), 26 deletions(-) diff --git a/README.md b/README.md index 89dbf29..92c915b 100644 --- a/README.md +++ b/README.md @@ -25,8 +25,8 @@ Scalatron is licensed under the Creative Commons Attribution 3.0 Unported Licens ## Build a new docker image - sbt docker +```sbt docker``` ## Run inside a docker container - docker run -d -p 8080:8080 -v /tmp/bots:/opt/Scalatron/bots scalatron/scalatron:latest +```docker run -d -p 8080:8080 -v /tmp/bots:/opt/Scalatron/bots --name scalatron scalatron/scalatron:latest``` diff --git a/ScalaMarkdown/src/org/fusesource/scalamd/Main.scala b/ScalaMarkdown/src/org/fusesource/scalamd/Main.scala index e5c0a81..556efcb 100644 --- a/ScalaMarkdown/src/org/fusesource/scalamd/Main.scala +++ b/ScalaMarkdown/src/org/fusesource/scalamd/Main.scala @@ -113,30 +113,28 @@ object Main { outputBaseDirectoryPath: String, verbose: Boolean): Unit = { val inputDirectory = new File(inputDirectoryPath) - if (inputDirectory.getName.startsWith("_")) { - // skip - } else { - val fileList = inputDirectory.listFiles() - if (fileList == null || fileList.isEmpty) { - System.err.println( - "warning: no files to process in: " + inputDirectoryPath) - } else { - val (directoryCollection, fileCollection) = - fileList.partition(_.isDirectory) - directoryCollection.foreach( - dir => - processDirectory(dir.getAbsolutePath, - outputDirectoryPath + "/" + dir.getName, - inputBaseDirectoryPath, - outputBaseDirectoryPath, - verbose)) - fileCollection.foreach( - file => - processFile(file.getAbsolutePath, - outputDirectoryPath, - inputBaseDirectoryPath, - outputBaseDirectoryPath, - verbose)) + + if (!inputDirectory.getName.startsWith("_")) { + val filesOpt = Option(inputDirectory.listFiles()).filter(_.nonEmpty) + if (filesOpt.isEmpty) + System.err.println("warning: no files to process in: " + inputDirectoryPath) + + filesOpt.foreach { fileList => + val (directoryCollection, fileCollection) = fileList.partition(_.isDirectory) + directoryCollection.foreach { dir => + processDirectory(dir.getAbsolutePath, + outputDirectoryPath + "/" + dir.getName, + inputBaseDirectoryPath, + outputBaseDirectoryPath, + verbose) + } + fileCollection.foreach { file => + processFile(file.getAbsolutePath, + outputDirectoryPath, + inputBaseDirectoryPath, + outputBaseDirectoryPath, + verbose) + } } } }