438339 - looking up null different in TT

Status: RESOLVED FIXED

(Tamarin Graveyard :: Tracing Virtual Machine, defect)

Description

[Tommy Reilly]

var o = {};
trace(o[null]);

In TC:
undefined

In TT:
ReferenceError: Error #1081: Property null not found on Object and there is no default value.
	at global$init()

Comment 1

[Steven Johnson]

also broken:

var o = [];
trace(o[null]);

and ditto for "undefined" for the key.

Comment 2

[Steven Johnson]

bug actually is that dynamic lookups weren't correct for keys of type null, undefined, boolean, or namespace. fix is in the works. obviously we need to add an acceptance test for this, I'll do one of those too.

Comment 3

[Steven Johnson]

Attachment: Patch

box2dynamevec was simply rejecting keys of type null/void/bool/ns; it should have converted them to interned strings.

Comment 4

[Steven Johnson]

Attachment: Acceptance test to verify this fix (passes now for both TC and TT)

Acceptance test, since existing sanities didn't catch this

Comment 5

[Steven Johnson]

pushed as changeset:   393:58fb13dc90c8