summary refs log tree commit diff
path: root/crosvm_plugin
diff options
context:
space:
mode:
authorDmitry Torokhov <dtor@chromium.org>2018-03-08 11:21:50 -0800
committerchrome-bot <chrome-bot@chromium.org>2018-03-08 19:28:40 -0800
commit7c2238bd15ec8b027c5508a05272e79b4bb44607 (patch)
tree92f0f2e6f8eb61894d10ed761890c438bec83734 /crosvm_plugin
parent609d0f041bb8bbf2674d2b2dcffde1d778a920ef (diff)
downloadcrosvm-7c2238bd15ec8b027c5508a05272e79b4bb44607.tar
crosvm-7c2238bd15ec8b027c5508a05272e79b4bb44607.tar.gz
crosvm-7c2238bd15ec8b027c5508a05272e79b4bb44607.tar.bz2
crosvm-7c2238bd15ec8b027c5508a05272e79b4bb44607.tar.lz
crosvm-7c2238bd15ec8b027c5508a05272e79b4bb44607.tar.xz
crosvm-7c2238bd15ec8b027c5508a05272e79b4bb44607.tar.zst
crosvm-7c2238bd15ec8b027c5508a05272e79b4bb44607.zip
crosvm_plugin: be more Rusty
Instead of writing "if let Some(x) = ret.ok()" let's write more
idiomatic "if let Ok(x) = ret".

BUG=none
TEST=sudo cargo test --features plugin

Change-Id: Id8c4d0fd2a05783759b5699b65d2333b2a1a6776
Signed-off-by: Dmitry Torokhov <dtor@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/956229
Reviewed-by: Dylan Reid <dgreid@chromium.org>
Reviewed-by: Zach Reizner <zachr@chromium.org>
Diffstat (limited to 'crosvm_plugin')
-rw-r--r--crosvm_plugin/src/lib.rs8
1 files changed, 4 insertions, 4 deletions
diff --git a/crosvm_plugin/src/lib.rs b/crosvm_plugin/src/lib.rs
index 4bb763f..e5c9844 100644
--- a/crosvm_plugin/src/lib.rs
+++ b/crosvm_plugin/src/lib.rs
@@ -919,7 +919,7 @@ pub unsafe extern "C" fn crosvm_check_extension(self_: *mut crosvm,
     let self_ = &mut (*self_);
     let ret = self_.check_extension(extension);
 
-    if let Some(supported) = ret.ok() {
+    if let Ok(supported) = ret {
         *has_extension = supported;
     }
     to_crosvm_rc(ret)
@@ -936,7 +936,7 @@ fn crosvm_get_supported_cpuid(this: *mut crosvm,
     let cpuid_entries = from_raw_parts_mut(cpuid_entries, entry_count as usize);
     let ret = this.get_supported_cpuid(cpuid_entries);
 
-    if let Some(num) = ret.ok() {
+    if let Ok(num) = ret {
         *out_count = num as u32;
     }
     to_crosvm_rc(ret)
@@ -953,7 +953,7 @@ fn crosvm_get_emulated_cpuid(this: *mut crosvm,
     let cpuid_entries = from_raw_parts_mut(cpuid_entries, entry_count as usize);
     let ret = this.get_emulated_cpuid(cpuid_entries);
 
-    if let Some(num) = ret.ok() {
+    if let Ok(num) = ret {
         *out_count = num as u32;
     }
     to_crosvm_rc(ret)
@@ -1033,7 +1033,7 @@ pub unsafe extern "C" fn crosvm_get_vcpu(self_: *mut crosvm,
     let self_ = &mut (*self_);
     let ret = self_.get_vcpu(cpu_id);
 
-    if let Some(vcpu) = ret.ok() {
+    if let Ok(vcpu) = ret {
         *out = vcpu;
     }
     to_crosvm_rc(ret)