refactor!: remove current_is_appropriate_end_tag_token from Emitter

1 files changed, 2 insertions, 37 deletions

diff --git a/src/emitter.rs b/src/emitter.rs
index ac0f9d2..769d233 100644
--- a/src/emitter.rs
+++ b/src/emitter.rs
@@ -15,7 +15,6 @@ use crate::Error;
 ///
 /// An emitter is assumed to have these internal states:
 ///
-/// * _last start tag_: The most recently emitted start tag's name
 /// * _current token_: Can be a tag, doctype or comment token. There's only one current token.
 /// * _current attribute_: The currently processed HTML attribute, consisting of two strings for name and value.
 ///
@@ -27,9 +26,6 @@ use crate::Error;
 ///   checks that would emit errors.
 ///
 /// * If you don't care about attributes at all, you can make all related methods a noop.
-///
-/// The state machine needs to have a functional implementation of
-/// `current_is_appropriate_end_tag_token` to do correct transitions, however.
 pub trait Emitter<R> {
     /// The token type emitted by this emitter. This controls what type of values the [`Tokenizer`](crate::Tokenizer)
     /// yields when used as an iterator.
@@ -63,8 +59,6 @@ pub trait Emitter<R> {
     /// Also get the current attribute and append it to the to-be-emitted tag. See docstring for
     /// [`Emitter::init_attribute_name`] for how duplicates should be handled.
     ///
-    /// If a start tag is emitted, update the _last start tag_.
-    ///
     /// If an end tag is emitted with attributes, an [`Error::EndTagWithAttributes`]
     /// error should be emitted.
     ///
@@ -162,23 +156,12 @@ pub trait Emitter<R> {
     ///
     /// If the current token is not a doctype, this method may panic.
     fn push_doctype_system_identifier(&mut self, s: &str);
-
-    /// Return true if all of these hold. Return false otherwise.
-    ///
-    /// * the _current token_ is an end tag
-    /// * the _last start tag_ exists
-    /// * the current end tag token's name equals to the last start tag's name.
-    ///
-    /// See also [WHATWG's definition of "appropriate end tag
-    /// token"](https://html.spec.whatwg.org/#appropriate-end-tag-token).
-    fn current_is_appropriate_end_tag_token(&mut self) -> bool;
 }
 
 /// The default implementation of [`Emitter`], used to produce tokens.
 pub struct DefaultEmitter<R, S> {
     current_characters: String,
     current_token: Option<Token<S>>,
-    last_start_tag: String,
     current_attribute: Option<(String, Attribute<S>)>,
     seen_attributes: BTreeSet<String>,
     emitted_tokens: VecDeque<Token<S>>,
@@ -191,7 +174,6 @@ impl<R, S> Default for DefaultEmitter<R, S> {
         DefaultEmitter {
             current_characters: String::new(),
             current_token: None,
-            last_start_tag: String::new(),
             current_attribute: None,
             seen_attributes: BTreeSet::new(),
             emitted_tokens: VecDeque::new(),
@@ -245,12 +227,6 @@ impl<R, S: Span<R>> DefaultEmitter<R, S> {
         // that exact
         self.emitted_tokens.push_front(Token::Error { error, span });
     }
-
-    pub(crate) fn set_last_start_tag(&mut self, last_start_tag: Option<&str>) {
-        self.last_start_tag.clear();
-        self.last_start_tag
-            .push_str(last_start_tag.unwrap_or_default());
-    }
 }
 
 impl<R, S: Span<R>> Emitter<R> for DefaultEmitter<R, S> {
@@ -293,7 +269,7 @@ impl<R, S: Span<R>> Emitter<R> for DefaultEmitter<R, S> {
     }
     fn emit_current_tag(&mut self) {
         self.flush_current_attribute();
-        let mut token = self.current_token.take().unwrap();
+        let token = self.current_token.take().unwrap();
         match token {
             Token::EndTag(_) => {
                 if !self.seen_attributes.is_empty() {
@@ -302,9 +278,7 @@ impl<R, S: Span<R>> Emitter<R> for DefaultEmitter<R, S> {
                 }
                 self.seen_attributes.clear();
             }
-            Token::StartTag(ref mut _tag) => {
-                self.set_last_start_tag(Some(&_tag.name));
-            }
+            Token::StartTag(_) => {}
             _ => debug_assert!(false),
         }
         self.emit_token(token);
@@ -458,15 +432,6 @@ impl<R, S: Span<R>> Emitter<R> for DefaultEmitter<R, S> {
             debug_assert!(false);
         }
     }
-
-    fn current_is_appropriate_end_tag_token(&mut self) -> bool {
-        match self.current_token {
-            Some(Token::EndTag(ref tag)) => {
-                !self.last_start_tag.is_empty() && self.last_start_tag == tag.name
-            }
-            _ => false,
-        }
-    }
 }
 
 /// An HTML start tag, such as `<p>` or `<a>`.