Skip to content
This repository was archived by the owner on Dec 16, 2020. It is now read-only.

Remove exception use in the WebAssembly VM.#86

Merged
jplevyak merged 1 commit intoenvoyproxy:masterfrom
jplevyak:wasm-noexcept
Jul 24, 2019
Merged

Remove exception use in the WebAssembly VM.#86
jplevyak merged 1 commit intoenvoyproxy:masterfrom
jplevyak:wasm-noexcept

Conversation

@jplevyak
Copy link
Contributor

Signed-off-by: John Plevyak jplevyak@gmail.com

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md

Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

Signed-off-by: John Plevyak <jplevyak@gmail.com>
@jplevyak jplevyak requested a review from PiotrSikora as a code owner July 22, 2019 22:10
@jplevyak jplevyak requested a review from duderino July 22, 2019 22:12
@jplevyak
Copy link
Contributor Author

See #9 for discussion.

@duderino
Copy link
Contributor

Thanks for doing this @jplevyak. Note to self: once it's merged, the wat/wasm files introduced in #72 need to be rebuilt

@duderino
Copy link
Contributor

/lgtm

@mandarjog
Copy link
Contributor

@duderino please also indicate a GitHub “lgtm” not the bot stuff.
@jplevyak this is ready for you to merge.

@jplevyak jplevyak merged commit be089a7 into envoyproxy:master Jul 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants