search

jgauffin / Griffin.Framework

Application framework for Business Applications
http://griffinframework.net
168 stars 62 forks source link

fix MicroMessageEncoder bug #11

Closed chummyhe89 closed 8 years ago

chummyhe89 commented 9 years ago

FixedHeaderLength is defined by sizeof(ushort) + sizeof (byte) + sizeof (int) + sizeof (byte); the header length field is included in _headerSize,so can't +2

jgauffin commented 9 years ago

Looks like you are correct. However, the unit tests says otherwise. they work with the current setup (look at the CodecTests.cs).

jgauffin commented 9 years ago

i.e. tell me why and I'll adjust them accordingly and accept your request.