|
Boost : |
Subject: Re: [boost] [xint] Boost.XInt formal review
From: Christopher Jefferson (chris_at_[hidden])
Date: 2011-03-03 13:30:05
On 3 Mar 2011, at 17:07, Jeffrey Lee Hellrung, Jr. wrote:
>
> You can't assume that everyone will be satisfied with your integer_t data structure. Someone may want a data structure that uses the small-object optimization, because they have a lot of tiny integers running around and occasionally have large ones. Or they may want to use a std::deque(-like) underlying storage. I don't know. But I do know that your library will be strictly more usable if you decouple the algorithms from the integer_t data structure.
>
> I'm hesitant to vote for acceptance unless this decoupling were effected; your mileage with other boosters will obviously vary, but I gather Mathias is of the same opinion.
My biggest concern with decoupling is that introduces a much larger public API. At the moment it would be possible to make very large changes to the internals of xint without changing the public interface. Also, without multiple different data structures to store the internal data, it is not clear what a clean decoupling would look like.
Decoupling would make evolution of the library much more difficult, whereas decoupling later would be easier.
Chris
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk