mirror of
https://chromium.googlesource.com/crosvm/crosvm
synced 2024-11-25 05:03:05 +00:00
bbb45a8e72
Some structs in vmm_vhost have `error` field and expose `set_failed` to allow callers to invalidate an instance. However, `set_failed` is not used in crosvm at all. In addition, the `error` field cannot be modified any private methods in vmm_vhost. This means that the callers can manage the error states entirely. So this CL removes the `error` fields and `set_failed` methods to make it clearer that the callers have responsibilities of error handling. BUG=none TEST=cargo test --all-features in vmm_vhost TEST=cargo test --all-features --target=x86_64-pc-windows-gnu in vmm_vhost Change-Id: Idf478790a0a75282a4c504456d93604f7369c015 Reviewed-on: https://chromium-review.googlesource.com/c/crosvm/crosvm/+/3863046 Reviewed-by: Noah Gold <nkgold@google.com> Auto-Submit: Keiichi Watanabe <keiichiw@chromium.org> Commit-Queue: Keiichi Watanabe <keiichiw@chromium.org> |
||
|---|---|---|
| .. | ||
| depot_tools@268d645853 | ||
| libslirp-rs | ||
| minigbm@ef852e007c | ||
| minijail@77c59db8ff | ||
| virglrenderer@4770f706db | ||
| vmm_vhost | ||