StrMap backing object inherits from Object.prototype and lookup uses in

[michaelficarra]

I imagine lookup will think that keys like "toString" and "hasOwnProperty" exist in the object, even though the functions that rely on Object.keys or iteration through for-in won't see these properties because they're non-enumerable. I haven't tested this, but I'm pretty sure I'm right just by looking at the implementation.

[garyb]

Yeah, I'm pretty sure you're right too.

[michaelficarra]

There's two possible solutions:

1. use a hasOwnProperty test instead of in
2. use Object.create(null) as the backing object

I'd probably decide which to do based on performance. I'm going to guess the first option is better optimised in most implementations right now, but tests should be done to confirm.

[garyb]

Interesting, I wasn't aware of option 2 - I just assumed we'd have to use hasOwnProperty. You get a truly empty prototype with Object.create(null) then?

[michaelficarra]

Yep.

[michaelficarra]

@garyb is this fixed now after cca72e0#diff-f398949aafcfd55358d80eec690ee116?

[garyb]

Yes! (Although it also broke it, #39 fixed the new bug though) Thanks for the reminder.