Merge: Text::is_int bugfix
authorJean Privat <jean@pryen.org>
Sat, 22 Aug 2015 02:59:08 +0000 (22:59 -0400)
committerJean Privat <jean@pryen.org>
Sat, 22 Aug 2015 02:59:08 +0000 (22:59 -0400)
Fix a bug that caused an assert failed when accessing an empty string in `Text::is_int`

Reported by @xymus with PR #1649

Pull-Request: #1650
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Jean Privat <jean@pryen.org>

lib/standard/fixed_ints.nit

index 6dc6550..daad211 100644 (file)
@@ -883,7 +883,9 @@ redef class Text
        #     assert "0b1011".is_int
        #     assert not "0x_".is_int
        #     assert not "0xGE".is_int
+       #     assert not "".is_int
        fun is_int: Bool do
+               if bytelen == 0 then return false
                var s = remove_all('_')
                var pos = 0
                while s[pos] == '-' do