>> http://codereview.chromium.org/8536042/diff/1/src/api.h >> File src/api.h (right): > >> http://codereview.chromium.org/8536042/diff/1/src/api.h#newcode164 >> src/api.h:164: class ExtensionStates : private HashMap { >> Use delegation instead of implementation inheritance. > > All other uses of HashMap in V8 use implementation inheritance.
This is not true, BTW, but that's not the point. I think we should try to improve the code where we can and especially when it's easy to do so. -- Vitaly -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
