
Marcin Kalicinski wrote:
Even boost guidelines are pretty clear on that, I should not use other libs unless they provide critical functionality.
Where did you find that guideline?
http://boost.org/more/library_reuse.htm
"A Boost library should use other Boost Libraries or the C++ Standard Library, but only when the benefits outweigh the costs."
Perhaps "critical functionality" was an overstatement, but anyway my point remains correct.
OTOH, that quote can be interpreted as just about anything :-) I think it is important to focus on the interface in the review, but I also see several benefits of an implementation that builds on Boost.MultiIndex: - fewer bugs like the one Joachin found - better space efficiency - exception-safety guarantees are immidiately full-filled (I haven't looked, but I suspect that there are several bugs in this area) Also, Joachin had a *very* interesting idea about flattening quereries like "foo.*" -Thorsten