alex/yomichan
1
Fork 0
Code Releases Activity

Remove unnecessary functions

Browse Source 
apiOptionsSet not required in bg/js/api.js after optionsSave; optionsSave already invokes apiOptionsSet.
apiOptionsSet not required in fg/js/api.js since it's never invoked by the foreground.
optionsSet handler not required in bg/js/backend.js since the message is never sent by the foreground.
This commit is contained in:
toasted-nutbread 2019-09-06 21:06:45 -04:00
parent 35ca0f35dd
commit 13b184707b
3 changed files with 0 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
ext/bg/js/api.js
View File

@ -138,7 +138,6 @@ async function apiCommandExec(command) {
const options = apiOptionsGetSync();
options.general.enable = !options.general.enable;
await optionsSave(options);
await apiOptionsSet(options);
}
};

4
ext/bg/js/backend.js
View File

@ -85,10 +85,6 @@ class Backend {
forward(apiOptionsGet(), callback);
},
optionsSet: ({options, callback}) => {
forward(apiOptionsSet(options), callback);
},
kanjiFind: ({text, callback}) => {
forward(apiKanjiFind(text), callback);
},

4
ext/fg/js/api.js
View File

@ -17,10 +17,6 @@
*/
function apiOptionsSet(options) {
return utilInvoke('optionsSet', {options});
}
function apiOptionsGet() {
return utilInvoke('optionsGet');
}