On Sun Jul 28 15:19:15 2024 +0000, Jacek Caban wrote:
It's not an object method. If anything, it would better fit `IWineJScript`. `dispex_builtin_props_to_json` also looks questionable. In any case, I think that it's not a good timing to send things like this.
Oh yeah, I forgot there's another interface now (I only had one on either jscript or mshtml side with everything to keep it simple).
BTW, what part of `dispex_builtin_props_to_json` is questionable? The fact it's generic? Should I hardcode the props list in each object instead, or? I tried to be thorough with tests to show that it's only the builtin props (and not functions, but props/accessors) that get placed in the JSON object, but if you have any ideas or think I missed something please let me know.
I made it generic to keep it simpler to avoid having to special case each object in turn.