BsonReader/Writer obsolete

[GarethPikPok]

Description

Using BsonReader and BsonWriter currently introduce the following warning:
warning CS0618: 'BsonWriter' is obsolete: 'BSON reading and writing has been moved to its own package. See https://www.nuget.org/packages/Newtonsoft.Json.Bson for more details.'

Why we need this

Bson is a much more compact and efficient file format. It is preferable for our project to use Bson.

Suggested solution

Either: A new package to handle Bson support OR extend this package to use the new Bson package.

[applejag]

Good note! Somewhat hoping this feature request would come later but it's interesting to pick up. Much smaller project than Newtonsoft.Json so it should be all fine.

Thanks for using the Newtonsoft.Json-for-Unity project!

[GarethPikPok]

It's likely that for now I'll continue to use this project and suppress the obsolete warnings :) Looking forward to having the Bson plugin! Thanks for your hard work :) Take it easy over Christmas though!

[applejag]

I think (haven't read through the source thoroughly yet) that Newtonsoft.Json.Bson actually don't use any DLR/Reflection.Emit code, so it would theoretically be all safe to just download the DLL from NuGet and put it into your Assets folder, even for AOT builds.

Adding Bson is just a matter of deploying the package to the registry, and preferably some CI in between for automated testing. You will see it pop up in the Unity Package Manager as available for download once it's deployed, then you can switch over to using the package instead.

And good Christmas spirit ;) 🎄 I want some time for adventofcode too. But soon^TM

[victor-kallai]

@jilleJr Hello, any news on this feature?

[applejag]

@victor-kallai No progress I'm sorry to say. It's quite far down in my priority list. Have not looked into how difficult it is, if it's on the easier side then I'll consider doing it sooner than later.

Can't promise anything though, I'm very obstructed by life at the moment.