Multi-Results: review suggestion

- get rid of a recursive call

Author: jszwec

rows.go

@@ -117,15 +117,25 @@ func (rows *mysqlRows) nextResultSet() (int, error) {
 	return rows.mc.readResultSetHeaderPacket()
 }
 
-func (rows *binaryRows) NextResultSet() error {
-	resLen, err := rows.nextResultSet()
-	if err != nil {
-		return err
-	}
+func (rows *mysqlRows) nextNotEmptyResultSet() (int, error) {
+	for {
+		resLen, err := rows.nextResultSet()
+		if err != nil {
+			return 0, err
+		}
+
+		if resLen > 0 {
+			return resLen, nil
+		}
 
-	if resLen == 0 {
 		rows.rs.done = true
-		return rows.NextResultSet()
+	}
+}
+
+func (rows *binaryRows) NextResultSet() (err error) {
+	resLen, err := rows.nextNotEmptyResultSet()
+	if err != nil {
+		return err
 	}
 
 	// get columns, if not cached, read them and cache them.
@@ -155,17 +165,12 @@ func (rows *binaryRows) Next(dest []driver.Value) error {
 	return io.EOF
 }
 
-func (rows *textRows) NextResultSet() error {
-	resLen, err := rows.nextResultSet()
+func (rows *textRows) NextResultSet() (err error) {
+	resLen, err := rows.nextNotEmptyResultSet()
 	if err != nil {
 		return err
 	}
 
-	if resLen == 0 {
-		rows.rs.done = true
-		return rows.NextResultSet()
-	}
-
 	rows.rs.columns, err = rows.mc.readColumns(resLen)
 	return err
 }