fix terminated

toml-rs · May 2, 2024 · f3b6185 · f3b6185
1 parent c3d0870
commit f3b6185
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/crates/toml_edit/src/parser/strings.rs b/crates/toml_edit/src/parser/strings.rs
@@ -218,15 +218,17 @@ fn mlb_quotes<'i>(
     move |input: &mut Input<'i>| {
         let start = input.checkpoint();
         let res = terminated(b"\"\"", peek(term.by_ref()))
-            // Safety: ???
+            // Safety: terminated returns the output of the first parser here,
+            // which only parses ASCII
             .map(|b| unsafe { from_utf8_unchecked(b, "`bytes` out non-ASCII") })
             .parse_next(input);
 
         match res {
             Err(winnow::error::ErrMode::Backtrack(_)) => {
                 input.reset(&start);
                 terminated(b"\"", peek(term.by_ref()))
-                    // Safety: ???
+                    // Safety: terminated returns the output of the first parser here,
+                    // which only parses ASCII
                     .map(|b| unsafe { from_utf8_unchecked(b, "`bytes` out non-ASCII") })
                     .parse_next(input)
             }
@@ -349,15 +351,17 @@ fn mll_quotes<'i>(
     move |input: &mut Input<'i>| {
         let start = input.checkpoint();
         let res = terminated(b"''", peek(term.by_ref()))
-            // Safety: ???
+            // Safety: terminated returns the output of the first parser here,
+            // which only parses ASCII
             .map(|b| unsafe { from_utf8_unchecked(b, "`bytes` out non-ASCII") })
             .parse_next(input);
 
         match res {
             Err(winnow::error::ErrMode::Backtrack(_)) => {
                 input.reset(&start);
                 terminated(b"'", peek(term.by_ref()))
-                    // Safety: ???
+                    // Safety: terminated returns the output of the first parser here,
+                    // which only parses ASCII
                     .map(|b| unsafe { from_utf8_unchecked(b, "`bytes` out non-ASCII") })
                     .parse_next(input)
             }