[cfe-commits] please review: fix for http://llvm.org/PR4407
Zhanyong Wan (λx.x x)
wan at google.com
Thu Oct 15 21:47:20 PDT 2009
Hi,
Can someone tell me what happens next? :-) Thanks,
2009/10/14 Zhanyong Wan (λx.x x) <wan at google.com>:
> Attached the updated patch, per Chris' comments. Please take another
> look. Thanks!
>
> 2009/10/14 Zhanyong Wan (λx.x x) <wan at google.com>:
>> Resending to cfe-commits. You can tell I'm doing this for the first time. :-)
>>
>> ---------- Forwarded message ----------
>> From: Zhanyong Wan (λx.x x) <wan at google.com>
>> Date: 2009/10/14
>> Subject: Re: please review: fix for http://llvm.org/PR4407
>> To: cfe-dev at cs.uiuc.edu
>>
>>
>> 2009/10/14 Zhanyong Wan (λx.x x) <wan at google.com>:
>>> Hi,
>>>
>>> I have a patch that fixes the first issue in http://llvm.org/PR4407 :
>>>
>>> Clang should warn on a case value that exceeds the range of the type in
>>> switch.
>>> const char ch = 'a';
>>> switch(ch) {
>>> case 1234: // expected-warning {{overflow converting case value}}
>>> break;
>>> }
>>>
>>> I uploaded the patch to http://codereview.appspot.com/130078/show.
>>
>> The URL doesn't include the period in the end:
>> http://codereview.appspot.com/130078/show
>>
>> Thanks,
>>
>>> Could someone review it?
>>>
>>> This is the first time I work on clang, so please let me know if I'm
>>> following the coding convention correctly. Thanks,
>>>
>>> --
>>> Zhanyong
>>>
>>
>>
>>
>> --
>> Zhanyong
>>
>
>
>
> --
> Zhanyong
>
--
Zhanyong
More information about the cfe-commits
mailing list