Implement message merging.

issuu / ocaml-protoc-plugin

ocaml-protoc-plugin

https://issuu.github.io/ocaml-protoc-plugin/

Other

48 stars 19 forks source link

Implement message merging. #60

Closed andersfugmann closed 6 months ago

andersfugmann commented 7 months ago

Per spec, if a single field message is received twice in a message the two messages should be merged recursively. The PR implement this functionality. Closes #57

Tests has been added and the test lib has been extended to verify parsing of multiple message fields as well as verifying full (slow) deserialization by injecting an out of order field at the head of the input buffer (reader)

While developing, a bug handling proto2 required message fields were also identified and fixed.

andersfugmann commented 7 months ago

This PR is based on #55 / #59 so it should be merged after these PR's

andersfugmann commented 7 months ago

Rebased after merging #55

andersfugmann commented 6 months ago

Closing as developement has moved to https://github.com/andersfugmann/ocaml-protoc-plugin