[erlang-patches] [patch] Added application:get_env/3

Björn-Egil Dahlberg egil@REDACTED
Mon Jan 14 16:00:09 CET 2013


On 2013-01-14 15:55, Fredrik wrote:
> Hello,
> The answer is nothing. Just remove the '?line' that you've added. The 
> principle is that if we adding code we don't include the ?line and if 
> we are changing the code we are removing ?line from the lines we are 
> changing. Not removing anything else right now.
I usually remove the rest of the "?line" macros in a separate commit 
when I'm changing the file anyways.

However, we want to keep patches from the community as small as 
possible, i.e. just remove them on the lines you change. Leave the rest 
to us as Fredrik says. =)

>
> BR Fredrik Gustafsson
> Erlang OTP Team
> On 01/14/2013 03:52 PM, Serge Aleynikov wrote:
>> Fredrik,
>>
>> What should be used in place of the ?line macro?  All test cases in that
>> file use the ?line macro, e.g. see application_SUITE:get_key/1.
>>
>> Serge
>>
>> On 1/14/2013 5:13 AM, Fredrik wrote:
>>> Hello Serge,
>>> Your patch is under review but I noticed during examination that you 
>>> are
>>> using the ?line macro in your testcase provided. This is not used
>>> anymore and therefore should not be included, could you fix this?
>>>
>>> BR Fredrik Gustafsson
>>> Erlang OTP Team
>>> On 01/10/2013 03:31 PM, Serge Aleynikov wrote:
>>>> Added an application:get_env/3 function variant that provides a 
>>>> default
>>>> value for a configuration parameter:
>>>>
>>>>      application:get_env(App, Key, Default) ->   Value.
>>>>
>>>> git fetch git://github.com/saleyn/otp.git get_env
>>>>
>>>> https://github.com/saleyn/otp/compare/erlang:master...get_env
>>>> https://github.com/saleyn/otp/compare/erlang:master...get_env.patch
>>>> _______________________________________________
>>>> erlang-patches mailing list
>>>> erlang-patches@REDACTED
>>>> http://erlang.org/mailman/listinfo/erlang-patches
>
> _______________________________________________
> erlang-patches mailing list
> erlang-patches@REDACTED
> http://erlang.org/mailman/listinfo/erlang-patches
>
>




More information about the erlang-patches mailing list