crates/nonstick: src/libpam/memory.rs comparison

Banish al(most al)l use of `i32` in favor of `c_int`.

comparison

equal deleted inserted replaced

-:42f747774d94
+:fb8b547b36b7
 /// Like the num_enum crate, but with no dependency on `syn`.
 macro_rules! num_enum {
 (
 $(#[$m:meta])*
-$viz:vis enum $name:ident($repr:ty) {
+$viz:vis enum $name:ident {
 $(
 $(#[$im:meta])*
 $item_name:ident = $item_value:path,
 )*
 }
 ) => {
+// This is the one place where we depend upon c_int being i32.
+// Ideally, we would be able to say `repr(c_int)` but we can't.
 $(#[$m])*
 #[derive(Clone, Copy, Debug, Eq, PartialEq)]
-#[repr($repr)]
+#[repr(i32)]
 $viz enum $name {
 $(
 $(#[$im])*
 $item_name = $item_value,
 )*
 }
-impl TryFrom<$repr> for $name {
+impl TryFrom<c_int> for $name {
 type Error = crate::constants::ErrorCode;
 #[allow(unused_doc_comments)]
-fn try_from(value: $repr) -> crate::constants::Result<$name> {
+fn try_from(value: c_int) -> crate::constants::Result<$name> {
 match value {
 $(
 $(#[$im])*
 $item_value => Ok(Self::$item_name),
 )*
 _ => Err(crate::constants::ErrorCode::BAD_CONST),
 }
 }
 }
-impl From<$name> for $repr {
+impl From<$name> for c_int {
-fn from(value: $name) -> $repr {
+fn from(value: $name) -> c_int {
-value as $repr
+value as c_int
 }
 }
 }
 }

Mercurial > crates > nonstick