open.source/stylus
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix: cacheFirstCall -> memoize

Browse Source
This commit is contained in:
eight 2018-09-02 14:36:18 +08:00
parent 8531d20c55
commit b27b6676fa
5 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
edit/linter-config-dialog.js
View File

@ -1,4 +1,4 @@
/* global cacheFirstCall editorWorker stylelint csslint showCodeMirrorPopup loadScript messageBox */ /* global memoize editorWorker stylelint csslint showCodeMirrorPopup loadScript messageBox */
'use strict'; 'use strict';
(() => { (() => {
@ -26,7 +26,7 @@
return; return;
} }
const storageName = linter === 'styleint' ? 'editorStylelintConfig' : 'editorCSSLintConfig'; const storageName = linter === 'styleint' ? 'editorStylelintConfig' : 'editorCSSLintConfig';
const getRules = cacheFirstCall(linter === 'stylelint' ? const getRules = memoize(linter === 'stylelint' ?
editorWorker.getStylelintRules : editorWorker.getCsslintRules); editorWorker.getStylelintRules : editorWorker.getCsslintRules);
const linterTitle = linter === 'stylelint' ? 'Stylelint' : 'CSSLint'; const linterTitle = linter === 'stylelint' ? 'Stylelint' : 'CSSLint';
const defaultConfig = stringifyConfig( const defaultConfig = stringifyConfig(

4
edit/linter-csslint.js
View File

@ -1,4 +1,4 @@
/* global linter editorWorker cacheFirstCall */ /* global linter editorWorker memoize */
'use strict'; 'use strict';
// eslint-disable-next-line no-var // eslint-disable-next-line no-var
@ -49,7 +49,7 @@ var csslint = (() => {
}; };
let config; let config;
const prepareConfig = cacheFirstCall(() => { const prepareConfig = memoize(() => {
chrome.storage.onChanged.addListener((changes, area) => { chrome.storage.onChanged.addListener((changes, area) => {
if (area !== 'sync' || !changes.hasOwnProperty('editorCSSLintConfig')) { if (area !== 'sync' || !changes.hasOwnProperty('editorCSSLintConfig')) {
return; return;

4
edit/linter-help-dialog.js
View File

@ -1,9 +1,9 @@
/* global showHelp editorWorker cacheFirstCall */ /* global showHelp editorWorker memoize */
'use strict'; 'use strict';
function createLinterHelpDialog(getIssues) { function createLinterHelpDialog(getIssues) {
let csslintRules; let csslintRules;
const prepareCsslintRules = cacheFirstCall(() => const prepareCsslintRules = memoize(() =>
editorWorker.getCsslintRules() editorWorker.getCsslintRules()
.then(rules => { .then(rules => {
csslintRules = rules; csslintRules = rules;

4
edit/linter-stylelint.js
View File

@ -1,4 +1,4 @@
/* global linter editorWorker cacheFirstCall */ /* global linter editorWorker memoize */
'use strict'; 'use strict';
// eslint-disable-next-line no-var // eslint-disable-next-line no-var
@ -174,7 +174,7 @@ var stylelint = (() => {
}; };
let config; let config;
const prepareConfig = cacheFirstCall(() => { const prepareConfig = memoize(() => {
chrome.storage.onChanged.addListener((changes, area) => { chrome.storage.onChanged.addListener((changes, area) => {
if (area !== 'sync' || !changes.hasOwnProperty('editorStylelintConfig')) { if (area !== 'sync' || !changes.hasOwnProperty('editorStylelintConfig')) {
return; return;

2
edit/util.js
View File

@ -123,7 +123,7 @@ function clipString(str, limit = 100) {
} }
// this is a decorator. Cache the first call // this is a decorator. Cache the first call
function cacheFirstCall(fn) { function memoize(fn) {
let cached = false; let cached = false;
let result; let result;
return (...args) => { return (...args) => {