[erlang-patches] [PATCH] erts: remove variable processes_busy

Jovi Zhang bookjovi@REDACTED
Tue Nov 29 05:41:06 CET 2011


2011/11/28 Björn Gustavsson <bgustavsson@REDACTED>:
> On 11/13/11, Jovi Zhang <bookjovi@REDACTED> wrote:
>> How about this patch?
>>
>> On Wed, Nov 9, 2011 at 5:04 PM, Jovi Zhang <bookjovi@REDACTED> wrote:
>>
>>>
>>> variable processes_busy is not used in current code, it don't have
>>> any meaning still stay on there, so remove it.
>>>
>>> Git Tree:               git pull git://github.com/bookjovi/otp.git
>>> Patch Review: *      *
>>> https://github.com/bookjovi/otp/commit/d428f18f1d03c133e2b59053978509bfae4d2c8c
>>>
>
> Thanks for noticing the useless variable. However, I don't
> agree with your way of fixing it. I will fix it like this:
>
> https://github.com/bjorng/otp/commit/7e8806063fb2d4eb9e15b48bd9f9c3f788fe3aa7
>
> --
> Björn Gustavsson, Erlang/OTP, Ericsson AB
>

By the way, my name is "Jovi Zhang", not "Jovi Shang", :)

But note that global variable is also not used in benchmark.c, that's
why I removed all processes_busy.

.jovi



More information about the erlang-patches mailing list