https://codereview.chromium.org/488073002/diff/20001/src/ic.cc
File src/ic.cc (right):
https://codereview.chromium.org/488073002/diff/20001/src/ic.cc#newcode283
src/ic.cc:283: if (!it.HasProperty()) return false;
On 2014/08/20 14:53:17, Yang wrote:
More a question than comment: why do we check HasProperty here and not
IsFound?
We check for IsFound first elsewhere.
Dictionary mode objects always result in PROPERTY (IsFound() is true),
but I'll just add IsFound() here for consistency as well.
https://codereview.chromium.org/488073002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
