Remove SliceJsonParser::parse_next

src/read.rs

@@ -292,24 +292,15 @@ impl BufferJsonParser {
 /// use json_event_parser::{JsonEvent, SliceJsonParser};
 /// use std::borrow::Cow;
 ///
-/// let mut reader = SliceJsonParser::new(b"{\"foo\": 1}");
-/// assert!(matches!(
-///     reader.parse_next(),
-///     Some(Ok(JsonEvent::StartObject))
-/// ));
-/// assert!(matches!(
-///     reader.parse_next(),
-///     Some(Ok(JsonEvent::ObjectKey(Cow::Borrowed("foo"))))
-/// ));
-/// assert!(matches!(
-///     reader.parse_next(),
-///     Some(Ok(JsonEvent::Number(Cow::Borrowed("1"))))
-/// ));
-/// assert!(matches!(
-///     reader.parse_next(),
-///     Some(Ok(JsonEvent::EndObject))
-/// ));
-/// assert!(matches!(reader.parse_next(), None));
+/// assert_eq!(
+///     SliceJsonParser::new(b"{\"foo\": 1}").collect::<Result<Vec<_>, _>>()?,
+///     vec![
+///         JsonEvent::StartObject,
+///         JsonEvent::ObjectKey(Cow::Borrowed("foo")),
+///         JsonEvent::Number(Cow::Borrowed("1")),
+///         JsonEvent::EndObject
+///     ]
+/// );
 /// # std::io::Result::Ok(())
 /// ```
 pub struct SliceJsonParser<'a> {
@@ -325,9 +316,12 @@ impl<'a> SliceJsonParser<'a> {
             parser: LowLevelJsonParser::new(),
         }
     }
+}
 
-    #[inline]
-    pub fn parse_next(&mut self) -> Option<Result<JsonEvent<'a>, JsonSyntaxError>> {
+impl<'a> Iterator for SliceJsonParser<'a> {
+    type Item = Result<JsonEvent<'a>, JsonSyntaxError>;
+
+    fn next(&mut self) -> Option<Result<JsonEvent<'a>, JsonSyntaxError>> {
         let LowLevelJsonParserResult {
             event,
             consumed_bytes,
@@ -337,14 +331,6 @@ impl<'a> SliceJsonParser<'a> {
     }
 }
 
-impl<'a> Iterator for SliceJsonParser<'a> {
-    type Item = Result<JsonEvent<'a>, JsonSyntaxError>;
-
-    fn next(&mut self) -> Option<Result<JsonEvent<'a>, JsonSyntaxError>> {
-        self.parse_next()
-    }
-}
-
 /// A low-level JSON parser acting on a provided buffer.
 ///
 /// Does not allocate except a stack to check if array and object opening and closing are properly nested.

tests/errors.rs

@@ -56,7 +56,7 @@ fn test_error_messages() {
     for (json, error) in entries {
         assert_eq!(
             SliceJsonParser::new(json)
-                .parse_next()
+                .next()
                 .unwrap()
                 .unwrap_err()
                 .to_string(),