<div dir="ltr"><div>Could you rebase the branch on master?<br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Dec 1, 2013 at 1:46 PM, Anthony Ramine <span dir="ltr"><<a href="mailto:n.oxyde@gmail.com" target="_blank">n.oxyde@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Amended with your fixup and some covering code for functions named with "_" in xref_SUITE.<br>
<br>
Please refetch.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Anthony Ramine<br>
<br>
Le 28 nov. 2013 à 13:52, Anthony Ramine <<a href="mailto:n.oxyde@gmail.com">n.oxyde@gmail.com</a>> a écrit :<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> Hello Björn,<br>
><br>
> Replied inline.<br>
><br>
> --<br>
> Anthony Ramine<br>
><br>
> Le 28 nov. 2013 à 13:44, Björn Gustavsson <<a href="mailto:bgustavsson@gmail.com">bgustavsson@gmail.com</a>> a écrit :<br>
><br>
>> On Wed, Nov 14, 2012 at 5:42 PM, Anthony Ramine <<a href="mailto:n.oxyde@gmail.com">n.oxyde@gmail.com</a>> wrote:<br>
>> This patch implements EEP37: Funs with names<br>
>><br>
>><br>
>> It is now much closer to being graduated.<br>
>><br>
>> I have added the branch to our daily builds for the master branch.<br>
><br>
> Cool.<br>
><br>
>> I would like a comment from Kostis about the Dialyzer changes.<br>
><br>
> Me too, he suggested to me that it might break HiPE too but I couldn’t make HiPE crash.<br>
><br>
>> What also remains is coverage in test suites. I have fixed a few,<br>
>> but xref_SUITE remains to be fixed (a named fun with the '_' name<br>
>> should be tested).<br>
><br>
> Will do.<br>
><br>
>> Here are my fixes for the test suites (also included as an attachment<br>
>> to make it easier to apply):<br>
><br>
> Will apply.<br>
><br>
>> diff --git a/lib/compiler/test/fun_SUITE.erl b/lib/compiler/test/fun_SUITE.erl<br>
>> index 0ff846e..e35692e 100644<br>
>> --- a/lib/compiler/test/fun_SUITE.erl<br>
>> +++ b/lib/compiler/test/fun_SUITE.erl<br>
>> @@ -197,10 +197,14 @@ external(Config) when is_list(Config) -><br>
>> call_me(I) -><br>
>>     {ok,I}.<br>
>><br>
>> -id(I) -><br>
>> -    I.<br>
>> -<br>
>> eep37(Config) when is_list(Config) -><br>
>>     F = fun Fact(N) when N > 0 -> N * Fact(N - 1); Fact(0) -> 1 end,<br>
>> +    Add = fun _(N) -> N + 1 end,<br>
>> +    UnusedName = fun BlackAdder(N) -> N + 42 end,<br>
>>     720 = F(6),<br>
>> +    10 = Add(9),<br>
>> +    50 = UnusedName(8),<br>
>>     ok.<br>
>> +<br>
>> +id(I) -><br>
>> +    I.<br>
>> diff --git a/lib/debugger/test/fun_SUITE.erl b/lib/debugger/test/fun_SUITE.erl<br>
>> index 569af6c..75e3b55 100644<br>
>> --- a/lib/debugger/test/fun_SUITE.erl<br>
>> +++ b/lib/debugger/test/fun_SUITE.erl<br>
>> @@ -288,10 +288,14 @@ external(Config) when is_list(Config) -><br>
>> call_me(I) -><br>
>>     {ok,I}.<br>
>><br>
>> -id(I) -><br>
>> -    I.<br>
>> -<br>
>> eep37(Config) when is_list(Config) -><br>
>>     F = fun Fact(N) when N > 0 -> N * Fact(N - 1); Fact(0) -> 1 end,<br>
>> +    Add = fun _(N) -> N + 1 end,<br>
>> +    UnusedName = fun BlackAdder(N) -> N + 42 end,<br>
>>     720 = F(6),<br>
>> +    10 = Add(9),<br>
>> +    50 = UnusedName(8),<br>
>>     ok.<br>
>> +<br>
>> +id(I) -><br>
>> +    I.<br>
>> diff --git a/lib/stdlib/test/erl_lint_SUITE.erl b/lib/stdlib/test/erl_lint_SUITE.erl<br>
>> index ea90793..ad51bfa 100644<br>
>> --- a/lib/stdlib/test/erl_lint_SUITE.erl<br>
>> +++ b/lib/stdlib/test/erl_lint_SUITE.erl<br>
>> @@ -644,7 +644,9 @@ unused_vars_warn_fun(Config) when is_list(Config) -><br>
>>               u() -><br>
>>                   fun U(U) -> foo end; % U shadowed. U unused.<br>
>>               u() -><br>
>> -                  fun U(1) -> U; U(U) -> foo end. % U shadowed. U unused.<br>
>> +                  fun U(1) -> U; U(U) -> foo end; % U shadowed. U unused.<br>
>> +              u() -><br>
>> +                  fun _(N) -> N + 1 end.  % Cover handling of '_' name.<br>
>>            ">>,<br>
>>            [warn_unused_vars],<br>
>>            {error,[{3,erl_lint,{unbound_var,'U'}}],<br>
>><br>
>><br>
>> id/1 is a common help function, so I want it to remain at the end<br>
>> of the file.<br>
><br>
> Thanks for nitpicking even more than myself :p<br>
><br>
>> /Bjorn<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Björn Gustavsson, Erlang/OTP, Ericsson AB
</div>