cue/load: move no-package-args logic

This is a small change to move the "no arguments means the
current package" logic earlier in the `Instances` function
so that we can expand the arguments before creating
the loader.

Signed-off-by: Roger Peppe <rogpeppe@gmail.com>
Change-Id: I1d300699ddee5823501baa9ef82b4f54018e8d1a

cue/load/instances.go

@@ -56,6 +56,9 @@ func Instances(args []string, c *Config) []*build.Instance {
 		return []*build.Instance{c.newErrInstance(err)}
 	}
 	c = newC
+	if len(args) == 0 {
+		args = []string{"."}
+	}
 
 	// TODO: This requires packages to be placed before files. At some point this
 	// could be relaxed.
@@ -95,7 +98,7 @@ func Instances(args []string, c *Config) []*build.Instance {
 	}
 
 	a := []*build.Instance{}
-	if len(args) == 0 || i > 0 {
+	if len(pkgArgs) > 0 {
 		for _, m := range l.importPaths(pkgArgs) {
 			if m.Err != nil {
 				inst := c.newErrInstance(m.Err)

cue/load/loader_common.go

@@ -460,9 +460,6 @@ func warnUnmatched(matches []*match) {
 // command line. It canonicalizes the patterns but does not
 // evaluate any matches.
 func cleanPatterns(patterns []string) []string {
-	if len(patterns) == 0 {
-		return []string{"."}
-	}
 	var out []string
 	for _, a := range patterns {
 		// Arguments are supposed to be import paths, but