[erlang-patches] revised patch for binary_to_term/2 and clarifying comment

Björn Gustavsson bgustavsson@REDACTED
Fri Jan 8 14:39:18 CET 2010


Thanks!

On Fri, Jan 8, 2010 at 10:06 AM, Jayson Vantuyl <kagato@REDACTED> wrote:
> I've pushed a branch with the following changes:
>
> 1.  I rebased the latest safety / binary_to_term/2 patch off of the now current ccase/r13b04_dev
> 2.  Otherwise, the first changeset is identical to the already submitted one

I made the following the following stylistic change to that commit
to match the style of the rest of the file:

diff --git a/erts/emulator/beam/external.c b/erts/emulator/beam/external.c
index e1c868b..088a551 100644
--- a/erts/emulator/beam/external.c
+++ b/erts/emulator/beam/external.c
@@ -2288,7 +2288,7 @@ dec_term_atom_common:
                    goto error;
                }
                if (edep && (edep->flags & ERTS_DIST_EXT_BTT_SAFE)) {
-                   if (!erts_find_export_entry(mod,name,arity))
+                   if (!erts_find_export_entry(mod, name, arity))
                        goto error;
                 }
                *objp = make_export(hp);

> 3.  I added an additional changeset with the comments we discussed (a warning that edep flags and mask shouldn't overlap)

The patch added two spaces at end of lines - I have eliminated them.

> I thought it deserved a separate commit, since it was really a separate change.  I would have made it a separate branch, but I didn't see the point, since it would have just conflicted with the addition of the BTT flag (thereby requiring an unnecessary merge).

Yes, I agree on both points.

The updated branch will be included in 'pu' when I'll push it later today.

-- 
Björn Gustavsson, Erlang/OTP, Ericsson AB


More information about the erlang-patches mailing list