Setfield.jl icon indicating copy to clipboard operation
Setfield.jl copied to clipboard

Drop optional dependency

Open ParadaCarleton opened this issue 2 years ago • 2 comments

The single line:

setindex(a::StaticArrays.StaticArray, args...) =
            Base.setindex(a, args...)

Makes up most of the package's compile time; it could easily be moved to StaticArrays.jl, avoiding the cost of having to use Requires.jl.

ParadaCarleton avatar Oct 22 '21 22:10 ParadaCarleton

I would love to drop this, but it is not as easy as you think. It would require a change in base, see https://github.com/JuliaLang/julia/pull/33495 If you want this, it would be awesome if you could revive that Base PR.

jw3126 avatar Oct 24 '21 07:10 jw3126

mostly solved by #174

jw3126 avatar Aug 05 '22 14:08 jw3126