Fixed the reported inconsistency in Event::Void

Author: audrius

src/de.rs

@@ -630,12 +630,6 @@ impl<'de, 'document> DeserializerFromEvents<'de, 'document> {
             .map(|parse_error| error::shared(Arc::clone(parse_error)))
     }
 
-    fn reader_error_on_void(&self) -> Option<Error> {
-        match self.document.events.get(*self.pos) {
-            Some((Event::Void, _)) => self.document_error(),
-            _ => None,
-        }
-    }
 
     fn peek_event(&self) -> Result<&'document Event<'de>> {
         self.peek_event_mark().map(|(event, _mark)| event)
@@ -1005,9 +999,6 @@ impl<'de> de::SeqAccess<'de> for SeqAccess<'de, '_, '_> {
         if self.empty {
             return Ok(None);
         }
-        if let Some(err) = self.de.reader_error_on_void() {
-            return Err(err);
-        }
         match self.de.peek_event()? {
             Event::SequenceEnd | Event::Void => Ok(None),
             _ => {
@@ -1049,9 +1040,6 @@ impl<'de> de::MapAccess<'de> for MapAccess<'de, '_, '_> {
         if self.empty {
             return Ok(None);
         }
-        if let Some(err) = self.de.reader_error_on_void() {
-            return Err(err);
-        }
         loop {
             match self.de.peek_event()? {
                 Event::MappingEnd | Event::Void => return Ok(None),
@@ -2016,9 +2004,6 @@ impl<'de> de::Deserializer<'de> for &mut DeserializerFromEvents<'de, '_> {
     where
         V: Visitor<'de>,
     {
-        if let Some(err) = self.reader_error_on_void() {
-            return Err(err);
-        }
         let is_some = match self.peek_event()? {
             &Event::Alias(mut pos) => {
                 *self.pos += 1;
@@ -2066,9 +2051,6 @@ impl<'de> de::Deserializer<'de> for &mut DeserializerFromEvents<'de, '_> {
     where
         V: Visitor<'de>,
     {
-        if let Some(err) = self.reader_error_on_void() {
-            return Err(err);
-        }
         let tagged_already = *self.enum_depth.borrow() > 0;
         let (next, mark) = self.next_event_mark()?;
         match next {
@@ -2116,9 +2098,6 @@ impl<'de> de::Deserializer<'de> for &mut DeserializerFromEvents<'de, '_> {
     where
         V: Visitor<'de>,
     {
-        if let Some(err) = self.reader_error_on_void() {
-            return Err(err);
-        }
         let (next, mark) = self.next_event_mark()?;
         match next {
             &Event::Alias(mut pos) => self.jump(&mut pos)?.deserialize_seq(visitor),
@@ -2167,9 +2146,6 @@ impl<'de> de::Deserializer<'de> for &mut DeserializerFromEvents<'de, '_> {
     where
         V: Visitor<'de>,
     {
-        if let Some(err) = self.reader_error_on_void() {
-            return Err(err);
-        }
         let (next, mark) = self.next_event_mark()?;
         match next {
             &Event::Alias(mut pos) => self.jump(&mut pos)?.deserialize_map(visitor),

tests/test_scalar_void.rs

@@ -0,0 +1,22 @@
+use serde::Deserialize;
+use serde_yaml_bw::{budget::Budget, Deserializer, DeserializerOptions};
+use std::io::Cursor;
+
+#[test]
+fn test_scalar_budget_breach_bool() {
+    let input = "true\n";
+    let options = DeserializerOptions {
+        budget: Some(Budget {
+            max_total_scalar_bytes: 0,
+            ..Budget::default()
+        }),
+        ..DeserializerOptions::default()
+    };
+    
+    let err = bool::deserialize(Deserializer::from_reader_with_options(
+        Cursor::new(input.as_bytes()),
+        &options,
+    )).unwrap_err();
+    
+    assert!(err.to_string().contains("budget exceeded"), "Expected budget exceeded, got: {:?}", err);
+}