James Turner

 

More curious...  

Potentially, yeah... I'll need to dig into this more tomorrow but that is a crazy bug if that is the case.

 An even more simplified example causing the problem - it definitely seems to be an upstream bug that I inadvertently triggered.

It gets weirder, both properties don't need to have `set` either - just one "fixes" this. Guessing it is definitely impacting the ordering then.   

Opened up a bug report for this on protobuf-net: https://github.com/protobuf-net/protobuf-net/issues/964

From Marc Gravell: > The _immediate_ problem here is that it is inferring the member order as a tuple-like type - meaning: when it finds no attributes (in `RuntimeTypeModel.Add`), it...

I realised after posting this that `SendHeader` on `SocketFrameHandler` does do synchronous sends so it isn't entirely pointless to set, now just rather potentially inconsistent behaviour between the socket writes...

Agreed - not a fan of the build information being added via Dependabot and manually do the package update to avoid that being added to the git history. My recent...

> That said the VS Code is all OSS. If you want syntax highlighting "that good" you might want to look at just using the same code Visual Studio uses...