Removed obsolete/unplanned TODO statements

Author: ngeor

rusty_parser/src/core/comment.rs

@@ -14,7 +14,6 @@ pub fn comment_p() -> impl Parser<StringView, Output = Statement, Error = Parser
 /// Parses a comment as a [String].
 /// Does not consume the EOL token.
 pub fn comment_as_string_p() -> impl Parser<StringView, Output = String, Error = ParserError> {
-    // TODO support ignoring tokens to avoid allocation
     any_symbol_of!('\'').and_keep_right(
         any_token_of!(TokenType::Eol ; mode = MatchMode::Exclude)
             .many_allow_none(StringManyCombiner),

rusty_parser/src/core/param_name.rs

@@ -92,7 +92,6 @@ fn parameter_p() -> impl Parser<StringView, Output = Parameter, Error = ParserEr
 
 fn array_indicator() -> impl Parser<StringView, Output = Option<(Token, Token)>, Error = ParserError>
 {
-    // TODO support ignoring token to avoid allocation
     seq2(any_symbol_of!('('), any_symbol_of!(')'), |l, r| (l, r)).to_option()
 }
 

rusty_parser/src/core/print.rs

@@ -171,7 +171,6 @@ fn any_print_arg() -> impl Parser<StringView, Output = PrintArg, Error = ParserE
 }
 
 fn delimiter_print_arg() -> impl Parser<StringView, Output = PrintArg, Error = ParserError> {
-    // TODO support char based tokens or make the next mapping friendlier
     any_symbol_of_ws!(';', ',').map(|t| {
         if ';'.matches_token(&t) {
             PrintArg::Semicolon

rusty_parser/src/expr/string_literal.rs

@@ -18,7 +18,6 @@ pub(super) fn parser() -> impl Parser<StringView, Output = ExpressionPos, Error
 }
 
 fn string_delimiter() -> impl Parser<StringView, Output = Token, Error = ParserError> {
-    // TODO support ignoring token to avoid allocation
     any_symbol_of!('"')
 }
 

rusty_parser/src/pc_specific/keyword.rs

@@ -8,8 +8,6 @@ use crate::pc_specific::whitespace_ignoring;
 use crate::tokens::{TokenMatcher, TokenType, any_token};
 use crate::{Keyword, ParserError};
 
-// TODO review usages of TokenType::Keyword
-
 /// Matches the specific keyword.
 ///
 /// Ensures that it is not followed by a dollar sign, in which case it is a valid identifier.
@@ -50,7 +48,6 @@ pub fn keyword_ws_p(k: Keyword) -> impl Parser<StringView, Output = (), Error =
     keyword_ignoring(k).and(whitespace_ignoring().to_fatal(), IgnoringBothCombiner)
 }
 
-// TODO add keyword_pair_ws
 /// Parses the first keyword, followed by mandatory whitespace,
 /// followed by the second keyword. If the first keyword is parsed,
 /// both the whitespace and the second keyword must be parsed.