Skip to content

Commit

Permalink
Fix a compiler warning.
Browse files Browse the repository at this point in the history
Fix the following warning from Rust 1.35:

warning: cannot borrow `*self` as mutable because it is also borrowed as immutable
   --> wasmtime-runtime/src/instance.rs:473:25
    |
465 |         } else if let Some(start_export) = self.module.exports.get("_start") {
    |                                            ----------- immutable borrow occurs here
...
473 |                         self.invoke_function(*func_index)
    |                         ^^^^                 ----------- immutable borrow later used here
    |                         |
    |                         mutable borrow occurs here
    |
    = note: #[warn(mutable_borrow_reservation_conflict)] on by default
    = warning: this borrowing pattern was not meant to be accepted, and may become a hard error in the future
    = note: for more information, see issue #59159 <rust-lang/rust#59159>
  • Loading branch information
sunfishcode authored and tschneidereit committed Jun 25, 2019
1 parent 76e9e81 commit f42110c
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions wasmtime-runtime/src/instance.rs
Original file line number Diff line number Diff line change
Expand Up @@ -466,11 +466,11 @@ impl Instance {
// As a compatibility measure, if the module doesn't have a start
// function but does have a _start function exported, call that.
match start_export {
wasmtime_environ::Export::Function(func_index) => {
let sig = &self.module.signatures[self.module.functions[*func_index]];
&wasmtime_environ::Export::Function(func_index) => {
let sig = &self.module.signatures[self.module.functions[func_index]];
// No wasm params or returns; just the vmctx param.
if sig.params.len() == 1 && sig.returns.is_empty() {
self.invoke_function(*func_index)
self.invoke_function(func_index)
} else {
Ok(())
}
Expand All @@ -482,11 +482,11 @@ impl Instance {
// start function or a _start function exported, but does have a main
// function exported, call that.
match main_export {
wasmtime_environ::Export::Function(func_index) => {
let sig = &self.module.signatures[self.module.functions[*func_index]];
&wasmtime_environ::Export::Function(func_index) => {
let sig = &self.module.signatures[self.module.functions[func_index]];
// No wasm params or returns; just the vmctx param.
if sig.params.len() == 1 && sig.returns.is_empty() {
self.invoke_function(*func_index)
self.invoke_function(func_index)
} else {
Ok(())
}
Expand Down

0 comments on commit f42110c

Please sign in to comment.