You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by ro...@apache.org on 2020/06/08 13:16:10 UTC

[james-project] 16/30: JAMES-3179 Deal with nullity in OsDetector

This is an automated email from the ASF dual-hosted git repository.

rouazana pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git

commit e7a191e9dfd4cd985d7cbe9400d719687e4c11b4
Author: Gautier DI FOLCO <gd...@linagora.com>
AuthorDate: Tue May 12 15:47:17 2020 +0200

    JAMES-3179 Deal with nullity in OsDetector
---
 .../src/test/java/org/apache/james/mailbox/maildir/OsDetector.java | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/OsDetector.java b/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/OsDetector.java
index 75fe97b..7e0fcd1 100644
--- a/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/OsDetector.java
+++ b/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/OsDetector.java
@@ -20,6 +20,7 @@
 package org.apache.james.mailbox.maildir;
 
 import java.util.Locale;
+import java.util.Optional;
 
 public class OsDetector {
 
@@ -29,8 +30,10 @@ public class OsDetector {
      * @return windows
      */
     public static boolean isWindows() {
-        String os = System.getProperty("os.name").toLowerCase(Locale.US);
-        return (os.contains("win"));
+        return Optional.ofNullable(System.getProperty("os.name"))
+            .orElse("")
+            .toLowerCase(Locale.US)
+            .contains("win");
     }
     
     


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org