svn commit: r317755 - head/sbin/ifconfig
Eric van Gyzen
eric at vangyzen.net
Wed May 3 19:48:55 UTC 2017
On 05/03/2017 14:38, Alan Somers wrote:
> On Wed, May 3, 2017 at 1:34 PM, Warner Losh <imp at bsdimp.com> wrote:
>> On Wed, May 3, 2017 at 1:32 PM, Alan Somers <asomers at freebsd.org> wrote:
>>> On Wed, May 3, 2017 at 12:16 PM, Ngie Cooper <yaneurabeya at gmail.com> wrote:
>>>>
>>>>> On May 3, 2017, at 10:21, Alan Somers <asomers at FreeBSD.org> wrote:
>>>>>
>>>>> Author: asomers
>>>>> Date: Wed May 3 17:21:01 2017
>>>>> New Revision: 317755
>>>>> URL: https://svnweb.freebsd.org/changeset/base/317755
>>>>>
>>>>> Log:
>>>>> Various Coverity fixes in ifconfig(8)
>>>>
>>>> ...
>>>>
>>>>> * Mark usage() as _Noreturn (1305806, 1305750)
>>>>
>>>> ...
>>>>
>>>>> -static void usage(void);
>>>>> +static void usage(void) _Noreturn;
>>>>
>>>> Hi Alan,
>>>> Please use __dead2 instead to be consistent with legacy use of similar gcc attributes.
>>>> Thanks,
>>>> -Ngie
>>>
>>> Why not use _Noreturn? It's standardized by C11, so tools understand
>>> it better than __dead2.
>>
>> Tools that can't understand #define __dead2 _Noreturn aren't worth supporting.
>>
>> Warner
>
> Some tools don't expand preprocessor macros. Like my editor, for
> example, which highlights _Noreturn as a keyword but not __dead2.
Please use _Noreturn, because it's standard. sys/cdefs.h already
defines it appropriately for C < C11.
Eric
More information about the svn-src-head
mailing list