internal: move DebugStr to new astinternal package

More robust. Change-Id: I4ab0b510be244106cdcf75535ebe7a37e8e8c269 Reviewed-on: https://cue-review.googlesource.com/c/cue/+/9367 Reviewed-by: CUE cueckoo <cueckoo@gmail.com> Reviewed-by: Paul Jolly <paul@myitcv.org.uk>
cue-lang · Apr 13, 2021 · 80a0a6e · 80a0a6e
1 parent 52db572
commit 80a0a6e
Show file tree

Hide file tree

Showing 12 changed files with 75 additions and 78 deletions.
diff --git a/cmd/cue/cmd/orphans.go b/cmd/cue/cmd/orphans.go
@@ -28,6 +28,7 @@ import (
 	"cuelang.org/go/cue/parser"
 	"cuelang.org/go/cue/token"
 	"cuelang.org/go/internal"
+	"cuelang.org/go/internal/astinternal"
 	"cuelang.org/go/internal/encoding"
 )
 
@@ -220,7 +221,7 @@ func placeOrphans(b *buildPlan, d *encoding.Decoder, pkg string, objs ...*ast.Fi
 						}
 						return nil, fmt.Errorf(
 							`error evaluating label %v: %v`,
-							internal.DebugStr(x), arg)
+							astinternal.DebugStr(x), arg)
 					}
 				}
 				a = append(a, cue.Label(label))
@@ -339,7 +340,7 @@ func newIndex() *listIndex {
 }
 
 func (x *listIndex) label(label ast.Label) *listIndex {
-	key := internal.DebugStr(label)
+	key := astinternal.DebugStr(label)
 	idx := x.index[key]
 	if idx == nil {
 		if x.field.Value == nil {

diff --git a/cue/parser/parser.go b/cue/parser/parser.go
@@ -25,8 +25,11 @@ import (
 	"cuelang.org/go/cue/scanner"
 	"cuelang.org/go/cue/token"
 	"cuelang.org/go/internal"
+	"cuelang.org/go/internal/astinternal"
 )
 
+var debugStr = astinternal.DebugStr
+
 // The parser structure holds the parser's internal state.
 type parser struct {
 	file    *token.File

diff --git a/cue/path.go b/cue/path.go
@@ -24,7 +24,7 @@ import (
 	"cuelang.org/go/cue/literal"
 	"cuelang.org/go/cue/parser"
 	"cuelang.org/go/cue/token"
-	"cuelang.org/go/internal"
+	"cuelang.org/go/internal/astinternal"
 	"cuelang.org/go/internal/core/adt"
 	"github.com/cockroachdb/apd/v2"
 )
@@ -176,7 +176,7 @@ func toSelectors(expr ast.Expr) []Selector {
 		if b, ok := x.Index.(*ast.BasicLit); !ok {
 			sel = Selector{pathError{
 				errors.Newf(token.NoPos, "non-constant expression %s",
-					internal.DebugStr(x.Index))}}
+					astinternal.DebugStr(x.Index))}}
 		} else {
 			sel = basicLitSelector(b)
 		}
@@ -188,7 +188,7 @@ func toSelectors(expr ast.Expr) []Selector {
 
 	default:
 		return []Selector{{pathError{
-			errors.Newf(token.NoPos, "invalid label %s ", internal.DebugStr(x)),
+			errors.Newf(token.NoPos, "invalid label %s ", astinternal.DebugStr(x)),
 		}}}
 	}
 }
@@ -267,7 +267,7 @@ func Label(label ast.Label) Selector {
 
 	default:
 		return Selector{pathError{
-			errors.Newf(token.NoPos, "invalid label %s ", internal.DebugStr(x)),
+			errors.Newf(token.NoPos, "invalid label %s ", astinternal.DebugStr(x)),
 		}}
 	}
 }

diff --git a/cue/types_test.go b/cue/types_test.go
@@ -29,7 +29,7 @@ import (
 
 	"cuelang.org/go/cue/ast"
 	"cuelang.org/go/cue/errors"
-	"cuelang.org/go/internal"
+	"cuelang.org/go/internal/astinternal"
 	"cuelang.org/go/internal/core/adt"
 	"cuelang.org/go/internal/core/debug"
 )
@@ -823,7 +823,7 @@ v: #X
 			t.Errorf("got %v; want %v", got, tc.raw)
 		}
 
-		got := fmt.Sprint(internal.DebugStr(v.Eval().Syntax()))
+		got := fmt.Sprint(astinternal.DebugStr(v.Eval().Syntax()))
 		if got != tc.eval {
 			t.Errorf("got %v; want %v", got, tc.eval)
 		}
@@ -845,7 +845,7 @@ v: #X
 			t.Errorf("got %v; want %v", got, tc.raw)
 		}
 
-		got = fmt.Sprint(internal.DebugStr(v.Eval().Syntax()))
+		got = fmt.Sprint(astinternal.DebugStr(v.Eval().Syntax()))
 		if got != tc.eval {
 			t.Errorf("got %v; want %v", got, tc.eval)
 		}

diff --git a/encoding/jsonschema/decode_test.go b/encoding/jsonschema/decode_test.go
@@ -34,7 +34,7 @@ import (
 	"cuelang.org/go/cue/token"
 	"cuelang.org/go/encoding/json"
 	"cuelang.org/go/encoding/yaml"
-	"cuelang.org/go/internal"
+	"cuelang.org/go/internal/astinternal"
 	"cuelang.org/go/internal/cuetest"
 	_ "cuelang.org/go/pkg"
 )
@@ -182,5 +182,5 @@ func TestX(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	t.Fatal(internal.DebugStr(expr))
+	t.Fatal(astinternal.DebugStr(expr))
 }