[cfe-commits] [PATCH] Begin to implement the C++0x narrowing error (issue4698047)

[Jeffrey Yasskin]

On Mon, Jul 18, 2011 at 10:05 AM, Jeffrey Yasskin <jyasskin at google.com> wrote:
> On Mon, Jul 18, 2011 at 9:59 AM, Sean Hunt <scshunt at csclub.uwaterloo.ca> wrote:
>> On 07/18/11 09:54, Jeffrey Yasskin wrote:
>>> I've attached the latest version of this patch.
>>
>> Perhaps it would be best to split the narrowing conversion and the C++0x
>> compat refactor into separate patches?
>
> You mean I should set the new warning to DefaultIgnore and revert the
> unittest fixes in this patch, and turn on the warning in a separate
> patch?

Here's a version with the warning off by default.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cxx0x_narrowing_error.diff
Type: application/octet-stream
Size: 24892 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110718/8175cb7c/attachment.obj>