On 2015/07/14 at 04:54:00, jarin wrote:
lgtm.

Even though it might be better to put the 'once' variable on the Shell class
(the Chrome build likes to complain about static initialization).

Should be OK: OnceType is an alias for AtomicWord. That being said, if it had a
constructor it would be problematic; GCC adds guards for threadsafe function
statics, but I'm not sure about other compilers pre-C++11.

Anyway, I've moved it out to make it clearer.

https://codereview.chromium.org/1230403003/

--
--
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.

Reply via email to