From 405945f52b88912bd58c9e483b6ba5664e1acfc9 Mon Sep 17 00:00:00 2001
From: Ariel Mashraki <7413593+a8m@users.noreply.github.com>
Date: Thu, 1 Aug 2019 23:39:16 +0300
Subject: [PATCH] rql: changing types to pointers (#19)

---
 rql.go | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/rql.go b/rql.go
index a1f1da8..b42c7bb 100644
--- a/rql.go
+++ b/rql.go
@@ -142,19 +142,19 @@ func MustNewParser(c Config) *Parser {
 // Parse parses the given buffer into a Param object. It returns an error
 // if the JSON is invalid, or its values don't follow the schema of rql.
 func (p *Parser) Parse(b []byte) (pr *Params, err error) {
-	q := new(Query)
+	q := &Query{}
 	if err := q.UnmarshalJSON(b); err != nil {
 		return nil, &ParseError{"decoding buffer to *Query: " + err.Error()}
 	}
-	return p.ParseQuery(*q)
+	return p.ParseQuery(q)
 }
 
 // ParseQuery parses the given struct into a Param object. It returns an error
 // if one of the query values don't follow the schema of rql.
-func (p *Parser) ParseQuery(q Query) (pr *Params, err error) {
+func (p *Parser) ParseQuery(q *Query) (pr *Params, err error) {
 	defer func() {
 		if e := recover(); e != nil {
-			perr, ok := e.(ParseError)
+			perr, ok := e.(*ParseError)
 			if !ok {
 				panic(e)
 			}
@@ -469,7 +469,7 @@ func (p *Parser) op(op Op) string {
 // expect panic if the condition is false.
 func expect(cond bool, msg string, args ...interface{}) {
 	if !cond {
-		panic(ParseError{fmt.Sprintf(msg, args...)})
+		panic(&ParseError{fmt.Sprintf(msg, args...)})
 	}
 }
 
@@ -477,7 +477,7 @@ func expect(cond bool, msg string, args ...interface{}) {
 func must(err error, msg string, args ...interface{}) {
 	if err != nil {
 		args = append(args, err)
-		panic(ParseError{fmt.Sprintf(msg+": %s", args...)})
+		panic(&ParseError{fmt.Sprintf(msg+": %s", args...)})
 	}
 }