From 82533eae2d44dd1079cbfb06e542f19df9ec3a69 Mon Sep 17 00:00:00 2001 From: Bryan Ashby Date: Thu, 1 Sep 2016 23:57:57 -0600 Subject: [PATCH] Don't re-add + log MCI cache if already cached. Also fix wording :) --- core/art.js | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/core/art.js b/core/art.js index 22408cc9..0e283218 100644 --- a/core/art.js +++ b/core/art.js @@ -49,7 +49,7 @@ function getFontNameFromSAUCE(sauce) { function sliceAtEOF(data, eofMarker) { let eof = data.length; const stopPos = Math.max(data.length - (256), 0); // 256 = 2 * sizeof(SAUCE) - + for(let i = eof - 1; i > stopPos; i--) { if(eofMarker === data[i]) { eof = i; @@ -256,16 +256,17 @@ function display(client, art, options, cb) { let mciMap; const mciCprQueue = []; let artHash; + let mciMapFromCache; function completed() { if(cprListener) { client.removeListener('cursor position report', cprListener); } - if(!options.disableMciCache) { + if(!options.disableMciCache && !mciMapFromCache) { // cache our MCI findings... client.mciCache[artHash] = mciMap; - client.log.trace( { artHash : artHash, mciMap : mciMap }, 'Added MCI map from cache'); + client.log.trace( { artHash : artHash, mciMap : mciMap }, 'Added MCI map to cache'); } ansiParser.removeAllListeners(); // :TODO: Necessary??? @@ -288,6 +289,7 @@ function display(client, art, options, cb) { } if(mciMap) { + mciMapFromCache = true; client.log.trace( { artHash : artHash, mciMap : mciMap }, 'Loaded MCI map from cache'); } else { // no cached MCI info