This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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



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.

Thanks,

Ed

Attachment: CL_erasure
Description: Text document

Attachment: patch_erasure
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]