Skip to content

Commit

Permalink
cty: Remove already-fixed FIXME for SetVal
Browse files Browse the repository at this point in the history
The immediately preceding code already unmarks the values and applies
the marking to the set, so this comment and panic are out of date.
  • Loading branch information
alisdair authored and apparentlymart committed Sep 23, 2020
1 parent 22bb512 commit 36785d4
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions cty/value_init.go
Original file line number Diff line number Diff line change
Expand Up @@ -247,11 +247,6 @@ func SetVal(vals []Value) Value {
val = unmarkedVal
markSets = append(markSets, marks)
}
if val.ContainsMarked() {
// FIXME: Allow this, but unmark the values and apply the
// marking to the set itself instead.
panic("set cannot contain marked values")
}
if elementType == DynamicPseudoType {
elementType = val.ty
} else if val.ty != DynamicPseudoType && !elementType.Equals(val.ty) {
Expand Down

0 comments on commit 36785d4

Please sign in to comment.