Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

capnp: add Canonicalize function #92

Closed
zombiezen opened this issue Aug 13, 2017 · 0 comments
Closed

capnp: add Canonicalize function #92

zombiezen opened this issue Aug 13, 2017 · 0 comments
Assignees
@zombiezen
Labels
enhancement
Milestone
2.17

Comments

@zombiezen
Copy link
Contributor

(This has been on my to-do list forever, but I guess I forgot to open an issue. Oops.)

The capnp package should include a Canonicalize function which implements the canonicalization algorithm described by the spec.
@zombiezen zombiezen self-assigned this Aug 13, 2017
@zombiezen zombiezen modified the milestone: 2.17 Aug 13, 2017
zombiezen added a commit that referenced this issue Aug 14, 2017
@zombiezen
capnp: avoid writing nulls for zero-sized structs
2f9e2c3 
This doesn't come up much in practice unless you are canonicalizing a
struct.  I'm copying what the C++ implementation does based on
experimentation with the canonicalize() routine.

Updates #92
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zombiezen zombiezen

Labels
enhancement
Projects
None yet
Milestone
2.17
Development

No branches or pull requests
1 participant
@zombiezen