diff options
| author | wmin0 <wmin0@cobinhood.com> | 2019-02-18 18:52:06 +0800 |
|---|---|---|
| committer | Jhih-Ming Huang <jm.huang@cobinhood.com> | 2019-04-11 10:39:58 +0800 |
| commit | 5aa18b7dadcd23802ced5ed27636841fb429d53e (patch) | |
| tree | dbe4fb5df1d59bfbe9167009e01db7c0c7cbbd12 | |
| parent | 85111874b93c3a1038c783d9b5964690a88d7cf8 (diff) | |
| download | dexon-5aa18b7dadcd23802ced5ed27636841fb429d53e.tar dexon-5aa18b7dadcd23802ced5ed27636841fb429d53e.tar.gz dexon-5aa18b7dadcd23802ced5ed27636841fb429d53e.tar.bz2 dexon-5aa18b7dadcd23802ced5ed27636841fb429d53e.tar.lz dexon-5aa18b7dadcd23802ced5ed27636841fb429d53e.tar.xz dexon-5aa18b7dadcd23802ced5ed27636841fb429d53e.tar.zst dexon-5aa18b7dadcd23802ced5ed27636841fb429d53e.zip | |
core: vm: sqlvm: parser: toDecimal handle .0
To cover the pitfall of decimal.fromString, we need to add leading 0 at
'^\.[0-9]+' case.
| -rw-r--r-- | core/vm/sqlvm/parser/parser.go | 3 | ||||
| -rw-r--r-- | core/vm/sqlvm/parser/parser_test.go | 1 |
2 files changed, 4 insertions, 0 deletions
diff --git a/core/vm/sqlvm/parser/parser.go b/core/vm/sqlvm/parser/parser.go index 76fa15cdc..2e882ea3c 100644 --- a/core/vm/sqlvm/parser/parser.go +++ b/core/vm/sqlvm/parser/parser.go @@ -95,6 +95,9 @@ func toUint(b []byte) (uint32, errors.ErrorCode) { } func toDecimal(b []byte) (decimal.Decimal, errors.ErrorCode) { + if len(b) > 0 && b[0] == byte('.') { + b = append([]byte{'0'}, b...) + } d, err := decimal.NewFromString(string(b)) return d, convertDecimalError(err) } diff --git a/core/vm/sqlvm/parser/parser_test.go b/core/vm/sqlvm/parser/parser_test.go index 77c3c16ff..bed734c60 100644 --- a/core/vm/sqlvm/parser/parser_test.go +++ b/core/vm/sqlvm/parser/parser_test.go @@ -22,6 +22,7 @@ func (s *ParserTestSuite) TestParse() { // Test expr. s.requireParseNoError(`select 1 + 2 * 3`) + s.requireParseNoError(`select .0`) s.requireParseNoError(`select a(1 + 1)`) s.requireParseNoError(`select hEx'12'`) s.requireParseNoError(`select x'12'`) |