Marmelator/pdf.js.mirror
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2026-08-02 04:17:24 +02:00
Code Issues Packages Projects Releases Wiki Activity

Remove Array.prototype.reduce usage from the src/core/xfa/template.js file

Browse Source 
Using `Array.prototype.reduce` often leads to less readable code, and in these cases we can replace that with other Array-methods instead.
This commit is contained in:
Jonas Jenwald 2025-03-20 08:18:15 +01:00
parent fceaab8864
commit 9456b88f6b
1 changed files with 10 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/core/xfa/template.js
View File

@ -5721,12 +5721,7 @@ class Text extends ContentObject {
if (typeof this[$content] === "string") { if (typeof this[$content] === "string") {
return this[$content] return this[$content]
.split(/[\u2029\u2028\n]/) .split(/[\u2029\u2028\n]/)
.reduce((acc, line) => { .filter(line => !!line)
if (line) {
acc.push(line);
}
return acc;
}, [])
.join("\n"); .join("\n");
} }
return this[$content][$text](); return this[$content][$text]();
@ -5748,18 +5743,15 @@ class Text extends ContentObject {
.map(para => .map(para =>
// Convert a paragraph into a set of <span> (for lines) // Convert a paragraph into a set of <span> (for lines)
// separated by <br>. // separated by <br>.
para.split(/[\u2028\n]/).reduce((acc, line) => { para.split(/[\u2028\n]/).flatMap(line => [
acc.push( {
{ name: "span",
name: "span", value: line,
value: line, },
}, {
{ name: "br",
name: "br", },
} ])
);
return acc;
}, [])
) )
.forEach(lines => { .forEach(lines => {
html.children.push({ html.children.push({