[erlang-patches] Serge's ei float patch
Björn-Egil Dahlberg
egil@REDACTED
Mon May 31 12:08:08 CEST 2010
This looks good but I have some concerns:
- Aesthetics, coding convention: please refrain from C++ commenting.
- Distribution flags, the DFLAG_NEW_FLOATS option should be enabled in
lib/erl_interface/src/connect/ei_connect.c: static int
send_name_or_challenge.
See lib/erl_interface/src/connect/ei_connect_int.h and
lib/kernel/src/dist.hrl for some details.
- I think that "legacy" should include this as well. It should,
unless someone can think of a reason not to include it.
Anyway, with DFLAGS_NEW_FLOATS flag enabled the code would be tested as
well which is a necessity in order for it to be included in the otp relase.
Regards,
Björn-Egil
Erlang/OTP
On 2010-05-30 02:44, Steve Vinoski wrote:
> This is a submission of Serge Aleynikov's ei patch that implements
> binary encoding for floats:
>
> git fetch git://github.com/vinoski/otp.git serge_new_float_ext
>
> Note there are two commits on this branch: 3b3bae9 and a1ae64e. One
> just fixes a typo in the ei_decode_ei_term documentation -- I kept
> that commit separate since it's independent. The other commit is
> Serge's original R12B-2 patch adjusted as needed for the current code
> base and with the addition of fixing all areas across Erlang/OTP
> affected by the change. The only exception is that I didn't change
> erl_marshal.c because it's "legacy" so I'm not sure it has to handle
> this new encoding or not.
>
> If anyone wants to see the exact changes for each commit, the links to
> the two commits are:
>
> http://github.com/vinoski/otp/commit/3b3bae9397b405ce5e782d684ed88e0442eab65e
> http://github.com/vinoski/otp/commit/a1ae64e5c30f89a2c6b9d1b83777a72235ffc003
>
> --steve
>
> ________________________________________________________________
> erlang-patches (at) erlang.org mailing list.
> See http://www.erlang.org/faq.html
> To unsubscribe; mailto:erlang-patches-unsubscribe@REDACTED
>
More information about the erlang-patches
mailing list