Remove deprecated method impls for std Error

ash/src/entry.rs

@@ -50,15 +50,7 @@ impl fmt::Display for LoadingError {
     }
 }
 
-impl Error for LoadingError {
+impl Error for LoadingError {}
-    fn description(&self) -> &str {
-        "LoadingError"
-    }
-
-    fn cause(&self) -> Option<&Error> {
-        None
-    }
-}
 
 #[derive(Debug)]
 pub enum InstanceError {
@@ -72,18 +64,7 @@ impl fmt::Display for InstanceError {
     }
 }
 
-impl Error for InstanceError {
+impl Error for InstanceError {}
-    fn description(&self) -> &str {
-        "InstanceError"
-    }
-
-    fn cause(&self) -> Option<&Error> {
-        if let &InstanceError::VkError(ref err) = self {
-            return err.source();
-        }
-        None
-    }
-}
 
 #[allow(non_camel_case_types)]
 pub trait EntryV1_0 {

ash/src/vk.rs

@@ -41314,42 +41314,7 @@ impl Result {
     #[doc = "A requested pool allocation has failed due to fragmentation of the pool\'s memory"]
     pub const ERROR_FRAGMENTED_POOL: Self = Result(-12);
 }
-impl ::std::error::Error for Result {
+impl std::error::Error for Result {}
-    fn description(&self) -> &str {
-        let name = match *self {
-            Result::SUCCESS => Some("Command completed successfully"),
-            Result::NOT_READY => Some("A fence or query has not yet completed"),
-            Result::TIMEOUT => Some("A wait operation has not completed in the specified time"),
-            Result::EVENT_SET => Some("An event is signaled"),
-            Result::EVENT_RESET => Some("An event is unsignaled"),
-            Result::INCOMPLETE => Some("A return array was too small for the result"),
-            Result::ERROR_OUT_OF_HOST_MEMORY => Some("A host memory allocation has failed"),
-            Result::ERROR_OUT_OF_DEVICE_MEMORY => Some("A device memory allocation has failed"),
-            Result::ERROR_INITIALIZATION_FAILED => Some("Initialization of a object has failed"),
-            Result::ERROR_DEVICE_LOST => {
-                Some("The logical device has been lost. See <<devsandqueues-lost-device>>")
-            }
-            Result::ERROR_MEMORY_MAP_FAILED => Some("Mapping of a memory object has failed"),
-            Result::ERROR_LAYER_NOT_PRESENT => Some("Layer specified does not exist"),
-            Result::ERROR_EXTENSION_NOT_PRESENT => Some("Extension specified does not exist"),
-            Result::ERROR_FEATURE_NOT_PRESENT => {
-                Some("Requested feature is not available on this device")
-            }
-            Result::ERROR_INCOMPATIBLE_DRIVER => Some("Unable to find a Vulkan driver"),
-            Result::ERROR_TOO_MANY_OBJECTS => {
-                Some("Too many objects of the type have already been created")
-            }
-            Result::ERROR_FORMAT_NOT_SUPPORTED => {
-                Some("Requested format is not supported on this device")
-            }
-            Result::ERROR_FRAGMENTED_POOL => Some(
-                "A requested pool allocation has failed due to fragmentation of the pool\'s memory",
-            ),
-            _ => None,
-        };
-        name.unwrap_or("unknown error")
-    }
-}
 impl fmt::Display for Result {
     fn fmt(&self, fmt: &mut fmt::Formatter) -> fmt::Result {
         let name = match *self {

ash/tests/display.rs

@@ -19,12 +19,3 @@ fn display_enum() {
 fn display_result() {
     assert_eq!(vk::Result::SUCCESS.to_string(), "SUCCESS");
 }
-
-#[test]
-fn error_result_description() {
-    use std::error::Error;
-    assert_eq!(
-        vk::Result::SUCCESS.description(),
-        "Command completed successfully"
-    );
-}

generator/src/lib.rs

@@ -15,9 +15,9 @@ use itertools::Itertools;
 use proc_macro2::{Literal, Term};
 use quote::Tokens;
 use std::collections::{BTreeMap, HashMap, HashSet};
+use std::fmt::Display;
 use std::path::Path;
 use syn::Ident;
-use std::fmt::Display;
 
 pub trait ExtensionExt {}
 #[derive(Copy, Clone, Debug)]
@@ -88,7 +88,10 @@ named!(cfloat<&str, f32>,
 );
 
 fn khronos_link<S: Display>(name: &S) -> Literal {
-    Literal::string(&format!("<https://www.khronos.org/registry/vulkan/specs/1.1-extensions/man/html/{name}.html>", name=name))
+    Literal::string(&format!(
+        "<https://www.khronos.org/registry/vulkan/specs/1.1-extensions/man/html/{name}.html>",
+        name = name
+    ))
 }
 
 pub fn define_handle_macro() -> Tokens {
@@ -1210,23 +1213,6 @@ pub fn generate_enum<'a>(
 }
 
 pub fn generate_result(ident: Ident, _enum: &vkxml::Enumeration) -> Tokens {
-    let description_notation = _enum.elements.iter().filter_map(|elem| {
-        let (variant_name, notation) = match *elem {
-            vkxml::EnumerationElement::Enum(ref constant) => (
-                constant.name.as_str(),
-                constant.notation.as_ref().map(|s| s.as_str()).unwrap_or(""),
-            ),
-            _ => {
-                return None;
-            }
-        };
-
-        let variant_ident = variant_ident(&_enum.name, variant_name);
-        Some(quote! {
-            #ident::#variant_ident => Some(#notation)
-        })
-    });
-
     let display_notation = _enum.elements.iter().filter_map(|elem| {
         let variant_name = match *elem {
             vkxml::EnumerationElement::Enum(ref constant) => constant.name.as_str(),
@@ -1242,15 +1228,8 @@ pub fn generate_result(ident: Ident, _enum: &vkxml::Enumeration) -> Tokens {
     });
 
     quote! {
-        impl ::std::error::Error for #ident {
+        impl std::error::Error for #ident {}
-            fn description(&self) -> &str {
+
-                let name = match *self {
-                    #(#description_notation),*,
-                    _ => None,
-                };
-                name.unwrap_or("unknown error")
-            }
-        }
         impl fmt::Display for #ident {
             fn fmt(&self, fmt: &mut fmt::Formatter) -> fmt::Result {
                 let name = match *self {