From 2e93a0cc9878c8493b146e76a08365e9afa3b505 Mon Sep 17 00:00:00 2001
From: Nicholas Nethercote <nnethercote@mozilla.com>
Date: Tue, 11 Mar 2014 21:48:50 -0700
Subject: [PATCH] Avoid allocations in bidi().

---
 src/core/bidi.js | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/core/bidi.js b/src/core/bidi.js
index 6c28dc4b3..566ed6cb1 100644
--- a/src/core/bidi.js
+++ b/src/core/bidi.js
@@ -145,6 +145,11 @@ var bidi = PDFJS.bidi = (function bidiClosure() {
     this.dir = (vertical ? 'ttb' : (isLTR ? 'ltr' : 'rtl'));
   }
 
+  // These are used in bidi(), which is called frequently. We re-use them on
+  // each call to avoid unnecessary allocations.
+  var chars = [];
+  var types = [];
+
   function bidi(str, startLevel, vertical) {
     var isLTR = true;
     var strLength = str.length;
@@ -153,8 +158,8 @@ var bidi = PDFJS.bidi = (function bidiClosure() {
     }
 
     // Get types and fill arrays
-    var chars = [];
-    var types = [];
+    chars.length = 0;
+    types.length = 0;
     var numBidi = 0;
 
     for (var i = 0; i < strLength; ++i) {