NET5_0 must enable all NETCOREAPP2_1 optimization

Agreed. But sadly, I just tested it and net5.0 target no longer defines NETCOREAPP as a preprocessor symbol. :( I'll research what's the best way to handle this and get back to you.

Oh never mind. NETCOREAPP is defined for net5.0. Here's the spec BTW.

@neuecc I believe this PR should be changed to target develop instead of master since it adds public API.

@neuecc this PR is just a little too new for Hosted agents on Azure Pipelines, which have not yet been upgraded to VS 16.9. It only has 16.8 which does not support .NET 5.0.
We can sit tight for a week or so until Microsoft updates the agents.
Alternatively, if you don't mind removing the VSIX project that includes the analyzers, we can switch from msbuild.exe to dotnet build on Azure Pipelines, in which case we can make .NET 5.0 work immediately. VSIX-deployed analyzers are something I think we can cut support for. Many people prefer the nuget deployed analyzers anyway. Thoughts?

thank you for check it.
ok to change to develop.

VSIX-deployed analyzers are something I think we can cut support for. Many people prefer the nuget deployed analyzers anyway. Thoughts?

Yes, I wish I could, but vsix is an easy alternative because it requires a hack to get Analyzer to work in Unity csproj.
but I think it can be removed.

Since the half value is ushort, it is better to serialize the value as it is without converting it to a float.

@crocuis While your suggested change results in a smaller msgpack stream, it lies about the type to do it. A message pack reader that is simply reading values as they are typed and displaying an object tree would then totally misrepresent the value.
IMO we should keep with @neuecc's original plan until the msgpack encoding spec itself is updated to account for a 16-bit floating point number, and we can switch to that.