Skip to content
This repository has been archived by the owner on Feb 8, 2022. It is now read-only.

Proto2 required fields are not properly serialized #86

Closed
jhugard opened this issue Apr 18, 2018 · 1 comment
Closed

Proto2 required fields are not properly serialized #86

jhugard opened this issue Apr 18, 2018 · 1 comment

Comments

@jhugard
Copy link
Collaborator

jhugard commented Apr 18, 2018

Proto2 mandates that required fields always appear in the serialized output, even when they match the default value. Not doing so will cause an error when interoperating with other implementations.
@jhugard
Copy link
Collaborator Author

jhugard commented Apr 19, 2018

Duplicate of #68, which was fixed with 717ae69.
(I had uncommitted files on one of my dev boxes, causing some confusion. These were already committed from a different PC.)

@jhugard jhugard closed this as completed Apr 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jhugard