Cache key in tapedfunction

[FredericWantiez]

Was looking at

Libtask.jl/src/tapedfunction.jl

Line 65 in 5ec9f2b

cache_key = (f, args_type...)

Shouldn't it be something like this instead ?

cache_key = (f, deepcopy_types, args_type...) 

Otherwise this is giving counterintuitive results:

struct Dummy 
    a::Array
end

function f(d::Dummy)
    produce(d.a)
end

d = Dummy([1])
t = TapedTask(f, d) # No Dummy in deepcopy_types
t2 = TapedTask(f, d; deepcopy_types=Dummy) # Dummy won't be part of the deepcopy_types

[FredericWantiez]

It's a weird edge case I agree

[yebai]

Thanks, @FredericWantiez. That's indeed a subtle bug - would you like to submit a PR?