[erlang-questions] rpc:call/4
Jayson Vantuyl
kagato@REDACTED
Fri Feb 19 13:12:51 CET 2010
I'll try to have a patch for the initial case (rpc:call/4) today.
Thanks,
On Feb 19, 2010, at 4:10 AM, Björn Gustavsson wrote:
> On Fri, Feb 19, 2010 at 11:09 AM, Jayson Vantuyl <kagato@REDACTED> wrote:
>
>> Is it worth my time to submit a patch?
>
> I would say yes, and good time would be now so that it has a chance
> to be included in R14.
>
> But we can't guarantee that we will include any patch until we have seen
> it. We will need to evaluate whether there are backward compatibility issues
> or other problems.
>
>> For that matter, I'd be interested in just eliminating this idiom from the whole of OTP.
>> I don't think it's ever justified anymore. Of course, a crude check shows 2082 occurrences. That would be a pretty big patch. Then again, 370 of them are in tests, so that might be a good place to start...
>
> For that kind of clean-up, I suggest that you have at least one commit
> per application,
> and probably also separate commits for test cases and changes in applications.
> For applications, it could further help to separate internal uses of
> of old-style "catch"
> inside modules and other uses that may be noticeable by outside callers.
>
> The purpose of all this is to help us (and everyone else who is interested) to
> review the changes, and to allow us include the safe changes first before all
> other changes have been cleared.
>
> And good commit messages, that explain why the change should be included
> always helps.
>
> --
> Björn Gustavsson, Erlang/OTP, Ericsson AB
--
Jayson Vantuyl
kagato@REDACTED
More information about the erlang-questions
mailing list