lib/core: `is_numeric` returns false on empty strings
authorAlexis Laferrière <alexis.laf@xymus.net>
Tue, 29 Dec 2015 18:51:27 +0000 (13:51 -0500)
committerAlexis Laferrière <alexis.laf@xymus.net>
Tue, 12 Jan 2016 04:17:42 +0000 (23:17 -0500)
Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/core/text/abstract_text.nit

index e201e0c..401f32c 100644 (file)
@@ -305,7 +305,7 @@ abstract class Text
                end
        end
 
-       # Returns `true` if the string contains only Numeric values (and one "," or one "." character)
+       # Is this string in a valid numeric format compatible with `to_f`?
        #
        #     assert "123".is_numeric  == true
        #     assert "1.2".is_numeric  == true
@@ -313,6 +313,7 @@ abstract class Text
        #     assert "1,2".is_numeric  == true
        #     assert "-1.23e-2".is_numeric == true
        #     assert "1..2".is_numeric == false
+       #     assert "".is_numeric     == false
        fun is_numeric: Bool
        do
                var has_point_or_comma = false
@@ -330,7 +331,7 @@ abstract class Text
                                end
                        end
                end
-               return true
+               return not is_empty
        end
 
        # Returns `true` if the string contains only Hex chars