From 9acaaf5126b7cb2b02dd72f45f42dfdf3d12c6b4 Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Mon, 5 Aug 2019 17:56:09 +0200
Subject: [PATCH] Use more compact keys in `PDFLinkService._pagesRefCache`

By using the same internal formatting here as in the `Ref.toString` method, in `src/core/primitives.js`, all cache-keys will become at least two bytes shorter (and most three bytes shorter).
Obviously this won't have a huge effect on memory since there's only one cache entry per page, but it nonetheless seems wasteful to use longer keys than strictly required.
---
 web/pdf_link_service.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/web/pdf_link_service.js b/web/pdf_link_service.js
index 44bd88a34..f7896f99f 100644
--- a/web/pdf_link_service.js
+++ b/web/pdf_link_service.js
@@ -344,12 +344,14 @@ class PDFLinkService {
     if (!pageRef) {
       return;
     }
-    let refStr = pageRef.num + ' ' + pageRef.gen + ' R';
+    const refStr = pageRef.gen === 0 ? `${pageRef.num}R` :
+                                       `${pageRef.num}R${pageRef.gen}`;
     this._pagesRefCache[refStr] = pageNum;
   }
 
   _cachedPageNumber(pageRef) {
-    let refStr = pageRef.num + ' ' + pageRef.gen + ' R';
+    const refStr = pageRef.gen === 0 ? `${pageRef.num}R` :
+                                       `${pageRef.num}R${pageRef.gen}`;
     return (this._pagesRefCache && this._pagesRefCache[refStr]) || null;
   }