Mark all public struct/enum as non_exhaustive

All the public struct/enum in nmstate might add members/type, hence
marking them as non_exhaustive which will prevent API user from
in-compatibility usage.

This allows us to only preserve API compatibility when adding new member
to struct or enum.

Please refer to https://doc.rust-lang.org/reference/attributes/type_system.html
for more detail.

Signed-off-by: Gris Ge <[email protected]>

Author: cathay4t

examples/new_rule.rs

@@ -11,25 +11,26 @@ fn main() {
     let _port_number = socket.bind_auto().unwrap().port_number();
     socket.connect(&SocketAddr::new(0, 0)).unwrap();
 
+    let mut rule_msg_hdr = RuleHeader::default();
+    rule_msg_hdr.family = AF_INET as u8;
+    rule_msg_hdr.table = RT_TABLE_DEFAULT;
+    rule_msg_hdr.action = FR_ACT_TO_TBL;
+
+    let mut rule_msg = RuleMessage::default();
+    rule_msg.header = rule_msg_hdr;
+    rule_msg.nlas = vec![
+        rule::Nla::Table(254),
+        rule::Nla::SuppressPrefixLen(4294967295),
+        rule::Nla::Priority(1000),
+        rule::Nla::Protocol(2),
+    ];
+
     let mut msg = NetlinkMessage {
         header: NetlinkHeader {
             flags: NLM_F_REQUEST | NLM_F_CREATE | NLM_F_EXCL | NLM_F_ACK,
             ..Default::default()
         },
-        payload: NetlinkPayload::from(RtnlMessage::NewRule(RuleMessage {
-            header: RuleHeader {
-                family: AF_INET as u8,
-                table: RT_TABLE_DEFAULT,
-                action: FR_ACT_TO_TBL,
-                ..Default::default()
-            },
-            nlas: vec![
-                rule::Nla::Table(254),
-                rule::Nla::SuppressPrefixLen(4294967295),
-                rule::Nla::Priority(1000),
-                rule::Nla::Protocol(2),
-            ],
-        })),
+        payload: NetlinkPayload::from(RtnlMessage::NewRule(rule_msg)),
     };
 
     msg.finalize();

src/rtnl/address/message.rs

@@ -9,12 +9,14 @@ use crate::{
 };
 
 #[derive(Debug, PartialEq, Eq, Clone, Default)]
+#[non_exhaustive]
 pub struct AddressMessage {
     pub header: AddressHeader,
     pub nlas: Vec<Nla>,
 }
 
 #[derive(Debug, PartialEq, Eq, Clone, Default)]
+#[non_exhaustive]
 pub struct AddressHeader {
     pub family: u8,
     pub prefix_len: u8,

src/rtnl/address/nlas/cache_info.rs

@@ -6,6 +6,7 @@ use crate::{
 };
 
 #[derive(Debug, Clone, Copy, Eq, PartialEq, Default)]
+#[non_exhaustive]
 pub struct CacheInfo {
     pub ifa_preferred: i32,
     pub ifa_valid: i32,

src/rtnl/address/nlas/mod.rs

@@ -17,6 +17,7 @@ use crate::{
 };
 
 #[derive(Debug, PartialEq, Eq, Clone)]
+#[non_exhaustive]
 pub enum Nla {
     Unspec(Vec<u8>),
     Address(Vec<u8>),

src/rtnl/link/header.rs

@@ -25,6 +25,7 @@ use crate::{
 ///
 /// `LinkHeader` exposes all these fields except for the "reserved" one.
 #[derive(Debug, PartialEq, Eq, Clone, Default)]
+#[non_exhaustive]
 pub struct LinkHeader {
     /// Address family: one of the `AF_*` constants.
     pub interface_family: u8,

src/rtnl/link/message.rs

@@ -9,6 +9,7 @@ use crate::{
 };
 
 #[derive(Debug, PartialEq, Eq, Clone, Default)]
+#[non_exhaustive]
 pub struct LinkMessage {
     pub header: LinkHeader,
     pub nlas: Vec<Nla>,

src/rtnl/link/nlas/af_spec_bridge.rs

@@ -15,6 +15,7 @@ use crate::{
 use byteorder::{ByteOrder, NativeEndian};
 
 #[derive(Clone, Eq, PartialEq, Debug)]
+#[non_exhaustive]
 pub enum AfSpecBridge {
     Flags(u16),
     VlanInfo(BridgeVlanInfo),
@@ -75,6 +76,7 @@ impl<'a, T: AsRef<[u8]> + ?Sized> Parseable<NlaBuffer<&'a T>> for AfSpecBridge {
 }
 
 #[derive(Debug, PartialEq, Eq, Clone, Copy, Default)]
+#[non_exhaustive]
 pub struct BridgeVlanInfo {
     pub flags: u16,
     pub vid: u16,

src/rtnl/link/nlas/af_spec_inet.rs

@@ -11,6 +11,7 @@ use crate::{
 };
 
 #[derive(Clone, Eq, PartialEq, Debug)]
+#[non_exhaustive]
 pub enum AfSpecInet {
     Unspec(Vec<u8>),
     Unix(Vec<u8>),

src/rtnl/link/nlas/bond.rs

@@ -16,6 +16,7 @@ use std::{
 };
 
 #[derive(Debug, Clone, Eq, PartialEq)]
+#[non_exhaustive]
 pub enum BondAdInfo {
     Aggregator(u16),
     NumPorts(u16),
@@ -152,6 +153,7 @@ impl Nla for BondIpAddrNla {
 }
 
 #[derive(Debug, PartialEq, Eq, Clone)]
+#[non_exhaustive]
 pub enum InfoBond {
     Mode(u8),
     ActiveSlave(u32),

src/rtnl/link/nlas/bridge.rs

@@ -24,6 +24,7 @@ const BRIDGE_QUERIER_IPV6_PORT: u16 = 6;
 const BRIDGE_QUERIER_IPV6_OTHER_TIMER: u16 = 7;
 
 #[derive(Debug, PartialEq, Eq, Clone)]
+#[non_exhaustive]
 pub enum InfoBridge {
     Unspec(Vec<u8>),
     GroupAddr([u8; 6]),
@@ -498,6 +499,7 @@ impl<'a, T: AsRef<[u8]> + ?Sized> Parseable<NlaBuffer<&'a T>> for InfoBridge {
 }
 
 #[derive(Debug, Clone, Eq, PartialEq)]
+#[non_exhaustive]
 pub enum BridgeQuerierState {
     Ipv4Address(Ipv4Addr),
     Ipv4Port(u32),