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

[Jeffrey Yasskin]

On Mon, Jul 18, 2011 at 10:48 AM, Jeffrey Yasskin <jyasskin at google.com> wrote:
> 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.

Here's another version with all the previously-committed pieces
excluded, and with the warning off by default. The latest version is
also still at http://codereview.appspot.com/4698047/.

Jeffrey
-------------- 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/20110721/fb4500ff/attachment.obj>