LGTM, maybe with a better name?
http://codereview.chromium.org/7912/diff/1/2 File include/v8.h (right): http://codereview.chromium.org/7912/diff/1/2#newcode1025 Line 1025: Local<Array> GetProperties(); Should this have a comment that describes what this function does (like the fact that you only get enumerable properties back)? I think it should be renamed to GetPropertyKeys, so it's clear you're not getting the values back. http://codereview.chromium.org/7912 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
