aboutsummaryrefslogtreecommitdiff
path: root/planetwars-matchrunner/src/pw_match.rs
diff options
context:
space:
mode:
Diffstat (limited to 'planetwars-matchrunner/src/pw_match.rs')
-rw-r--r--planetwars-matchrunner/src/pw_match.rs6
1 files changed, 5 insertions, 1 deletions
diff --git a/planetwars-matchrunner/src/pw_match.rs b/planetwars-matchrunner/src/pw_match.rs
index 737aa00..62a07e5 100644
--- a/planetwars-matchrunner/src/pw_match.rs
+++ b/planetwars-matchrunner/src/pw_match.rs
@@ -1,3 +1,4 @@
+use crate::match_context::RequestError;
use crate::match_log::MatchLogMessage;
use super::match_context::{MatchCtx, RequestResult};
@@ -89,7 +90,8 @@ impl PwMatch {
fn execute_action(&mut self, player_num: usize, turn: RequestResult<Vec<u8>>) -> PlayerAction {
let data = match turn {
- Err(_timeout) => return PlayerAction::Timeout,
+ Err(RequestError::Timeout) => return PlayerAction::Timeout,
+ Err(RequestError::BotTerminated) => return PlayerAction::Terminated,
Ok(data) => data,
};
@@ -123,6 +125,7 @@ impl PwMatch {
PlayerAction::Timeout => self.match_ctx.log(MatchLogMessage::Timeout {
player_id: player_id as u32,
}),
+ PlayerAction::Terminated => (), // TODO: should something be logged here?
PlayerAction::ParseError { data, error } => {
// TODO: can this be handled better?
let command =
@@ -156,6 +159,7 @@ pub struct PlayerCommand {
// TODO: can we name this better? Is this a "play"?
pub enum PlayerAction {
Timeout,
+ Terminated,
ParseError {
data: Vec<u8>,
error: serde_json::Error,