Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

In src/core/evaluator.js, convert a couple of if (!someVariable) { error(...); } instances to assert(someVariable); instead

Browse Source 
Rather than, in a number of places, basically duplicating the logic of `assert` we can simply utilize the function directly instead.
This commit is contained in:
Jonas Jenwald 2017-03-26 13:53:13 +02:00
parent b7ba44b530
commit e40fd63bd3
1 changed files with 4 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/core/evaluator.js
View File

@ -1071,14 +1071,10 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
case OPS.shadingFill:
var shadingRes = resources.get('Shading');
if (!shadingRes) {
error('No shading resource found');
}
assert(shadingRes, 'No shading resource found');
var shading = shadingRes.get(args[0].name);
if (!shading) {
error('No shading object found');
}
assert(shading, 'No shading object found');
var shadingFill = Pattern.parseShading(shading, null, xref,
resources, self.handler);
@ -2166,9 +2162,7 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
// - set the type according to the descendant font
// - get the FontDescriptor from the descendant font
var df = dict.get('DescendantFonts');
if (!df) {
error('Descendant fonts are not specified');
}
assert(df, 'Descendant fonts are not specified');
dict = (isArray(df) ? xref.fetchIfRef(df[0]) : df);
type = dict.get('Subtype');
@ -2261,9 +2255,7 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
// FontDescriptor was not required.
// This case is here for compatibility.
var baseFontName = dict.get('BaseFont');
if (!isName(baseFontName)) {
error('Base font is not specified');
}
assert(isName(baseFontName), 'Base font is not specified');
// Using base font name as a font name.
baseFontName = baseFontName.name.replace(/[,_]/g, '-');