Merge pull request #8 from re-gmbh/feature/ord-node-id

Derive Ord for NodeId

lib/src/types/byte_string.rs

@@ -17,7 +17,7 @@ use crate::types::{
 };
 
 /// A sequence of octets.
-#[derive(Eq, PartialEq, Debug, Clone, Hash, Serialize, Deserialize)]
+#[derive(Eq, PartialEq, Debug, Clone, Hash, Serialize, Deserialize, PartialOrd, Ord)]
 pub struct ByteString {
     pub value: Option<Vec<u8>>,
 }

lib/src/types/guid.rs

@@ -15,7 +15,7 @@ use uuid::Uuid;
 use crate::types::encoding::*;
 
 /// A Guid is a 16 byte Globally Unique Identifier.
-#[derive(Eq, PartialEq, Clone, Hash)]
+#[derive(Eq, PartialEq, Clone, Hash, PartialOrd, Ord)]
 pub struct Guid {
     uuid: Uuid,
 }

lib/src/types/node_id.rs

@@ -24,7 +24,7 @@ use crate::types::{
 };
 
 /// The kind of identifier, numeric, string, guid or byte
-#[derive(Eq, PartialEq, Clone, Debug, Hash, Serialize, Deserialize)]
+#[derive(Eq, PartialEq, Clone, Debug, Hash, Serialize, Deserialize, PartialOrd, Ord)]
 pub enum Identifier {
     Numeric(u32),
     String(UAString),
@@ -123,7 +123,7 @@ impl fmt::Display for NodeIdError {
 impl std::error::Error for NodeIdError {}
 
 /// An identifier for a node in the address space of an OPC UA Server.
-#[derive(PartialEq, Eq, Clone, Debug, Hash, Serialize, Deserialize)]
+#[derive(PartialEq, Eq, Clone, Debug, Hash, Serialize, Deserialize, PartialOrd, Ord)]
 pub struct NodeId {
     /// The index for a namespace
     pub namespace: u16,

lib/src/types/string.rs

@@ -23,7 +23,7 @@ use crate::types::{
 /// A string contains UTF-8 encoded characters or a null value. A null value is distinct from
 /// being an empty string so internally, the code maintains that distinction by holding the value
 /// as an `Option<String>`.
-#[derive(Eq, PartialEq, Debug, Clone, Hash, Serialize, Deserialize)]
+#[derive(Eq, PartialEq, Debug, Clone, Hash, Serialize, Deserialize, PartialOrd, Ord)]
 pub struct UAString {
     value: Option<String>,
 }