[erlang-patches] Fix documentation for erlang:process_info/2

Henrik Nord henrik@REDACTED
Mon Oct 3 17:03:02 CEST 2011


On 09/29/2011 07:15 PM, Filipe David Manana wrote:
> On Thu, Sep 29, 2011 at 5:35 AM, Henrik Nord<henrik@REDACTED>  wrote:
>> On 09/20/2011 07:57 PM, Filipe David Manana wrote:
>>> git  fetch  git://github.com/fdmanana/otp.git  process_info_doc_fix
>>>
>>> Web diff:
>>> https://github.com/fdmanana/otp/compare/process_info_doc_fix
>>>
>> Hi
>>
>> We will not include this patch as it removes the description of the max
>> priority.
> I think it was misunderstood the goal of the patch.
> The same exact paragraph, about process priorities, is duplicated in 2 places:
>
> https://github.com/erlang/otp/blob/dev/erts/doc/src/erlang.xml#L3660
>
> and
>
> https://github.com/erlang/otp/blob/dev/erts/doc/src/erlang.xml#L3707
>
> The first one makes sense, as it's under the section describing
> process_flag(priority, Level). The second one however seems like a
> copy paste accident because it's under the section describing
> process_flag(save_calls, N).
>
>> However it might not be the best written part of the documentation, and if
>> you want to reword it to make it easy for the reader we are willing to
>> accept such a patch.
>>
>> --
>> /Henrik Nord Erlang/OTP
>>
>> _______________________________________________
>> erlang-patches mailing list
>> erlang-patches@REDACTED
>> http://erlang.org/mailman/listinfo/erlang-patches
>>
>
>
Ah
we managed to miss that even when we had 2 pairs of eyes on the code.

I will include the patch

Thank you for the contribution!


-- 
/Henrik Nord Erlang/OTP




More information about the erlang-patches mailing list