Boost logo

Boost :

Subject: Re: [boost] [Review] ITL review starts today, February 18th
From: Joachim Faulhaber (afojgo_at_[hidden])
Date: 2010-02-28 15:06:26


2010/2/28 Barend Gehrels <barend_at_[hidden]>

> Hi Daniel,
>
>
>
> For sets (o.a. interval_set), the following operators are appropriate,
>>> as also used in Boost.dynamic_bitset:
>>> &= for intersection
>>> |= for union
>>> ^= for symmetric difference
>>> -= for difference
>>> So this is how it is implemented but I would remove += as a synonym for
>>> |=
>>>
>>> For interval_map, it is currently much less clear to me now. The
>>> operators do two things at the same time, which is confusing me. ...
>>>
>>>
>>
>>
>> I won't be able to review ITL, but I would like to comment that I
>> think that this issue of operators needs to be resolved before ITL is
>> included in Boost.
>>
>>
> Since I wrote that above, I'll react on this. I didn't intend such a
> reaction in my review. The ITL library is excellent, the thing (and the only
> thing I found) which was not logical (in my opinion, maybe Joachim will
> explain it all) is that the -= operator is not the set theoretic difference
> but the mathematical subtraction of values. I really don't think that this
> is a major concern.
>
>
actually the semantics of this generalized addition, subtraction and
intersection (symmetric difference is induced), is at the core of the whole
library. I will happily take the opportunity to explain .... stay tuned ;-)

Joachim


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk