Improve empty query errors

src/sql/parser.rs

@@ -5,23 +5,20 @@ use std::str;
 
 #[allow(dead_code)]
 pub fn parse(input: &str) -> Result<Query, Error> {
-	match query(input) {
-		Ok((_, query)) => {
-			if query.empty() {
-				Err(Error::EmptyError)
-			} else {
-				Ok(query)
-			}
-		}
-		Err(Err::Error(e)) => Err(Error::ParseError {
-			pos: input.len() - e.input.len(),
-			sql: String::from(e.input),
-		}),
-		Err(Err::Failure(e)) => Err(Error::ParseError {
-			pos: input.len() - e.input.len(),
-			sql: String::from(e.input),
-		}),
-		Err(Err::Incomplete(_)) => Err(Error::EmptyError),
+	match input.trim().len() {
+		0 => Err(Error::EmptyError),
+		_ => match query(input) {
+			Ok((_, query)) => Ok(query),
+			Err(Err::Error(e)) => Err(Error::ParseError {
+				pos: input.len() - e.input.len(),
+				sql: String::from(e.input),
+			}),
+			Err(Err::Failure(e)) => Err(Error::ParseError {
+				pos: input.len() - e.input.len(),
+				sql: String::from(e.input),
+			}),
+			Err(Err::Incomplete(_)) => Err(Error::EmptyError),
+		},
 	}
 }
 

src/sql/query.rs

@@ -11,10 +11,6 @@ pub struct Query {
 }
 
 impl Query {
-	pub fn empty(&self) -> bool {
-		self.statements.len() == 0
-	}
-
 	pub fn statements(&self) -> &Vec<Statement> {
 		&self.statements.0
 	}