Re: [gtk-vnc-devel] [Patch] Not use range case for switch statement
- From: "Daniel P. Berrange" <berrange redhat com>
- To: Halton Huo <Halton Huo Sun COM>
- Cc: gtk-vnc-devel <gtk-vnc-devel lists sourceforge net>
- Subject: Re: [gtk-vnc-devel] [Patch] Not use range case for switch statement
- Date: Fri, 11 Jan 2008 03:39:07 +0000
On Fri, Jan 11, 2008 at 11:27:09AM +0800, Halton Huo wrote:
> Hi All,
>
> I reported a bug http://bugzilla.gnome.org/show_bug.cgi?id=508488
>
> It is because range case statement is not a C/C++ standard, seems only
> gcc have this extension now.
>
> I attach the fix in bug report, please review it.
ACK, the patch gets my vote. Much as I like some gcc extensions we should
aim to be standards compliant unless the code degrades gracefully.
Dan.
--
|=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=|
|=- Perl modules: http://search.cpan.org/~danberr/ -=|
|=- Projects: http://freshmeat.net/~danielpb/ -=|
|=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]