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 other libraries?