The ForwardDeclare macros always use |class|, but many of those types are declared with |struct|, so MSVC generates tons of warnings. This is annoying. Maybe we should just change those structs to classes now? They are much more class-like anyway.
We could, certainly. If any of these are JSAPI-visible, that'll require embedders to make the same change to avoid warnings. I think we want to make the change eventually, so maybe we just bite the bullet and do it. Although, to be perfectly honest, the ForwardDeclare* macros look truly awful to me, and I wonder if there's not some better way to do this that avoids this madness entirely.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 823080
You need to log in before you can comment on or make changes to this bug.