From: Philipp Hagemeister Date: Mon, 20 Jun 2016 11:29:13 +0000 (+0200) Subject: Revert "[jsinterp] Avoid double key lookup for setting new key" X-Git-Url: http://git.cielonegro.org/gitweb.cgi?a=commitdiff_plain;h=1f749b6658439049b952fdb979acb6c4422a358a;hp=819707920a63946ea1e4f0ae2bf842425d22c2e9;p=youtube-dl.git Revert "[jsinterp] Avoid double key lookup for setting new key" This reverts commit 7c05097633138459e9bdf7e10738e021b04689a7. --- diff --git a/youtube_dl/jsinterp.py b/youtube_dl/jsinterp.py index 4a5a0dbc3..a7440c582 100644 --- a/youtube_dl/jsinterp.py +++ b/youtube_dl/jsinterp.py @@ -131,8 +131,9 @@ class JSInterpreter(object): if variable in local_vars: obj = local_vars[variable] else: - obj = self._objects.setdefault( - variable, self.extract_object(variable)) + if variable not in self._objects: + self._objects[variable] = self.extract_object(variable) + obj = self._objects[variable] if arg_str is None: # Member access @@ -203,7 +204,8 @@ class JSInterpreter(object): argvals = tuple([ int(v) if v.isdigit() else local_vars[v] for v in m.group('args').split(',')]) - self._functions.setdefault(fname, self.extract_function(fname)) + if fname not in self._functions: + self._functions[fname] = self.extract_function(fname) return self._functions[fname](argvals) raise ExtractorError('Unsupported JS expression %r' % expr)