Skip to content

Commit f8c66c8

Browse files
author
José Valim
committed
Remove unecessary vars cache
1 parent f090045 commit f8c66c8

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

lib/elixir/include/elixir.hrl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
module=nil, %% the current module
1818
function=nil, %% the current function
1919
vars=[], %% a dict of defined variables and their alias
20-
list_vars=nil, %% a list of vars passed down to Macro.Env
2120
backup_vars=nil, %% a copy of vars to be used on ^var
2221
temp_vars=nil, %% a set of all variables defined in a particular assign
2322
clause_vars=nil, %% a dict of all variables defined in a particular clause

lib/elixir/src/elixir_scope.erl

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -81,24 +81,21 @@ build_ex_var(Line, Key, Name, S) when is_integer(Line) ->
8181
% Handle Macro.Env conversion
8282

8383
to_erl_env({ 'Elixir.Macro.Env', Module, File, _Line, Function, Aliases, Context,
84-
Requires, Functions, Macros, ContextModules, MacroAliases, Vars }) ->
84+
Requires, Functions, Macros, ContextModules, MacroAliases, _Vars }) ->
8585
#elixir_scope{module=Module,file=File,
8686
function=Function,aliases=Aliases,context=Context,
8787
requires=Requires,macros=Macros,functions=Functions,
88-
context_modules=ContextModules,macro_aliases=MacroAliases,
89-
list_vars=Vars}.
88+
context_modules=ContextModules,macro_aliases=MacroAliases}.
9089

9190
to_ex_env({ Line, #elixir_scope{module=Module,file=File,
9291
function=Function,aliases=Aliases,context=Context,
9392
requires=Requires,macros=Macros,functions=Functions,
94-
context_modules=ContextModules,macro_aliases=MacroAliases,
95-
vars=Vars,list_vars=ListVars} }) when is_integer(Line) ->
93+
context_modules=ContextModules,macro_aliases=MacroAliases, vars=Vars} }) when is_integer(Line) ->
9694
{ 'Elixir.Macro.Env', Module, File, Line, Function, Aliases,
9795
Context, Requires, Functions, Macros, ContextModules, MacroAliases,
98-
list_vars(ListVars, Vars) }.
96+
list_vars(Vars) }.
9997

100-
list_vars(nil, Vars) -> [K || { K, _ } <- Vars];
101-
list_vars(Other, _) -> Other.
98+
list_vars(Vars) -> [K || { K, _ } <- Vars].
10299

103100
% Provides a tuple with only the scope information we want to serialize.
104101

0 commit comments

Comments
 (0)