lgtm
This does reduce boilerplate, but honestly I'm not sure whether it's really
an
improvement. This is way deep in 'bikeshed' territory, so please decide as
you
see fit.
My concern is this: When you recently used one of these asserts, I spent
quite a
bit of time reading through the code review to figure out /how/ this
actually
worked. But at least on the logic was plain C++, so I could work my way
through
it. I'm fairly certain that - in this newer version, with the macro soup
and all
- I would have had to spent a lot more time on figuring it out.
A related but minor point is that I'm certain tools (code searching,
refactoring, etc.) will have a hard time with this. (It's a minor point
since we
don't use a whole lot of those.)
https://codereview.chromium.org/396923002/
--
--
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.
