[PATCH] D16605: Implement `std::experimental::ostream_joiner`

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 27 15:59:50 PST 2016


EricWF added inline comments.

================
Comment at: include/experimental/iterator:85
@@ +84,3 @@
+    template<typename _Tp>
+    ostream_joiner& operator=(const _Tp& __v)
+    {
----------------
EricWF wrote:
> People are going to try and use this as an assignment operator. We either need to static assert in the body, or disable the operator entirely.
> I'm not sure which is better. It depends on if we want a generated operator=. 
Disregard the comment above. It's not possible to call this with an instance of `ostream_joiner`. I thought it might be possible like it is with constructors.


http://reviews.llvm.org/D16605





More information about the cfe-commits mailing list