Bug 1530028 - Remove now-unnecessary NsresultExt trait, r=froydnj

Differential Revision: https://phabricator.services.mozilla.com/D20880

--HG--
extra : moz-landing-system : lando

storage/variant/src/lib.rs

@@ -8,7 +8,7 @@ extern crate nsstring;
 extern crate xpcom;
 
 use libc::{c_double, int64_t, uint16_t};
-use nserror::{nsresult, NsresultExt, NS_OK};
+use nserror::{nsresult, NS_OK};
 use nsstring::{nsACString, nsAString, nsCString, nsString};
 use xpcom::{getter_addrefs, interfaces::nsIVariant, RefPtr};
 

toolkit/components/kvstore/src/error.rs

@@ -4,7 +4,7 @@
 
 use libc::uint16_t;
 use nserror::{
-    nsresult, NsresultExt, NS_ERROR_FAILURE, NS_ERROR_NOT_IMPLEMENTED, NS_ERROR_NO_INTERFACE,
+    nsresult, NS_ERROR_FAILURE, NS_ERROR_NOT_IMPLEMENTED, NS_ERROR_NO_INTERFACE,
     NS_ERROR_NULL_POINTER, NS_ERROR_UNEXPECTED,
 };
 use nsstring::nsCString;

toolkit/components/kvstore/src/owned_value.rs

@@ -4,7 +4,6 @@
 
 use error::KeyValueError;
 use libc::int32_t;
-use nserror::NsresultExt;
 use nsstring::nsString;
 use rkv::OwnedValue;
 use storage_variant::{

toolkit/components/kvstore/src/task.rs

@@ -7,7 +7,7 @@ extern crate xpcom;
 use crossbeam_utils::atomic::AtomicCell;
 use error::KeyValueError;
 use moz_task::Task;
-use nserror::{nsresult, NsresultExt, NS_ERROR_FAILURE};
+use nserror::{nsresult, NS_ERROR_FAILURE};
 use nsstring::nsCString;
 use owned_value::owned_to_variant;
 use rkv::{Manager, OwnedValue, Rkv, SingleStore, StoreError, StoreOptions, Value};

xpcom/idl-parser/xpidl/rust.py

@@ -242,7 +242,7 @@ infallible_impl_tmpl = """\
 pub unsafe fn %(name)s(&self) -> %(realtype)s {
     let mut result = <%(realtype)s as ::std::default::Default>::default();
     let _rv = ((*self.vtable).%(name)s)(self, &mut result);
-    debug_assert!(::nserror::NsresultExt::succeeded(_rv));
+    debug_assert!(_rv.suceeded());
     result
 }
 """

xpcom/rust/gtest/xpcom/test.rs

@@ -13,7 +13,7 @@ use std::os::raw::c_char;
 use std::ptr;
 use std::ffi::{CStr, CString};
 use xpcom::interfaces;
-use nserror::{NsresultExt, nsresult, NS_OK};
+use nserror::{nsresult, NS_OK};
 
 #[no_mangle]
 pub unsafe extern fn Rust_ObserveFromRust() -> *const interfaces::nsIObserverService {

xpcom/rust/moz_task/src/lib.rs

@@ -12,7 +12,7 @@ extern crate nsstring;
 #[macro_use]
 extern crate xpcom;
 
-use nserror::{nsresult, NsresultExt, NS_OK};
+use nserror::{nsresult, NS_OK};
 use nsstring::{nsACString, nsCString};
 use std::{
     ptr,

xpcom/rust/nserror/src/lib.rs

@@ -7,30 +7,19 @@ use nsstring::{nsCString, nsACString};
 /// as the C++ equivalent.
 #[repr(transparent)]
 #[allow(non_camel_case_types)]
-#[derive(Clone, Copy, Debug)]
+#[derive(Clone, Copy, Debug, Eq, PartialEq, Ord, PartialOrd, Hash)]
 pub struct nsresult(pub u32);
 
-/// An extension trait that adds methods to `nsresult` types.
-pub trait NsresultExt {
-    fn failed(self) -> bool;
-    fn succeeded(self) -> bool;
-    fn to_result(self) -> Result<(), nsresult>;
-
-    /// Get a printable name for the nsresult error code. This function returns
-    /// a nsCString<'static>, which implements `Display`.
-    fn error_name(self) -> nsCString;
-}
-
-impl NsresultExt for nsresult {
-    fn failed(self) -> bool {
+impl nsresult {
+    pub fn failed(self) -> bool {
         (self.0 >> 31) != 0
     }
 
-    fn succeeded(self) -> bool {
+    pub fn succeeded(self) -> bool {
         !self.failed()
     }
 
-    fn to_result(self) -> Result<(), nsresult> {
+    pub fn to_result(self) -> Result<(), nsresult> {
         if self.failed() {
             Err(self)
         } else {
@@ -38,7 +27,9 @@ impl NsresultExt for nsresult {
         }
     }
 
-    fn error_name(self) -> nsCString {
+    /// Get a printable name for the nsresult error code. This function returns
+    /// a nsCString<'static>, which implements `Display`.
+    pub fn error_name(self) -> nsCString {
         let mut cstr = nsCString::new();
         unsafe {
             Gecko_GetErrorName(self, &mut *cstr);

xpcom/rust/xpcom/src/base.rs

@@ -8,7 +8,6 @@ use {
     GetterAddrefs
 };
 use interfaces::nsISupports;
-use nserror::NsresultExt;
 
 #[repr(C)]
 #[derive(Copy, Clone, Eq, PartialEq)]

xpcom/rust/xpcom/src/reexports.rs

@@ -13,6 +13,6 @@ pub extern crate libc;
 
 pub use nsstring::{nsACString, nsAString, nsCString, nsString};
 
-pub use nserror::{nsresult, NsresultExt, NS_ERROR_NO_INTERFACE, NS_OK};
+pub use nserror::{nsresult, NS_ERROR_NO_INTERFACE, NS_OK};
 
 pub use std::ops::Deref;

xpcom/rust/xpcom/src/refptr.rs

@@ -9,7 +9,7 @@ use std::marker::PhantomData;
 use std::cell::Cell;
 use std::sync::atomic::{self, AtomicUsize, Ordering};
 
-use nserror::{NsresultExt, nsresult, NS_OK};
+use nserror::{nsresult, NS_OK};
 
 use libc;
 

xpcom/rust/xpcom/src/statics.rs

@@ -4,7 +4,6 @@
 
 use std::ffi::CStr;
 use std::ptr;
-use nserror::NsresultExt;
 use {
     RefPtr,
     GetterAddrefs,

xpcom/rust/xpcom/xpcom_macros/src/lib.rs

@@ -619,9 +619,7 @@ fn xpcom(init: DeriveInput) -> Result<Tokens, Box<Error>> {
             {
                 let mut ga = ::xpcom::GetterAddrefs::<T>::new();
                 unsafe {
-                    if ::xpcom::reexports::NsresultExt::succeeded(
-                        self.QueryInterface(&T::IID, ga.void_ptr()),
-                    ) {
+                    if self.QueryInterface(&T::IID, ga.void_ptr()).succeeded() {
                         ga.refptr()
                     } else {
                         None