[cfe-commits] please review: fix for http://llvm.org/PR4407
Zhanyong Wan (λx.x x)
wan at google.com
Thu Oct 15 23:12:18 PDT 2009
On Thursday, October 15, 2009, Chris Lattner <clattner at apple.com> wrote:
>
> On Oct 15, 2009, at 9:47 PM, Zhanyong Wan (λx.x x) wrote:
>
>
> Hi,
>
> Can someone tell me what happens next? :-) Thanks,
>
>
> Hi Zhanyong,
>
> Did you see my comments here:
> http://lists.cs.uiuc.edu/pipermail/cfe-dev/2009-October/006686.html
Yes.
>
> Do you have an updated patch?
Yes. The one attached on this thread is the updated version.
> Also, as a general comment, sometimes it takes a while to review patches,
I understand. I was trying to make sure that I have done my part
correctly and you are not waiting on me. Thanks,
>
> -Chris
>
>
>
> 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
>
>
>
--
Zhanyong
More information about the cfe-commits
mailing list