Comment #11 on issue 4387 by [email protected]: Computed property names
don't work in nested literals
https://code.google.com/p/v8/issues/detail?id=4387
I'm not disagreeing with you. If you believe the third option is more
correct, by all means go for it. I'm happy to see this bug fixed.
I'm just saying that debugging this code was confusing because I
incorrectly believed IsBoilerplateProperty() was doing what its name
suggested. Just consider giving the function a more apt name or put a
comment above it:
// IsBoilerplateProperty() only makes sense on properties whose
// index is less than the first property with a computed name
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
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.
