niebayes opened a new issue, #1593:
URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1593

   We have designed an SQL parser using sqlparser's `Parser`, and we wish to 
implement two helper methods for consuming tokens—similar to 
`Parser::consume_token` and `Parser::consume_tokens`—but with the ability to 
accept string inputs and perform case-insensitive matching.
   
   However, since the Parser does not provide a method to mutate the 
`Parser::index`, we are unable to correctly implement our own `consume_tokens` 
because we need to revert the parser state if consumption fails mid-way.
   
   We propose adding and exposing a new method in sqlparser's `Parser`, such as 
`set_index` or `index_mut`, to allow users to mutate the index.
   
   The expected implementation of our `consume_token` and `consume_tokens`:
   
   ``` rust
   /// Consumes the next token if it matches the expected token, otherwise 
return false.
   ///
   /// Note, the matching is not case sensitive.
   fn consume_token(&mut self, expected: &str) -> bool {
       if self.parser.peek_token().to_string().to_uppercase() == 
*expected.to_uppercase() {
           self.parser.next_token();
           true
       } else {
           false
       }
   }
   
   /// If the current and subsequent tokens exactly match the `tokens` 
sequence, consume them and returns true.
   /// Otherwise, no tokens are consumed and returns false
   ///
   /// Note, the matching is not case sensitive.
   fn consume_tokens(&mut self, tokens: &[&str]) -> bool {
       let index = self.parser.index();
       for token in tokens {
           if !self.consume_token(*token) {
               // Or: `*self.parser.index_mut() = index;`
               self.parser.set_index(index);
               return false;
           }
       }
       true
   }
   
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to