[erlang-patches] Serge's ei float patch
Steve Vinoski
vinoski@REDACTED
Mon May 31 16:42:30 CEST 2010
2010/5/31 Björn-Egil Dahlberg <egil@REDACTED>:
> This looks good but I have some concerns:
>
> - Aesthetics, coding convention: please refrain from C++ commenting.
Oops, I didn't even notice this in the original R12B-2 patch. Will fix.
> - 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.
Does the recv_challenge function in ei_connect.c need to verify that
DFLAG_NEW_FLOATS is set in the flags, treating it as an error if not
set (e.g. the same way DFLAG_EXTENDED_REFERENCES is handled)?
> - I think that "legacy" should include this as well. It should, unless
> someone can think of a reason not to include it.
OK, I'll work on that.
> 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.
Which tests would exercise the DFLAGS_NEW_FLOATS flag?
Note that the code already submitted in this patch is definitely
tested by the erl_interface ei_print suite, since tests there were
failing with just the original R12B-2 patch.
thanks,
--steve
More information about the erlang-patches
mailing list