Add Ref-like broadcast

[serenity4]

Currently, broadcasting over e.g. a Vector{<:Result} complains about Base.length(::Result) not defined, as Result is assumed to be an iterator type. Since Result is not an iterator, I believe it would make sense to adapt Base.broadcastable so that we can broadcast on Result containers.

I don't think this requires any tests, but I can add one if you prefer.

EDIT: my bad, broadcasting on a vector containing Results is actually fine. The problem happens when broadcasting with a Result not Refed. I still believe this to be nice to have, though.

[0x0f0f0f]

@serenity4 long time no see!

EDIT: my bad, broadcasting on a vector containing Results is actually fine. The problem happens when broadcasting with a Result not Refed. I still believe this to be nice to have, though.

Do you have an example?

[serenity4]

👋

Do you have an example?

Sure:

julia> using ResultTypes

julia> x = [Result(1), Result(2)]
2-element Vector{Result{Int64, Exception}}:
 Result(1)
 Result(2)

julia> unwrap.(x)
2-element Vector{Int64}:
 1
 2
 
julia> x = Result(1)
Result(1)

julia> unwrap.(x)
ERROR: MethodError: no method matching length(::Result{Int64, Exception})
The function `length` exists, but no method is defined for this combination of argument types.

[0x0f0f0f]

@serenity4 nice! I'll approve if you add a test (now maintainer of the pkg)

[serenity4]

Congrats! I was a bit worried that this package goes unmaintained, good to know it's not the case :)

I added the test.

[serenity4]

Thanks!