[PATCH] [libcxx] Fix use of operator comma where the types can be user defined

Eric Fiselier eric at efcs.ca
Thu Oct 23 12:46:36 PDT 2014


I think that it's better to make the call to the comma operator a compile failure as opposed to a runtime error. It's easier to track down and doesn't require 100% test coverage to find all comma operator uses. @Marshell, Please let me know if you have an objection.

http://reviews.llvm.org/D5929

Files:
  include/algorithm
  include/deque
  include/forward_list
  include/locale
  include/numeric
  include/string
  include/utility
  test/containers/sequences/forwardlist/forwardlist.cons/assign_range.pass.cpp
  test/support/test_iterators.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5929.15342.patch
Type: text/x-patch
Size: 15949 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141023/e6fc121d/attachment.bin>


More information about the cfe-commits mailing list