Empty string shows error: unsupported number: -Infinity

[tobwithu]

Thank you for this great tool.
However, I found a bug.
Empty string shows error: unsupported number: -Infinity

Code

<Document>
        <Page>
            <View style={{alignItems: 'center'}}>
                <Text style={{textAlign: "center"}}>{""}</Text>
            </View>
        </Page>
</Document>

Result

Error: unsupported number: -Infinity
    at Function.number (pdfkit.browser.es.js:273:1)
    at pdfkit.browser.es.js:1866:1
    at Array.map (<anonymous>)
    at PDFDocument.transform (pdfkit.browser.es.js:1865:1)
    at PDFDocument.translate (pdfkit.browser.es.js:1871:1)
    at renderText (renderText.js:233:1)
    at _curryN.js:34:1
    at _arity.js:10:1
    at cond.js:42:1
    at _arity.js:10:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _arity.js:10:1
    at renderNode.js:80:1
    at Array.forEach (<anonymous>)
    at renderNode.js:72:1
    at when (when.js:32:1)
    at _curry3.js:28:1
    at f1 (_curry1.js:16:1)
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _arity.js:10:1
    at renderNode.js:80:1
    at Array.forEach (<anonymous>)
    at renderNode.js:72:1
    at when (when.js:32:1)
    at _curry3.js:28:1
    at f1 (_curry1.js:16:1)
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _pipe.js:3:1
    at _arity.js:10:1
    at renderNode.js:80:1
    at Array.forEach (<anonymous>)
    at index.js:15:1
    at render (index.js:21:1)
    at _callee$ (index.js:35:1)
    at tryCatch (regeneratorRuntime.js:86:1)
    at Generator._invoke (regeneratorRuntime.js:66:1)
    at Generator.next (regeneratorRuntime.js:117:1)
    at asyncGeneratorStep (asyncToGenerator.js:3:1)
    at _next (asyncToGenerator.js:25:1)

Desktop (please complete the following information):

• OS: [MacOS]
• Browser [chrome]
• React-pdf version [e.g. v2.1.2]

[modosc]

i think the issue is here:

react-pdf/packages/layout/src/text/linesWidth.js

Lines 10 to 11 in 7c1d373

	if (!node.lines) return -1;
	return Math.max(...node.lines.map(line => advanceWidth(line)));

i think if (!node.lines) should be if (!node.lines || node.lines.length === 0) because otherwise when node.lines = [] Math.max() is called without any args and returns -Infinity

[diegomura]

Thanks! Fixed. Weird, I'm also able to replicate it in older versions. Seems it's been here for awhile? Maybe I tested it wrong