Updates and improvements to comments
This commit is contained in:
parent
8d8389da74
commit
20ee878d0e
|
@ -157,7 +157,11 @@ type ChangeSrc struct {
|
||||||
// storage.
|
// storage.
|
||||||
Before, After DynamicValue
|
Before, After DynamicValue
|
||||||
|
|
||||||
// Marked Paths
|
// BeforeValMarks and AfterValMarks are stored path+mark combinations
|
||||||
|
// that might be discovered when encoding a change. Marks are removed
|
||||||
|
// to enable encoding (marked values cannot be marshalled), and so storing
|
||||||
|
// the path+mark combinations allow us to re-mark the value later
|
||||||
|
// when, for example, displaying the diff to the UI.
|
||||||
BeforeValMarks, AfterValMarks []cty.PathValueMarks
|
BeforeValMarks, AfterValMarks []cty.PathValueMarks
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -148,8 +148,7 @@ func (n *EvalDiff) Eval(ctx EvalContext) (interface{}, error) {
|
||||||
var unmarkedPaths []cty.PathValueMarks
|
var unmarkedPaths []cty.PathValueMarks
|
||||||
if configVal.ContainsMarked() {
|
if configVal.ContainsMarked() {
|
||||||
// store the marked values so we can re-mark them later after
|
// store the marked values so we can re-mark them later after
|
||||||
// we've sent things over the wire. Right now this stores
|
// we've sent things over the wire.
|
||||||
// one path for proof of concept, but we should store multiple
|
|
||||||
unmarkedConfigVal, unmarkedPaths = configVal.UnmarkDeepWithPaths()
|
unmarkedConfigVal, unmarkedPaths = configVal.UnmarkDeepWithPaths()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue