From a2c77bd2f8110c505464077c22d489d6a6d4c8c8 Mon Sep 17 00:00:00 2001
From: Daniel Schadt <kingdread@gmx.de>
Date: Tue, 19 May 2020 15:24:28 +0200
Subject: 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.
---
 src/event.rs      |  3 ++-
 src/raw/parser.rs |  4 ++--
 src/raw/types.rs  | 10 ++++++++++
 3 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/event.rs b/src/event.rs
index cf2a42f..df0d5d3 100644
--- a/src/event.rs
+++ b/src/event.rs
@@ -365,7 +365,8 @@ impl TryFrom<&raw::CbtEvent> for Event {
             | CbtStateChange::SkillTiming
             | CbtStateChange::BreakbarState
             | CbtStateChange::BreakbarPercent
-            | CbtStateChange::Error => {
+            | CbtStateChange::Error
+            | CbtStateChange::Unknown => {
                 return Err(FromRawEventError::UnknownStateChange(
                     raw_event.is_statechange,
                 ))
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 {
-- 
cgit v1.2.3