Revert "Expand macros and quoted variables inside << >>, closes #1063"

It broke the build. Reopens #1063.

This reverts commit 93f71c8.

Author: José Valim

lib/elixir/src/elixir_literal.erl

@@ -10,9 +10,9 @@ translate({ '<<>>', Meta, Args } = Original, S) when is_list(Args) ->
     error ->
       case S#elixir_scope.context of
         match ->
-          build_bitstr(fun elixir_translator:translate_each/2, Args, Meta, nil, S);
+          build_bitstr(fun elixir_translator:translate_each/2, Args, Meta, S);
         _ ->
-          { TArgs, { SC, SV } } = build_bitstr(fun elixir_translator:translate_arg/2, Args, Meta, nil, { S, S }),
+          { TArgs, { SC, SV } } = build_bitstr(fun elixir_translator:translate_arg/2, Args, Meta, { S, S }),
           { TArgs, umergec(SV, SC) }
       end;
     Else -> Else
@@ -75,72 +75,58 @@ translate(Bitstring, S) when is_bitstring(Bitstring) ->
 % * If '::' is given, extract the bitstring information
 % * All the other types are simply translated and handled with Erlang's default
 %
-build_bitstr(Fun, Exprs, Meta, Env, S) ->
-  { Final, FinalS } = build_bitstr_each(Fun, Exprs, Meta, Env, S, []),
+build_bitstr(Fun, Exprs, Meta, S) ->
+  { Final, FinalS } = build_bitstr_each(Fun, Exprs, Meta, S, []),
   { { bin, ?line(Meta), lists:reverse(Final) }, FinalS }.
 
-build_bitstr_each(_Fun, [], _Meta, _Env, S, Acc) ->
+build_bitstr_each(_Fun, [], _Meta, S, Acc) ->
   { Acc, S };
 
-build_bitstr_each(Fun, [{'::',_,[H,V]}|T], Meta, Env, S, Acc) ->
+build_bitstr_each(Fun, [{'::',_,[H,V]}|T], Meta, S, Acc) ->
   %% Variables defined outside the binary can be accounted
   %% on subparts, however we can't assign new variables.
   case S of
     { ES, _ } -> [];                      %% translate_arg,  no assigns
     _ -> ES = S#elixir_scope{context=nil} %% translate_each, revert assigns
   end,
 
-  { ExpandedH, ExpandedEnv } = expand(Meta, H, Env, S),
-  { Size, Types } = extract_bit_values(Meta, V, ExpandedEnv, ES),
-  build_bitstr_each(Fun, T, Meta, ExpandedEnv, S, Acc, ExpandedH, Size, Types);
+  { Size, Types } = extract_bit_values(Meta, V, ES),
+  build_bitstr_each(Fun, T, Meta, S, Acc, H, Size, Types);
 
-build_bitstr_each(Fun, [H|T], Meta, Env, S, Acc) ->
-  { ExpandedH, ExpandedEnv } = expand(Meta, H, Env, S),
-  build_bitstr_each(Fun, T, Meta, ExpandedEnv, S, Acc, ExpandedH, default, default).
+build_bitstr_each(Fun, [H|T], Meta, S, Acc) ->
+  build_bitstr_each(Fun, T, Meta, S, Acc, H, default, default).
 
-expand(_Meta, H, Env, _S) when is_integer(H); is_float(H); is_atom(H); is_atom(H); is_bitstring(H) ->
-  { H, Env };
-
-expand(Meta, H, nil, { S, _ }) ->
-  expand(Meta, H, elixir_scope:to_ex_env({ ?line(Meta), S }), S);
-
-expand(Meta, H, nil, S) ->
-  expand(Meta, H, elixir_scope:to_ex_env({ ?line(Meta), S }), S);
-
-expand(_Meta, H, Env, _S) ->
-  { 'Elixir.Macro':expand(H, Env), Env }.
-
-build_bitstr_each(Fun, T, Meta, Env, S, Acc, H, Size, Types) when is_list(H) ->
+build_bitstr_each(Fun, T, Meta, S, Acc, H, Size, Types) when is_list(H) ->
   case is_default_or_utf(Types) of
     true ->
       { NewAcc, NewS } = lists:foldl(fun(L, { LA, LS }) ->
         { FL, FS } = Fun(L, LS),
         { [{ bin_element, ?line(Meta), FL, Size, Types }|LA], FS }
       end, { Acc, S }, H),
