[Pkg-javascript-commits] [pdf.js] 193/414: Replace `getAll` with `getKeys` in `loadType3Data`

David Prévot taffit at moszumanska.debian.org
Tue Jun 28 17:12:21 UTC 2016


This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository pdf.js.

commit f7f60197ce6de81fcd0b3bdc177c0e55cd39f813
Author: Jonas Jenwald <jonas.jenwald at gmail.com>
Date:   Wed Feb 10 20:19:14 2016 +0100

    Replace `getAll` with `getKeys` in `loadType3Data`
    
    Not only is `getAll` less efficient, but given that we actually need the keys here, using `getKeys` seems much more suitable.
---
 src/core/evaluator.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/core/evaluator.js b/src/core/evaluator.js
index 0a3a5df..2fa42d7 100644
--- a/src/core/evaluator.js
+++ b/src/core/evaluator.js
@@ -2100,13 +2100,13 @@ var TranslatedFont = (function TranslatedFontClosure() {
 
       var translatedFont = this.font;
       var loadCharProcsPromise = Promise.resolve();
-      var charProcs = this.dict.get('CharProcs').getAll();
+      var charProcs = this.dict.get('CharProcs');
       var fontResources = this.dict.get('Resources') || resources;
-      var charProcKeys = Object.keys(charProcs);
+      var charProcKeys = charProcs.getKeys();
       var charProcOperatorList = Object.create(null);
       for (var i = 0, n = charProcKeys.length; i < n; ++i) {
         loadCharProcsPromise = loadCharProcsPromise.then(function (key) {
-          var glyphStream = charProcs[key];
+          var glyphStream = charProcs.get(key);
           var operatorList = new OperatorList();
           return evaluator.getOperatorList(glyphStream, task, fontResources,
                                            operatorList).then(function () {

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/pdf.js.git



More information about the Pkg-javascript-commits mailing list