From 9cb09324d2af13bc2f5a441ea87e3a02c18507b8 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 1 Aug 2014 18:23:19 +0200 Subject: [PATCH 1/6] Add strict equalities in src/core/annotation.js --- src/core/annotation.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/core/annotation.js b/src/core/annotation.js index c0219529f..3bf8a8249 100644 --- a/src/core/annotation.js +++ b/src/core/annotation.js @@ -357,7 +357,7 @@ var WidgetAnnotation = (function WidgetAnnotationClosure() { var j, jj; for (j = 0, jj = kids.length; j < jj; j++) { var kidRef = kids[j]; - if (kidRef.num == ref.num && kidRef.gen == ref.gen) { + if (kidRef.num === ref.num && kidRef.gen === ref.gen) { break; } } From 2162a19ed956c0cef6e7dc3bffc5f0cc58be5d11 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 1 Aug 2014 18:23:49 +0200 Subject: [PATCH 2/6] Add strict equalities in src/core/arithmetic_decoder.js --- src/core/arithmetic_decoder.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/core/arithmetic_decoder.js b/src/core/arithmetic_decoder.js index 6d21cf63a..29f3a60f2 100644 --- a/src/core/arithmetic_decoder.js +++ b/src/core/arithmetic_decoder.js @@ -99,7 +99,7 @@ var ArithmeticDecoder = (function ArithmeticDecoderClosure() { byteIn: function ArithmeticDecoder_byteIn() { var data = this.data; var bp = this.bp; - if (data[bp] == 0xFF) { + if (data[bp] === 0xFF) { var b1 = data[bp + 1]; if (b1 > 0x8F) { this.clow += 0xFF00; From 84503c656d7549b0a59f05e2e12ec7e67b47d171 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 1 Aug 2014 18:24:12 +0200 Subject: [PATCH 3/6] Add strict equalities in src/core/bidi.js --- src/core/bidi.js | 55 ++++++++++++++++++++++++------------------------ 1 file changed, 28 insertions(+), 27 deletions(-) diff --git a/src/core/bidi.js b/src/core/bidi.js index a89e713b5..9d2969278 100644 --- a/src/core/bidi.js +++ b/src/core/bidi.js @@ -78,7 +78,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { function findUnequal(arr, start, value) { for (var j = start, jj = arr.length; j < jj; ++j) { - if (arr[j] != value) { + if (arr[j] !== value) { return j; } } @@ -138,7 +138,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { } else if (0x0700 <= charCode && charCode <= 0x08AC) { charType = 'AL'; } - if (charType == 'R' || charType == 'AL' || charType == 'AN') { + if (charType === 'R' || charType === 'AL' || charType === 'AN') { numBidi++; } types[i] = charType; @@ -153,7 +153,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { return createBidiText(str, isLTR); } - if (startLevel == -1) { + if (startLevel === -1) { if ((strLength / numBidi) < 0.3) { isLTR = true; startLevel = 0; @@ -182,7 +182,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { */ var lastType = sor; for (i = 0; i < strLength; ++i) { - if (types[i] == 'NSM') { + if (types[i] === 'NSM') { types[i] = lastType; } else { lastType = types[i]; @@ -198,9 +198,9 @@ var bidi = PDFJS.bidi = (function bidiClosure() { var t; for (i = 0; i < strLength; ++i) { t = types[i]; - if (t == 'EN') { - types[i] = (lastType == 'AL') ? 'AN' : 'EN'; - } else if (t == 'R' || t == 'L' || t == 'AL') { + if (t === 'EN') { + types[i] = (lastType === 'AL') ? 'AN' : 'EN'; + } else if (t === 'R' || t === 'L' || t === 'AL') { lastType = t; } } @@ -210,7 +210,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { */ for (i = 0; i < strLength; ++i) { t = types[i]; - if (t == 'AL') { + if (t === 'AL') { types[i] = 'R'; } } @@ -221,11 +221,12 @@ var bidi = PDFJS.bidi = (function bidiClosure() { type changes to that type: */ for (i = 1; i < strLength - 1; ++i) { - if (types[i] == 'ES' && types[i - 1] == 'EN' && types[i + 1] == 'EN') { + if (types[i] === 'ES' && types[i - 1] === 'EN' && types[i + 1] === 'EN') { types[i] = 'EN'; } - if (types[i] == 'CS' && (types[i - 1] == 'EN' || types[i - 1] == 'AN') && - types[i + 1] == types[i - 1]) { + if (types[i] === 'CS' && + (types[i - 1] === 'EN' || types[i - 1] === 'AN') && + types[i + 1] === types[i - 1]) { types[i] = types[i - 1]; } } @@ -235,18 +236,18 @@ var bidi = PDFJS.bidi = (function bidiClosure() { to all European numbers: */ for (i = 0; i < strLength; ++i) { - if (types[i] == 'EN') { + if (types[i] === 'EN') { // do before var j; for (j = i - 1; j >= 0; --j) { - if (types[j] != 'ET') { + if (types[j] !== 'ET') { break; } types[j] = 'EN'; } // do after for (j = i + 1; j < strLength; --j) { - if (types[j] != 'ET') { + if (types[j] !== 'ET') { break; } types[j] = 'EN'; @@ -259,7 +260,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { */ for (i = 0; i < strLength; ++i) { t = types[i]; - if (t == 'WS' || t == 'ES' || t == 'ET' || t == 'CS') { + if (t === 'WS' || t === 'ES' || t === 'ET' || t === 'CS') { types[i] = 'ON'; } } @@ -272,9 +273,9 @@ var bidi = PDFJS.bidi = (function bidiClosure() { lastType = sor; for (i = 0; i < strLength; ++i) { t = types[i]; - if (t == 'EN') { - types[i] = ((lastType == 'L') ? 'L' : 'EN'); - } else if (t == 'R' || t == 'L') { + if (t === 'EN') { + types[i] = ((lastType === 'L') ? 'L' : 'EN'); + } else if (t === 'R' || t === 'L') { lastType = t; } } @@ -286,7 +287,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { end-of-level-run (eor) are used at level run boundaries. */ for (i = 0; i < strLength; ++i) { - if (types[i] == 'ON') { + if (types[i] === 'ON') { var end = findUnequal(types, i + 1, 'ON'); var before = sor; if (i > 0) { @@ -297,13 +298,13 @@ var bidi = PDFJS.bidi = (function bidiClosure() { if (end + 1 < strLength) { after = types[end + 1]; } - if (before != 'L') { + if (before !== 'L') { before = 'R'; } - if (after != 'L') { + if (after !== 'L') { after = 'R'; } - if (before == after) { + if (before === after) { setValues(types, i, end, before); } i = end - 1; // reset to end (-1 so next iteration is ok) @@ -314,7 +315,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { N2. Any remaining neutrals take the embedding direction. */ for (i = 0; i < strLength; ++i) { - if (types[i] == 'ON') { + if (types[i] === 'ON') { types[i] = e; } } @@ -329,13 +330,13 @@ var bidi = PDFJS.bidi = (function bidiClosure() { for (i = 0; i < strLength; ++i) { t = types[i]; if (isEven(levels[i])) { - if (t == 'R') { + if (t === 'R') { levels[i] += 1; - } else if (t == 'AN' || t == 'EN') { + } else if (t === 'AN' || t === 'EN') { levels[i] += 2; } } else { // isOdd - if (t == 'L' || t == 'AN' || t == 'EN') { + if (t === 'L' || t === 'AN' || t === 'EN') { levels[i] += 1; } } @@ -414,7 +415,7 @@ var bidi = PDFJS.bidi = (function bidiClosure() { var result = ''; for (i = 0, ii = chars.length; i < ii; ++i) { var ch = chars[i]; - if (ch != '<' && ch != '>') { + if (ch !== '<' && ch !== '>') { result += ch; } } From 0012b8803c3f9871d74b6d8d286def608d067fb5 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 1 Aug 2014 18:25:58 +0200 Subject: [PATCH 4/6] Add strict equalities in src/core/colorspace.js --- src/core/colorspace.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/core/colorspace.js b/src/core/colorspace.js index 7f01840fa..0f1dc36e2 100644 --- a/src/core/colorspace.js +++ b/src/core/colorspace.js @@ -84,7 +84,7 @@ var ColorSpace = (function ColorSpaceClosure() { var count = originalWidth * originalHeight; var rgbBuf = null; var numComponentColors = 1 << bpc; - var needsResizing = originalHeight != height || originalWidth != width; + var needsResizing = originalHeight !== height || originalWidth !== width; var i, ii; if (this.isPassthrough(bpc)) { @@ -277,11 +277,11 @@ var ColorSpace = (function ColorSpaceClosure() { var stream = xref.fetchIfRef(cs[1]); var dict = stream.dict; numComps = dict.get('N'); - if (numComps == 1) { + if (numComps === 1) { return 'DeviceGrayCS'; - } else if (numComps == 3) { + } else if (numComps === 3) { return 'DeviceRgbCS'; - } else if (numComps == 4) { + } else if (numComps === 4) { return 'DeviceCmykCS'; } break; @@ -342,7 +342,7 @@ var ColorSpace = (function ColorSpaceClosure() { return true; } for (var i = 0, ii = decode.length; i < ii; i += 2) { - if (decode[i] !== 0 || decode[i + 1] != 1) { + if (decode[i] !== 0 || decode[i + 1] !== 1) { return false; } } @@ -607,7 +607,7 @@ var DeviceRgbCS = (function DeviceRgbCSClosure() { return (inputLength * (3 + alpha01) / 3) | 0; }, isPassthrough: function DeviceRgbCS_isPassthrough(bits) { - return bits == 8; + return bits === 8; }, fillRgb: ColorSpace.prototype.fillRgb, isDefaultDecode: function DeviceRgbCS_isDefaultDecode(decodeMap) { From 83a4c68df9ab8c9703d3da2c424db0b08926da2a Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 1 Aug 2014 18:24:51 +0200 Subject: [PATCH 5/6] Add strict equalities in src/core/core.js --- src/core/core.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/core/core.js b/src/core/core.js index 4d130b0fc..d61aedca6 100644 --- a/src/core/core.js +++ b/src/core/core.js @@ -311,7 +311,7 @@ var PDFDocument = (function PDFDocumentClosure() { var str = strBuf.join(''); stream.pos = pos; var index = backwards ? str.lastIndexOf(needle) : str.indexOf(needle); - if (index == -1) { + if (index === -1) { return false; /* not found */ } stream.pos += index; From 87038e44cd4f29dc40a8a152120dc44f351fa0ec Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 1 Aug 2014 18:25:21 +0200 Subject: [PATCH 6/6] Add strict equalities in src/core/evaluator.js --- src/core/evaluator.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/core/evaluator.js b/src/core/evaluator.js index abd9be3ef..82c669561 100644 --- a/src/core/evaluator.js +++ b/src/core/evaluator.js @@ -1436,7 +1436,7 @@ var PartialEvaluator = (function PartialEvaluatorClosure() { firstWidth = glyphWidth; continue; } - if (firstWidth != glyphWidth) { + if (firstWidth !== glyphWidth) { isMonospace = false; break; }