From c6f92eaedf197beafef461ee6c2bd067e7160c49 Mon Sep 17 00:00:00 2001
From: Titus Wormer <tituswormer@gmail.com>
Date: Tue, 21 Jun 2022 16:06:50 +0200
Subject: Refactor to improve a bunch of states
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

*   Improve passing stuff around
*   Add traits to enums for markers and such
*   Fix “life time” stuff I didn’t understand
---
 src/construct/code_text.rs | 26 ++++++++------------------
 1 file changed, 8 insertions(+), 18 deletions(-)

(limited to 'src/construct/code_text.rs')

diff --git a/src/construct/code_text.rs b/src/construct/code_text.rs
index 1f34e41..94e0106 100644
--- a/src/construct/code_text.rs
+++ b/src/construct/code_text.rs
@@ -113,9 +113,7 @@ fn sequence_open(tokenizer: &mut Tokenizer, code: Code, size: usize) -> StateFnR
     if let Code::Char('`') = code {
         tokenizer.consume(code);
         (
-            State::Fn(Box::new(move |tokenizer, code| {
-                sequence_open(tokenizer, code, size + 1)
-            })),
+            State::Fn(Box::new(move |t, c| sequence_open(t, c, size + 1))),
             None,
         )
     } else {
@@ -138,9 +136,7 @@ fn between(tokenizer: &mut Tokenizer, code: Code, size_open: usize) -> StateFnRe
             tokenizer.consume(code);
             tokenizer.exit(TokenType::CodeTextLineEnding);
             (
-                State::Fn(Box::new(move |tokenizer, code| {
-                    between(tokenizer, code, size_open)
-                })),
+                State::Fn(Box::new(move |t, c| between(t, c, size_open))),
                 None,
             )
         }
@@ -168,12 +164,7 @@ fn data(tokenizer: &mut Tokenizer, code: Code, size_open: usize) -> StateFnResul
         }
         _ => {
             tokenizer.consume(code);
-            (
-                State::Fn(Box::new(move |tokenizer, code| {
-                    data(tokenizer, code, size_open)
-                })),
-                None,
-            )
+            (State::Fn(Box::new(move |t, c| data(t, c, size_open))), None)
         }
     }
 }
@@ -193,8 +184,8 @@ fn sequence_close(
         Code::Char('`') => {
             tokenizer.consume(code);
             (
-                State::Fn(Box::new(move |tokenizer, code| {
-                    sequence_close(tokenizer, code, size_open, size + 1)
+                State::Fn(Box::new(move |t, c| {
+                    sequence_close(t, c, size_open, size + 1)
                 })),
                 None,
             )
@@ -205,12 +196,11 @@ fn sequence_close(
             (State::Ok, Some(vec![code]))
         }
         _ => {
-            let tail_index = tokenizer.events.len();
-            let head_index = tokenizer.events.len() - 1;
+            let index = tokenizer.events.len();
             tokenizer.exit(TokenType::CodeTextSequence);
             // Change the token type.
-            tokenizer.events[head_index].token_type = TokenType::CodeTextData;
-            tokenizer.events[tail_index].token_type = TokenType::CodeTextData;
+            tokenizer.events[index - 1].token_type = TokenType::CodeTextData;
+            tokenizer.events[index].token_type = TokenType::CodeTextData;
             between(tokenizer, code, size_open)
         }
     }
-- 
cgit