[cfe-commits] [Patch][Review] Tests for explicit conversion operators

[Jonathan Sauer]

Hello,

> as promised in an earlier mail, I wrote some more tests for C++0x's explicit conversion operators (and
> found a clang bug along the way, PR10453). I think they are complete now (except for the interaction with
> constexpr, which is not implemented yet as far as I know).
> 
> I tried to follow the other tests, although I'm not sure about two points:
> 
> 1. LLVM's coding guidelines state that lines must remain below 80 characters. This does not seem to be
>   followed in clang's tests, so I didn't, either.
> 2. I added the FDIS section numbers corresponding to the tests as comments which I think makes following
>   them easier. OTOH, they will most likely get out of sync with C++16.

Attached is a new patch (which replaces the previous one) which both fixes the above bug PR10453 and
contains the (updated) test. I ran the clang test suite on my local machine, and it passed.

Please let me know what you think!


Jonathan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PR10453.diff
Type: application/octet-stream
Size: 6089 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110723/c25d5102/attachment.obj>
-------------- next part --------------