Remove VarInfo(VarInfo, params) (#870)

penelopeysm · web-flow · commit cc5e581ee004 · 2025-03-28T17:04:32.000Z
diff --git a/HISTORY.md b/HISTORY.md
@@ -4,6 +4,10 @@
 
 **Breaking changes**
 
+### VarInfo constructor
+
+`VarInfo(vi::VarInfo, values)` has been removed. You can replace this directly with `unflatten(vi, values)` instead.
+
 ### VarName prefixing behaviour
 
 The way in which VarNames in submodels are prefixed has been changed.
diff --git a/src/varinfo.jl b/src/varinfo.jl
@@ -100,8 +100,6 @@ const TypedVarInfo = VarInfo{<:NamedTuple}
 const VarInfoOrThreadSafeVarInfo{Tmeta} = Union{
     VarInfo{Tmeta},ThreadSafeVarInfo{<:VarInfo{Tmeta}}
 }
-# TODO: Remove this
-@deprecate VarInfo(vi::VarInfo, x::AbstractVector) unflatten(vi, x)
 
 # NOTE: This is kind of weird, but it effectively preserves the "old"
 # behavior where we're allowed to call `link!` on the same `VarInfo`

Original file line number	Diff line number	Diff line change
`@@ -100,8 +100,6 @@ const TypedVarInfo = VarInfo{<:NamedTuple}`
`100`	`100`	`const VarInfoOrThreadSafeVarInfo{Tmeta} = Union{`
`101`	`101`	`VarInfo{Tmeta},ThreadSafeVarInfo{<:VarInfo{Tmeta}}`
`102`	`102`	`}`
`103`		`-# TODO: Remove this`
`104`		`-@deprecate VarInfo(vi::VarInfo, x::AbstractVector) unflatten(vi, x)`
`105`	`103`
`106`	`104`	`# NOTE: This is kind of weird, but it effectively preserves the "old"`
`107`	`105`	# behavior where we're allowed to call `link!` on the same `VarInfo`