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 to include, rather than this one. Since there is no >> <erase_if> header that's wrong. I'd just omit the @headername. > > Done. >> >> This file doesn't really seem like a .tcc to me, it isn't providing >> definitions of templates declared elsewhere (specifically in an >> erase_if.h header). >> >> Maybe we want an experimental/bits/ directory for this sort of thing >> (which I could also use for the filesystem headers I'm about to >> commit) but in the meanwhile I think just experimental/erase_if.h is a >> better name. > > Done. (I didn't make experimental/bits - I'll move erase_if.h after you add > the bits directory). >> >> OK for trunk with those changes (remove make_array, rename erase_if.tcc) > > Rebuilt, retested and committed as 222630. > Altered patch attached.
Shouldn't the "one-liner" forwarding function templates be declared as inline? - Daniel