-      build_bitstr_each(Fun, T, Meta, Env, NewS, NewAcc);
+      build_bitstr_each(Fun, T, Meta, NewS, NewAcc);
     false ->
-      build_bitstr_default(Fun, T, Meta, Env, S, Acc, H, Size, Types)
+      build_bitstr_default(Fun, T, Meta, S, Acc, H, Size, Types)
   end;
 
-build_bitstr_each(Fun, T, Meta, Env, S, Acc, H, Size, Types) when is_bitstring(H) ->
+build_bitstr_each(Fun, T, Meta, S, Acc, H, Size, Types) when is_bitstring(H) ->
   case is_default_or_utf(Types) of
     true ->
       Line = ?line(Meta),
       { bin, _, Elements } = elixir_tree_helpers:elixir_to_erl(H),
       NewAcc = lists:foldl(fun({ bin_element, _, Expr, _, _ }, FinalAcc) ->
         [{ bin_element, Line, Expr, Size, Types }|FinalAcc]
       end, Acc, Elements),
-      build_bitstr_each(Fun, T, Meta, Env, S, NewAcc);
+      build_bitstr_each(Fun, T, Meta, S, NewAcc);
     false ->
-      build_bitstr_default(Fun, T, Meta, Env, S, Acc, H, Size, Types)
+      build_bitstr_default(Fun, T, Meta, S, Acc, H, Size, Types)
   end;
 
-build_bitstr_each(Fun, T, Meta, Env, S, Acc, H, Size, Types) ->
-  build_bitstr_default(Fun, T, Meta, Env, S, Acc, H, Size, Types).
+build_bitstr_each(Fun, T, Meta, S, Acc, H, Size, Types) ->
+  build_bitstr_default(Fun, T, Meta, S, Acc, H, Size, Types).
 
-build_bitstr_default(Fun, T, Meta, Env, S, Acc, H, Size, Types) ->
+build_bitstr_default(Fun, T, Meta, S, Acc, H, Size, Types) ->
   { Expr, NS } = Fun(H, S),
-  build_bitstr_each(Fun, T, Meta, Env, NS, [{ bin_element, ?line(Meta), Expr, Size, Types }|Acc]).
+  build_bitstr_each(Fun, T, Meta, NS, [{ bin_element, ?line(Meta), Expr, Size, Types }|Acc]).
 
 is_default_or_utf(default) -> true;
 is_default_or_utf([UTF|_]) when UTF == utf8; UTF == utf16; UTF == utf32 -> true;
@@ -149,46 +135,46 @@ is_default_or_utf([]) -> false.
 
 %% Extra bitstring specifiers
 
-extract_bit_values(Meta, V, Env, S) when is_list(V) ->
-  extract_bit_values(Meta, V, default, [], Env, S);
+extract_bit_values(Meta, V, S) when is_list(V) ->
+  extract_bit_values(Meta, V, default, [], S);
 
-extract_bit_values(Meta, V, Env, S) ->
-  extract_bit_values(Meta, [V], Env, S).
+extract_bit_values(Meta, V, S) ->
+  extract_bit_values(Meta, [V], S).
 
-extract_bit_values(Meta, [{ Value, _Meta, Atom }|T] = All, Size, Types, Env, S)
+extract_bit_values(Meta, [{ Value, _Meta, Atom }|T] = All, Size, Types, S)
     when is_atom(Value) andalso (is_atom(Atom) orelse Atom == []) ->
   case extract_bit_type(Value, Types) of
     { error, unknown } ->
-      handle_unknown_specifier(Meta, All, Size, Types, Env, S);
+      handle_unknown_specifier(Meta, All, Size, Types, S);
     NewTypes when is_list(NewTypes) ->
-      extract_bit_values(Meta, T, Size, NewTypes, Env, S)
+      extract_bit_values(Meta, T, Size, NewTypes, S)
   end;
 
-extract_bit_values(Meta, [{ Value, CallMeta, [_] = Args }|T] = All, Size, Types, Env, S) when is_atom(Value) ->
+extract_bit_values(Meta, [{ Value, CallMeta, [_] = Args }|T] = All, Size, Types, S) when is_atom(Value) ->
   { TArgs, _ } = elixir_translator:translate_args(Args, S),
   case extract_bit_type_or_size(Value, TArgs, Size, Types) of
     { error, unknown } ->
-      handle_unknown_specifier(Meta, All, Size, Types, Env, S);
+      handle_unknown_specifier(Meta, All, Size, Types, S);
     { error, Msg } ->
       elixir_errors:syntax_error(CallMeta, S#elixir_scope.file, Msg);
     { NewSize, NewTypes } ->
-      extract_bit_values(Meta, T, NewSize, NewTypes, Env, S)
+      extract_bit_values(Meta, T, NewSize, NewTypes, S)
   end;
 
-extract_bit_values(Meta, [Size|T], default, Types, Env, S) when is_integer(Size) andalso Size >= 0 ->
-  extract_bit_values(Meta, T, {integer, Meta, Size}, Types, Env, S);
+extract_bit_values(Meta, [Size|T], default, Types, S) when is_integer(Size) andalso Size >= 0 ->
+  extract_bit_values(Meta, T, {integer, Meta, Size}, Types, S);
 
-extract_bit_values(Meta, [{ '|', _, [Left, Right] }], Size, Types, Env, S) ->
-  { Expanded, ExpandedEnv } = expand(Meta, Right, Env, S),
-  extract_bit_values(Meta, [Left|join_expansion(Expanded,[])], Size, Types, ExpandedEnv, S);
+extract_bit_values(Meta, [{ '|', _, [Left, Right] }], Size, Types, S) ->
+  Expanded = 'Elixir.Macro':expand(Right, elixir_scope:to_ex_env({ ?line(Meta), S })),
+  extract_bit_values(Meta, [Left|join_expansion(Expanded,[])], Size, Types, S);
 
-extract_bit_values(Meta, [_|_] = All, Size, Types, Env, S) ->
-  handle_unknown_specifier(Meta, All, Size, Types, Env, S);
+extract_bit_values(Meta, [_|_] = All, Size, Types, S) ->
+  handle_unknown_specifier(Meta, All, Size, Types, S);
 
-extract_bit_values(_Meta, [], Size, [], _Env, _S) ->
+extract_bit_values(_Meta, [], Size, [], _S) ->
   { Size, default };
 
-extract_bit_values(_Meta, [], Size, Types, _Env, _S) ->
+extract_bit_values(_Meta, [], Size, Types, _S) ->
   { Size, lists:reverse(Types) }.
 
 extract_bit_type(Value, Types) when
@@ -220,12 +206,12 @@ extract_bit_type_or_size(unit, _Args, _Other, _Types) ->
 extract_bit_type_or_size(_Value, _Args, _Other, _Types) ->
   { error, unknown }.
 
-handle_unknown_specifier(Meta, [H|T], Size, Types, Env, S) ->
-  case expand(Meta, H, Env, S) of
-    { H, _ } ->
+handle_unknown_specifier(Meta, [H|T], Size, Types, S) ->
+  case 'Elixir.Macro':expand(H, elixir_scope:to_ex_env({ ?line(Meta), S })) of
+    H ->
       elixir_errors:syntax_error(Meta, S#elixir_scope.file, "unknown bitstring specifier ~ts", ['Elixir.Macro':to_binary(H)]);
-    { E, ExpandedEnv } ->
-      extract_bit_values(Meta, join_expansion(E,T), Size, Types, ExpandedEnv, S)
+    E ->
+      extract_bit_values(Meta, join_expansion(E,T), Size, Types, S)
   end.
 
 join_expansion({ '__block__', _, [Expanded] }, Tail) -> join_expansion(Expanded, Tail);

lib/elixir/test/elixir/binary_test.exs

@@ -98,14 +98,6 @@ bar
     assert <<0,0,0,106,0,0,0,111,0,0,0,115,0,0,0,101>> == << 'jose' :: utf32 >>
   end
 
-  @binary   "new "
-  @charlist 'old '
-
-  test :bitsyntax_with_expansion do
-    assert <<@binary, "world">> == "new world"
-    assert <<@charlist, "world">> == "old world"
-  end
-
   test :bitsyntax_translation do
     refb = "sample"
     sec_data = "another"