You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2021/11/07 12:52:13 UTC

[commons-lang] branch master updated: Add TODO comment.

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

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-lang.git


The following commit(s) were added to refs/heads/master by this push:
     new 858d536  Add TODO comment.
     new f892c90  Merge branch 'master' of https://gitbox.apache.org/repos/asf/commons-lang.git
858d536 is described below

commit 858d53687312aae22c050cad24274a7d08227319
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Nov 7 07:51:35 2021 -0500

    Add TODO comment.
---
 src/main/java/org/apache/commons/lang3/LocaleUtils.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/main/java/org/apache/commons/lang3/LocaleUtils.java b/src/main/java/org/apache/commons/lang3/LocaleUtils.java
index b57e3c0..dbade05 100644
--- a/src/main/java/org/apache/commons/lang3/LocaleUtils.java
+++ b/src/main/java/org/apache/commons/lang3/LocaleUtils.java
@@ -311,6 +311,7 @@ public class LocaleUtils {
      */
     public static Locale toLocale(final String str) {
         if (str == null) {
+            // TODO Should this return the default locale?
             return null;
         }
         if (str.isEmpty()) { // LANG-941 - JDK 8 introduced an empty locale where all fields are blank