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. >