[erlang-patches] Serge's ei float patch

Björn-Egil Dahlberg egil@REDACTED
Tue Jun 1 15:00:49 CEST 2010


On 2010-06-01 09:06, Steve Vinoski wrote:
> I've pushed an amended commit to github that I believe addresses all
> your suggestions:
>
> git fetch git://github.com/vinoski/otp.git serge_new_float_ext
>
> Thanks to your feedback, I found a number of other places that needed
> attention as part of this patch. If you could take another careful
> look at the new changes, that would be good.

Oops. I forgot something. (And yet again erl_interface is faced with a 
problem). ei_get_type exposes the internal representation (external 
format) of types. This is a problem. Earlier implementation had its own 
types ... which seemingly was a bad idea ...

Angels are crying for suggesting this but, what do you think of letting 
get_type return ERL_FLOAT_EXT for both ERL_FLOAT_EXT and NEW_FLOAT_EXT?

My aim is to remove the necessity for changing original source code 
dependent on erl_interface, if that is at all possible.

What do you think?

Hmm, actually this is somewhat more serious then I first thought. I need 
to talk it over with my team.

> I reinstated these tests and modified them to match the style of other
> tests in the same suite. I also found that match_float could
> incorrectly fail when dealing with the float value 0.0 so I augmented
> it with another function clause that allows for simple equality.

Great!

// Björn-Egil


More information about the erlang-patches mailing list