On Thu, Aug 22, 2013 at 8:26 PM, Luke Wagner <l...@mozilla.com> wrote: > > Just so I understand, was your goal to remove the #include "RootingAPI.h" > from jsapi.h and was the problem you had the inability to forward-declare a > typedef, or was there more too it?
Not quite. The problem is that some code outside SpiderMonkey currently uses, for example, |HandleValue|, which requires including js/RootingAPI.h. Whereas if I change it to |Handle<Value>| I can get by with just forward declarations of |Handle| and |Value| and don't need to include any headers. So, outside of SpiderMonkey, the non-typedef version is definitely preferable. Nick _______________________________________________ dev-tech-js-engine-internals mailing list dev-tech-js-engine-internals@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-tech-js-engine-internals
