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

Make map serialization/deserialization utils public #125

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Make map serialization/deserialization utils public #125

wants to merge 2 commits into from

Commits on Sep 16, 2021

  1. Avoid taking Vec as argument in hostcalls::utils 

    We don't need ownership, so slice should be enough.

Note: Public-facing functions still do take Vecs as arguments, to avoid
breaking changes. They could also be changed to take slices in the
future.

Signed-off-by: Michał Krasnoborski <[email protected]>
    @krdln
    krdln committed Sep 16, 2021
    Configuration menu
    Copy the full SHA
    1c885e4 View commit details
    Browse the repository at this point in the history

  2. Make map serialization/deserialization utils public 

    These functions are helpful as "serialized map" format is used in some
other places, eg. in envoy `get_property(["metadata", "filter_metadata",
"xxx"])` may return a map in such a format.

Signed-off-by: Michał Krasnoborski <[email protected]>
    @krdln
    krdln committed Sep 16, 2021
    Configuration menu
    Copy the full SHA
    3c20993 View commit details
    Browse the repository at this point in the history