Subject: Re: [boost] Review Request: Boost.Itl; The Interval Template Library
From: Jeff Flinn (TriumphSprint2000_at_[hidden])
Date: 2009-10-01 10:46:19
Joachim Faulhaber wrote:
> 2009/10/1 Jeff Flinn <TriumphSprint2000_at_[hidden]>:
>> One more warning in interval.hpp, the enum BoundedTypes last value has an
>> inappropriate trailing ','.
> trailing ',' deleted ;-)
>> I should clarify that I still get compilation errors/warnings using
>> split_interval_map::operator+=, but rather I'm using the add method.
> I'm afraid this is more difficult. Currently I don't have the time
> (and compiler) to deal with this. But if you'd find a patch, I
> would be happy to integrate it.
It's not high on my list either, as add works. I was just wanting to be
more accurate with what was compilable. More importantly, I'd like to be
able to use std::inserter of std::back_inserter with std::copy or
std::transform to fill the split_interval_map from my domain data. I see
there is an insert method, but I'm not sure just how it differs from
add. I'd like to see the public interface paired down and just expose
the more std::map like interface.