[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute |
Date: |
Sun, 30 Nov 2008 15:36:09 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Thiemo Seufer wrote:
> Jan Kiszka wrote:
>> Avi Kivity wrote:
>>> Jan Kiszka wrote:
>>>>> Breaking the standard is what brings us the joys of recently (re)posted
>>>>> patch for NetBSD and [u]intXX fun.
>>>>>
>>>> I have no problem with calling it 'noreturn' instead.
>>>>
>>> That will break code that wants to use 'noreturn' as a local variable.
>>> I think ATTR_NORETURN, while a lot uglier, is safer.
>> Do you have such code already? Is it exported beyond qemu scope? Then
>> why not going for our own convention "'noreturn' is reserved as function
>> attribute"? (And yes, your macro is ugly :) ).
>
> "Macro names should be in upper case" is also a useful convention.
Generally yes. But there are exceptions when the macro is used in a
context where upper case disturbs the readability instead of improving
it. I would argue that this is the case here, but it's always a matter
of taste.
>
> FWIW, I agree with Stefan, there's currently not much need to isolate
> gcc-isms.
If everyone prefers having __attribute__ in the function prototypes
directly -- OK. All I want is to get rid of the warnings without
changing the code into the wrong direction.
Jan
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, (continued)
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Thiemo Seufer, 2008/11/28
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Avi Kivity, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Stefan Weil, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Laurent Desnogues, 2008/11/30
- [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Jan Kiszka, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Stefan Weil, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Andreas Färber, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, M. Warner Losh, 2008/11/30
- [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Jan Kiszka, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Thiemo Seufer, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute,
Jan Kiszka <=