ctaggart / froto

Froto: F# Protocol Buffers
MIT License
146 stars 27 forks source link

Improve serializer performance by 25% #25

Closed jhugard closed 8 years ago

jhugard commented 8 years ago

In reference to code gen in #15, note the change to build the decoder ring in the constructor, rather than every time the message is deserialized. Also note the method name change from EncoderRing to Encode.

Would be even better to make the decoder ring a static... just thought of that.

ctaggart commented 8 years ago

I'd like to be able to generate/use F# records that represent the messages. A static decoder ring would probably help that. I like that you were able to combine to ZeroCopyBuffer.