Skip to content

Commit 5f9957a

Browse files
committed
sql-parser: clean-up of dbg! macros
1 parent 83ae225 commit 5f9957a

File tree

5 files changed

+0
-20
lines changed

5 files changed

+0
-20
lines changed

src/parser/expression/function_expr.rs

-3
Original file line numberDiff line numberDiff line change
@@ -268,9 +268,6 @@ impl<'a> FunctionParser for Parser<'a> {
268268
}
269269
};
270270

271-
dbg!(&frame_type);
272-
dbg!(&self.peek_token());
273-
274271
// consume the frame type token
275272
self.consume_token()?;
276273

src/parser/expression/identifier.rs

-3
Original file line numberDiff line numberDiff line change
@@ -24,14 +24,12 @@ impl<'a> IdentifierParser for Parser<'a> {
2424
}
2525
}
2626

27-
dbg!("Expected next identifier: {}", expected_next_identifier);
2827
if expected_next_identifier {
2928
return Err(ParsingError::UnexpectedToken(
3029
"Expected identifier".to_string(),
3130
));
3231
}
3332

34-
dbg!("Parsed components: {:?}", &components);
3533
match components.len() {
3634
0 => Err(ParsingError::UnexpectedToken(
3735
"Expected identifier".to_string(),
@@ -45,7 +43,6 @@ impl<'a> IdentifierParser for Parser<'a> {
4543
}
4644
2 => {
4745
if components[1] == "*" {
48-
dbg!("Parsed table name with wildcard {}", &components[0]);
4946
Ok(Identifier::NameWithWildcard(components[0].to_string()))
5047
} else {
5148
Ok(Identifier::Compound(components))

src/parser/expression/mod.rs

-2
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,6 @@ pub trait ExpressionParser {
9595
impl<'a> ExpressionParser for Parser<'a> {
9696
/// Parse an SQLite3 [expr](https://www.sqlite.org/lang_expr.html#the_expr_list)
9797
fn parse_expression(&mut self) -> Result<Expression, ParsingError> {
98-
dbg!("Parsing expression");
9998
// Check if it's one of the special expressions
10099
if let Ok(keyword) = self.peek_as_keyword() {
101100
match keyword {
@@ -220,7 +219,6 @@ impl<'a> ExpressionParser for Parser<'a> {
220219
}
221220
}
222221

223-
dbg!("Parsed expression: {:?}", &expression);
224222
Ok(expression)
225223
}
226224

src/parser/select/mod.rs

-11
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ impl<'a> SelectStatementParser for Parser<'a> {
4545
}
4646

4747
fn parse_select_columns(&mut self) -> Result<Vec<SelectItem>, ParsingError> {
48-
dbg!("Parsing select columns");
4948
let mut select_items = Vec::new();
5049

5150
loop {
@@ -56,24 +55,17 @@ impl<'a> SelectStatementParser for Parser<'a> {
5655
break;
5756
}
5857
}
59-
60-
dbg!("Parsed select columns");
61-
dbg!(&select_items);
6258
Ok(select_items)
6359
}
6460

6561
fn parse_select_column(&mut self) -> Result<SelectItem, ParsingError> {
66-
dbg!("Parsing select column");
6762
if self.consume_as(TokenType::Star).is_ok() {
6863
return Ok(SelectItem::Expression(Expression::Identifier(
6964
Identifier::Wildcard,
7065
)));
7166
}
7267

73-
dbg!("Parsing select column as expression");
7468
if let Ok(expression) = self.parse_expression() {
75-
dbg!("Parsed select column as expression, {:?}", &expression);
76-
7769
// Consume the AS keyword if it exists
7870
let _ = self.consume_as_keyword(Keyword::As);
7971
if let Ok(alias) = self.consume_as_id() {
@@ -83,10 +75,7 @@ impl<'a> SelectStatementParser for Parser<'a> {
8375
return Ok(SelectItem::Expression(expression));
8476
}
8577

86-
dbg!("Parsing select column as table name with wildcard");
8778
if let Ok(table_name) = self.peek_as_string() {
88-
dbg!("Parsing select column as table name with wildcard");
89-
dbg!(&table_name);
9079
self.consume_token()?;
9180
self.consume_as(TokenType::Dot)?;
9281
self.consume_as(TokenType::Star)?;

src/parser/sqlite/vacuum.rs

-1
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ impl<'a> VacuumStatementParser for Parser<'a> {
3737
file_name: None,
3838
}
3939
};
40-
dbg!(&self.peek_token()?);
4140
self.finalize_statement_parsing()?;
4241
Ok(Statement::Vacuum(vacuum_statement))
4342
}

0 commit comments

Comments
 (0)