mirrors/pdf.js
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2025-04-24 09:08:07 +02:00
Code Activity

Use String.prototype.repeat() in a couple of spots

Browse source 
Rather than using a temporary Array to manually create repeated strings, we can use `String.prototype.repeat()` instead.
The reason that we didn't use this from the start is most likely because some browsers, notably IE, didn't support this; note https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/repeat#browser_compatibility
This commit is contained in:
Jonas Jenwald 2022-03-30 15:38:07 +02:00
parent d6592b5e37
commit addb4cb12b
3 changed files with 5 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

5
test/unit/evaluator_spec.js
View file

@ -261,10 +261,9 @@ describe("evaluator", function () {
"(bug 1443140)",
async function () {
const NUM_INVALID_OPS = 25;
const tempArr = new Array(NUM_INVALID_OPS + 1);
// Non-path operators, should be ignored.
const invalidMoveText = tempArr.join("10 Td\n");
const invalidMoveText = "10 Td\n".repeat(NUM_INVALID_OPS);
const moveTextStream = new StringStream(invalidMoveText);
const result = await runOperatorListCheck(
partialEvaluator,
@ -275,7 +274,7 @@ describe("evaluator", function () {
expect(result.fnArray).toEqual([]);
// Path operators, should throw error.
const invalidLineTo = tempArr.join("20 l\n");
const invalidLineTo = "20 l\n".repeat(NUM_INVALID_OPS);
const lineToStream = new StringStream(invalidLineTo);
try {

5
test/unit/util_spec.js
View file

@ -49,9 +49,8 @@ describe("util", function () {
bytes[i] = "a".charCodeAt(0);
}
// Create a string with `length` 'a' characters. We need an array of size
// `length + 1` since `join` puts the argument between the array elements.
const string = Array(length + 1).join("a");
// Create a string with `length` 'a' characters.
const string = "a".repeat(length);
expect(bytesToString(bytes)).toEqual(string);
});