diff options
| author | Ting-Wei Lan <tingwei.lan@cobinhood.com> | 2019-02-21 16:00:23 +0800 |
|---|---|---|
| committer | Jhih-Ming Huang <jm.huang@cobinhood.com> | 2019-05-06 10:44:04 +0800 |
| commit | 069cf994d2436a99c30775fb9839075a6c29938a (patch) | |
| tree | 0e3551b9187e25904c337907d7a8d7809602599a | |
| parent | 124183ce94ef9941a077ff7e2a84738dfd7f8dcb (diff) | |
| download | dexon-069cf994d2436a99c30775fb9839075a6c29938a.tar.gz dexon-069cf994d2436a99c30775fb9839075a6c29938a.tar.zst dexon-069cf994d2436a99c30775fb9839075a6c29938a.zip | |
core: vm: sqlvm: ast: remove pointer indicator in printer output
Since our 'Node' interface includes methods which must be implemented
with pointer receivers, all AST nodes are now referenced with pointers
and the '*' pointer indicator is no longer useful.
| -rw-r--r-- | core/vm/sqlvm/ast/printer.go | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/core/vm/sqlvm/ast/printer.go b/core/vm/sqlvm/ast/printer.go index 56cfd07b9..e9b289411 100644 --- a/core/vm/sqlvm/ast/printer.go +++ b/core/vm/sqlvm/ast/printer.go @@ -44,18 +44,17 @@ func printAST(w io.Writer, n interface{}, depth int, base string, detail bool) { } typeOf := reflect.TypeOf(n) valueOf := reflect.ValueOf(n) - name := "" - if typeOf.Kind() == reflect.Ptr { + kind := typeOf.Kind() + if kind == reflect.Ptr { if valueOf.IsNil() { fmt.Fprintf(w, "%snil\n", indent) return } - name = "*" valueOf = valueOf.Elem() typeOf = typeOf.Elem() + kind = typeOf.Kind() } - kind := typeOf.Kind() - name = name + typeOf.Name() + name := typeOf.Name() if op, ok := n.(UnaryOperator); ok { fmt.Fprintf(w, "%s%s:\n", indent, name) |