Skip to content

Commit

Permalink
make default_executor non pub
Browse files Browse the repository at this point in the history
Signed-off-by: jiaxiao zhou <jiazho@microsoft.com>
  • Loading branch information
Mossaka committed Aug 4, 2023
1 parent 4bd6862 commit b9ab440
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion crates/containerd-shim-wasmtime/src/executors/container.rs
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ use std::{fs::OpenOptions, path::PathBuf};

#[derive(Default)]
pub struct LinuxContainerExecutor {
pub default_executor: DefaultExecutor,
default_executor: DefaultExecutor,
}

impl Executor for LinuxContainerExecutor {
Expand Down
4 changes: 1 addition & 3 deletions crates/containerd-shim-wasmtime/src/instance.rs
Original file line number Diff line number Diff line change
Expand Up @@ -227,9 +227,7 @@ impl Wasi {
stdout,
stderr,
});
let default_executor = Box::new(LinuxContainerExecutor {
..Default::default()
});
let default_executor = Box::new(LinuxContainerExecutor::default());

Check failure on line 230 in crates/containerd-shim-wasmtime/src/instance.rs

View workflow job for this annotation

GitHub Actions / fmt

`Box::new(_)` of default value
let container = ContainerBuilder::new(self.id.clone(), syscall.as_ref())
.with_executor(vec![default_executor, wasmtime_executor])?
.with_root_path(self.rootdir.clone())?
Expand Down

0 comments on commit b9ab440

Please sign in to comment.