add CbtStateChange::Unknown

This is not defined by arcdps, but we'd have to adjust evtclib every
time a new statechange is introduced. This way, we stay
forward-compatible.

2 files changed, 12 insertions, 2 deletions

diff --git a/src/raw/parser.rs b/src/raw/parser.rs
index bf61f8c..8401146 100644
--- a/src/raw/parser.rs
+++ b/src/raw/parser.rs
@@ -350,7 +350,7 @@ pub fn parse_event_rev0<R: Read>(mut input: R) -> ParseResult<CbtEvent> {
     let is_fifty = input.read_u8()? != 0;
     let is_moving = input.read_u8()? != 0;
     let is_statechange =
-        CbtStateChange::from_u8(input.read_u8()?).ok_or(ParseError::InvalidData)?;
+        CbtStateChange::from_u8(input.read_u8()?).unwrap_or(CbtStateChange::Unknown);
     let is_flanking = input.read_u8()? != 0;
     let is_shields = input.read_u8()? != 0;
 
@@ -411,7 +411,7 @@ pub fn parse_event_rev1<R: Read>(mut input: R) -> ParseResult<CbtEvent> {
     let is_fifty = input.read_u8()? != 0;
     let is_moving = input.read_u8()? != 0;
     let is_statechange =
-        CbtStateChange::from_u8(input.read_u8()?).ok_or(ParseError::InvalidData)?;
+        CbtStateChange::from_u8(input.read_u8()?).unwrap_or(CbtStateChange::Unknown);
     let is_flanking = input.read_u8()? != 0;
     let is_shields = input.read_u8()? != 0;
     let is_offcycle = input.read_u8()? != 0;
diff --git a/src/raw/types.rs b/src/raw/types.rs
index 820d7a5..9931ea9 100644
--- a/src/raw/types.rs
+++ b/src/raw/types.rs
@@ -201,6 +201,16 @@ pub enum CbtStateChange {
     BreakbarPercent,
     /// `time` is the start of the error string.
     Error,
+    /// The given state change is unknown.
+    ///
+    /// Note that this is not defined by arcdps itself, we just use this value as an easy way out
+    /// for when unknown statechanges are introduces.
+    ///
+    /// This is for future proofing, whenever a new event is added, we'd otherwise have to update
+    /// `evtclib` as well. The arcdps evtc README states to *make sure to ignore unknown
+    /// statechange types.*
+    // Keep this as the highest value, so we don't clash with future statechange additions.
+    Unknown = 255,
 }
 
 impl Default for CbtStateChange {