Subject: Re: [boost] [CallableTraits] The formal review begins today
From: Gavin Lambert (gavinl_at_[hidden])
Date: 2017-04-05 06:11:02
On 5/04/2017 16:13, Barrett Adair wrote:
> On Tue, Apr 4, 2017 at 11:52 AM, Klemens Morgenstern wrote:
>> Additionally I felt, that there are a few to many algorithms, I would've
>> expected the library to only provide an argument tuple so I can manipulate
>> that with fusion, hana or whatever. Now I guess tha variadic C parameter is
>> the reason, but that can only be pushed to the end anyway. So I'm not sure
>> why the library would've just go with std::tuple and let some other tool do
>> the rest.
> Good point. These were some of the last features I added to the library, and
> perhaps the most indulgent. I'm not opposed to removing them. The presence
> of these algorithms make the preprocessed header length quite large, which
> is my main gripe with my implementation. I'd hoped to merge the "cleanup"
> branch before the review, which is more optimized for this, but it
> still generates
> nearly 20k lines of source code with GCC 6.3/c++1z (before counting std
> headers). Removing the algorithms will yield a very large reduction in
> code size.
Perhaps add some example documentation for equivalent usage of those
other libraries for common usage patterns with CallableTraits, for the
people who want to use CallableTraits but are less familiar with the
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk