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

Convert the pagesRefCache, on PDFLinkService, from an Object to a Map

Browse Source 
This seems like a more appropriate data structure, and as part of these changes the property was also converted to a *private* one.
This commit is contained in:
Jonas Jenwald 2022-01-03 13:57:53 +01:00
parent fc31e1ba87
commit 00aa9811e6
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
web/pdf_link_service.js
View File

@ -103,6 +103,8 @@ function addLinkAttributes(link, { url, target, rel, enabled = true } = {}) {
* @implements {IPDFLinkService} * @implements {IPDFLinkService}
*/ */
class PDFLinkService { class PDFLinkService {
#pagesRefCache = new Map();
/** /**
* @param {PDFLinkServiceOptions} options * @param {PDFLinkServiceOptions} options
*/ */
@ -122,14 +124,12 @@ class PDFLinkService {
this.pdfDocument = null; this.pdfDocument = null;
this.pdfViewer = null; this.pdfViewer = null;
this.pdfHistory = null; this.pdfHistory = null;
this._pagesRefCache = null;
} }
setDocument(pdfDocument, baseUrl = null) { setDocument(pdfDocument, baseUrl = null) {
this.baseUrl = baseUrl; this.baseUrl = baseUrl;
this.pdfDocument = pdfDocument; this.pdfDocument = pdfDocument;
this._pagesRefCache = Object.create(null); this.#pagesRefCache.clear();
} }
setViewer(pdfViewer) { setViewer(pdfViewer) {
@ -505,7 +505,7 @@ class PDFLinkService {
} }
const refStr = const refStr =
pageRef.gen === 0 ? `${pageRef.num}R` : `${pageRef.num}R${pageRef.gen}`; pageRef.gen === 0 ? `${pageRef.num}R` : `${pageRef.num}R${pageRef.gen}`;
this._pagesRefCache[refStr] = pageNum; this.#pagesRefCache.set(refStr, pageNum);
} }
/** /**
@ -517,7 +517,7 @@ class PDFLinkService {
} }
const refStr = const refStr =
pageRef.gen === 0 ? `${pageRef.num}R` : `${pageRef.num}R${pageRef.gen}`; pageRef.gen === 0 ? `${pageRef.num}R` : `${pageRef.num}R${pageRef.gen}`;
return this._pagesRefCache?.[refStr] || null; return this.#pagesRefCache.get(refStr) || null;
} }
/** /**