New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memoize ELECTRON_NO_ASAR env var check #7978

Merged
merged 1 commit into from Nov 16, 2016
Jump to file or symbol
Failed to load files and symbols.
+2 鈭7
Diff settings

Always

Just for now

Memoize ELECTRON_NO_ASAR env var check

  • Loading branch information...
kevinsawicki committed Nov 15, 2016
commit bea1a0643c30f0a9c7da28e56e1549c46a63d7cf
View
@@ -10,14 +10,9 @@
// Cache asar archive objects.
const cachedArchives = {}
const envNoAsar = process.env.ELECTRON_NO_ASAR && process.type !== 'browser' && process.type !== 'renderer'
const isAsarDisabled = function () {
if (process.noAsar) {
return true
}
if (process.env.ELECTRON_NO_ASAR && process.type !== 'browser' && process.type !== 'renderer') {
return true
}
return false
return process.noAsar || envNoAsar
}
const getOrCreateArchive = function (p) {
ProTip! Use n and p to navigate between commits in a pull request.