Yep. Agreed. Totally redundant :-)

Cheers,

Colin

On 4 June 2016 at 05:11, DNF <oyv...@gmail.com> wrote:

> Your in-method is a bit odd:
>
> Base.in{T}(x::T, r::BasicInterval{T}) = (r.start <= x <= r.stop) ? true :
> false
>
> Why don't you just write
>
> Base.in{T}(x::T, r::BasicInterval{T}) = (r.start <= x <= r.stop)
> ?
>
> The extra stuff is redundant.
>

Reply via email to