Add MESSAGEPACK_FORCE_AOT preprocessor directive - #1701

Why not just set ENABLE_IL2CPP in the editor?

Why not just set ENABLE_IL2CPP in the editor?

I'd considered this, but had concerns about the impact it could have on other packages relying on ENABLE_IL2CPP if set globally in the Editor, so opted to target the functionality in MessagePack directly

Thank you for the changes @brwhelan-msft . For maintainability, at least where it is used often, I think it would be good to have this defined:

#if !(MESSAGEPACK_FORCE_AOT || ENABLE_IL2CPP)
#define DYNAMIC_GENERATION
#endif

And check for DYNAMIC_GENERATION instead.

at least where it is used often

I like the suggestion, but I'd go further and say it should be used consistently at the top of any file that uses any of these Defines symbols. Otherwise it seems likely we'll regress in later code changes.

Great idea - I've just pushed an update which switches to defining DYNAMIC_GENERATION at the top of any files affected, and using it throughout