Merge pull request #10789 from Snuffleupagus/handleColorN-error

Change `PartialEvaluator.handleColorN` to throw when no valid pattern is found
This commit is contained in:
Tim van der Meij 2019-05-04 15:02:10 +02:00 committed by GitHub
commit 6cfb1e1a63
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -853,9 +853,7 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
} }
}, },
handleColorN: function PartialEvaluator_handleColorN(operatorList, fn, args, async handleColorN(operatorList, fn, args, cs, patterns, resources, task) {
cs, patterns,
resources, task) {
// compile tiling patterns // compile tiling patterns
var patternName = args[args.length - 1]; var patternName = args[args.length - 1];
// SCN/scn applies patterns along with normal colors // SCN/scn applies patterns along with normal colors
@ -875,13 +873,11 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
pattern = Pattern.parseShading(shading, matrix, this.xref, resources, pattern = Pattern.parseShading(shading, matrix, this.xref, resources,
this.handler, this.pdfFunctionFactory); this.handler, this.pdfFunctionFactory);
operatorList.addOp(fn, pattern.getIR()); operatorList.addOp(fn, pattern.getIR());
return Promise.resolve(); return undefined;
} }
return Promise.reject(new Error('Unknown PatternType: ' + typeNum)); throw new FormatError(`Unknown PatternType: ${typeNum}`);
} }
// TODO shall we fail here? throw new FormatError(`Unknown PatternName: ${patternName}`);
operatorList.addOp(fn, args);
return Promise.resolve();
}, },
getOperatorList({ stream, task, resources, operatorList, getOperatorList({ stream, task, resources, operatorList,