mvdan.cc / cue
this repo has no description
0
fork

Configure Feed

Select the types of activity you want to include in your feed.

internal/core/adt: remove unused parameters in Vertex.AddStruct

These have been unused since my simplification
in https://cuelang.org/cl/1226965.

Signed-off-by: Daniel Martí <mvdan@mvdan.cc>
Change-Id: I1ab396347fa573195302e5ca7ed39e0928cb4ff3
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1229432
TryBot-Result: CUEcueckoo <cueckoo@cuelang.org>
Unity-Result: CUE porcuepine <cue.porcuepine@gmail.com>
Reviewed-by: Matthew Sackman <matthew@cue.works>

Daniel Martí a8d5eef3 54e81b3c

+5 -5
+1 -1
internal/core/adt/composite.go
··· 1408 1408 v.Conjuncts = append(v.Conjuncts, c) 1409 1409 } 1410 1410 1411 - func (v *Vertex) AddStruct(s *StructLit, env *Environment, ci CloseInfo) { 1411 + func (v *Vertex) AddStruct(s *StructLit) { 1412 1412 for i, t := range v.Structs { 1413 1413 if t.StructLit == s { 1414 1414 v.Structs[i].Repeats++
+1 -1
internal/core/adt/comprehension.go
··· 233 233 isComprehension: true, 234 234 } 235 235 } 236 - node.AddStruct(st, env, ci) 236 + node.AddStruct(st) 237 237 switch { 238 238 case !ec.done: 239 239 ec.structs = append(ec.structs, st)
+1 -1
internal/core/adt/conjunct.go
··· 239 239 s.Init(n.ctx) 240 240 241 241 // TODO: do we still need to AddStruct? 242 - n.node.AddStruct(s, childEnv, ci) 242 + n.node.AddStruct(s) 243 243 244 244 // TODO(perf): precompile whether struct has embedding. 245 245 loop1:
+2 -2
internal/core/convert/go.go
··· 405 405 } 406 406 407 407 // There is no closedness or cycle info for Go structs, so we pass an empty CloseInfo. 408 - v.AddStruct(sl, env, adt.CloseInfo{}) 408 + v.AddStruct(sl) 409 409 v.SetValue(ctx, &adt.StructMarker{}) 410 410 v.ForceDone() 411 411 return v ··· 450 450 } 451 451 } 452 452 453 - v.AddStruct(obj, env, adt.CloseInfo{}) 453 + v.AddStruct(obj) 454 454 v.SetValue(ctx, &adt.StructMarker{}) 455 455 v.ForceDone() 456 456 return v