boostorg / wave

Boost.org wave module
http://boost.org/libs/wave
21 stars 49 forks source link

Simplify allocator use via core/allocator_access #93

Closed glenfe closed 4 years ago

jefftrull commented 4 years ago

LGTM too but out of curiosity - what's the motivation for the use of allocator_XXX_type<A>::type as opposed to allocator_traits<A>::XXX_type as used in the standard?

glenfe commented 4 years ago

As we've found in the standards committee, allocator_traits as one big collection of traits is mistake. Users specialize it, and then we can't extend it without breaking user specializations. We realize that this (individual traits) is the way we should have had it since the beginning.