shallowEqual returns true when dealing with NaN

[rickbeerendonk]

Fixes #101

[dashed]

You should also Object.is here: https://github.com/rickbeerendonk/fbjs/blob/83abbb77a41ec5feb8873b5427fddc80966e720c/src/core/shallowEqual.js#L24-L26

[rickbeerendonk]

@dashed That is a different issue, but you are correct in that it should also be handled.

[syranide]

I'm not sure how perf-critical this is, but it could be worth considering A === B || Object.is(A, B) if it is.

[zpao]

Thanks for doing this however… I'm going to close out because we already have this change internally and will be part of the next sync out. Sorry it's not already here - I need to be careful when it gets synced out for compatibility reasons

[rickbeerendonk]

@zpao Thank you for explaining, no worries.