Re: [PATCH] [libstdc++] Add uniform container erasure.

2015-05-01 Thread Jonathan Wakely
On 01/05/15 07:01 -0400, Ed Smith-Rowland wrote: On 04/30/2015 02:01 PM, Daniel Krügler wrote: Shouldn't the "one-liner" forwarding function templates be declared as inline? - Daniel You are right. This builds and tests clean on x86_64-linux. OK? Yes OK, thanks.

Re: [PATCH] [libstdc++] Add uniform container erasure.

2015-05-01 Thread Ed Smith-Rowland
On 04/30/2015 02:01 PM, Daniel Krügler wrote: Shouldn't the "one-liner" forwarding function templates be declared as inline? - Daniel You are right. This builds and tests clean on x86_64-linux. OK? 2015-05-01 Edward Smith-Rowland <3dw...@verizon.net> Inline one-line erasure dispat

Re: [PATCH] [libstdc++] Add uniform container erasure.

2015-04-30 Thread Daniel Krügler
2015-04-30 16:44 GMT+02:00 Ed Smith-Rowland <3dw...@verizon.net>: > >> And make_array, which isn't in the working paper yet, so I'd prefer to >> leave that part out for now. > > D'oh! Sorry about that.. Removed. >> >> The Doxygen @headername command tells users which header they are >> supposed t

Re: [PATCH] [libstdc++] Add uniform container erasure.

2015-04-30 Thread Ed Smith-Rowland
And make_array, which isn't in the working paper yet, so I'd prefer to leave that part out for now. D'oh! Sorry about that.. Removed. The Doxygen @headername command tells users which header they are supposed to include, rather than this one. Since there is no header that's wrong. I'd just

Re: [PATCH] [libstdc++] Add uniform container erasure.

2015-04-30 Thread Jonathan Wakely
On 30/04/15 07:33 -0400, Ed Smith-Rowland wrote: This has been in me tree for a good while. It is fairly simple and adds C++ experimental container erasure. And make_array, which isn't in the working paper yet, so I'd prefer to leave that part out for now. Index: include/experimental/erase_

[PATCH] [libstdc++] Add uniform container erasure.

2015-04-30 Thread Ed Smith-Rowland
This has been in me tree for a good while. It is fairly simple and adds C++ experimental container erasure. Builds and tests cleanly on x86_64-linux. OK? Index: include/Makefile.am === --- include/Makefile.am (revision 222573) +++