Commit ab53d0ad authored Mar 03, 2023 by Charisee Committed by Charisee Chiw Mar 13, 2023

Change request for Rust v1.68.0

error: this `impl` can be derived
   --> frameworks/native/libs/binder/rust/src/binder.rs:113:1
    |
113 | / impl Default for Stability {
114 | |     fn default() -> Self {
115 | |         Stability::Local
116 | |     }
117 | | }
    | |_^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls
    = note: `-D clippy::derivable-impls` implied by `-D warnings`
    = help: remove the manual implementation...
help: ...and instead derive it...
    |
104 | #[derive(Default)]
    |
help: ...and mark the default variant
    |
107 ~     #[default]
108 ~     Local,
    |

bug: 270756468
Test: test_compiler.py
Change-Id: Iafe539357b050069babf9a910dc3877a9bf2c75e

parent 19ce0cba

libs/binder/rust/src/binder.rs

+2 −7

Original line number	Diff line number	Diff line
		@@ -100,22 +100,17 @@ where
		/// An interface can promise to be a stable vendor interface ([`Vintf`]), or
		/// makes no stability guarantees ([`Local`]). [`Local`] is
		/// currently the default stability.
		#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord)]
		#[derive(Debug, Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Default)]
		pub enum Stability {
		/// Default stability, visible to other modules in the same compilation
		/// context (e.g. modules on system.img)
		#[default]
		Local,

		/// A Vendor Interface Object, which promises to be stable
		Vintf,
		}

		impl Default for Stability {
		fn default() -> Self {
		Stability::Local
		}
		}

		impl From<Stability> for i32 {
		fn from(stability: Stability) -> i32 {
		use Stability::*;