Skip to content

Commit 9b70284

Browse files
committed
fix problem where trailing blanks confused toNumber()
1 parent 3b95a6e commit 9b70284

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/functions.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2549,7 +2549,7 @@ export default function functions(
25492549
debug.push(`Failed to convert "${num}" base "${base}" to number`);
25502550
return null;
25512551
}
2552-
const parts = num.split('.');
2552+
const parts = num.split('.').map(p => p.trim());
25532553

25542554
let decimal = 0;
25552555
if (parts.length > 1) {

test/functions.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1157,6 +1157,8 @@
11571157
{ "expression": "toNumber(\" 011\\n\", 8)", "result": 9},
11581158
{ "expression": "toNumber(\" -011\\n\", 8)", "result": -9},
11591159
{ "expression": "toNumber(\" +011\\n\", 8)", "result": 9},
1160+
{ "expression": "toNumber(\" 070.070 \", 8)", "result": 56.109375},
1161+
{ "expression": "toNumber(\" 0F0.0F0 \", 16)", "result": 240.05859375},
11601162
{
11611163
"expression": "'toString'(`1.0`)",
11621164
"error": "SyntaxError"

0 commit comments

Comments
 (0)