Re: CVS commit: othersrc/external/bsd/arfe

2015-12-05 Thread Roy Marples
Hi Thomas

On 05/12/2015 08:47, Thomas Klausner wrote:
> Module Name:  othersrc
> Committed By: wiz
> Date: Sat Dec  5 08:47:24 UTC 2015
> 
> Modified Files:
>   othersrc/external/bsd/arfe/dt: dt.c
>   othersrc/external/bsd/arfe/it: it.c
>   othersrc/external/bsd/arfe/tt: tt.c
> 
> Log Message:
> Sprinkle __attribute__((__noreturn__)) to fix build on 7.99.23 with clang.

Why don't you use __dead as defined by sys/cdefs.h?

Roy


Re: CVS commit: othersrc/external/bsd/arfe

2015-12-05 Thread David Young
On Sat, Dec 05, 2015 at 07:43:58PM +, Roy Marples wrote:
> Hi Thomas
> 
> On 05/12/2015 08:47, Thomas Klausner wrote:
> > Module Name:othersrc
> > Committed By:   wiz
> > Date:   Sat Dec  5 08:47:24 UTC 2015
> > 
> > Modified Files:
> > othersrc/external/bsd/arfe/dt: dt.c
> > othersrc/external/bsd/arfe/it: it.c
> > othersrc/external/bsd/arfe/tt: tt.c
> > 
> > Log Message:
> > Sprinkle __attribute__((__noreturn__)) to fix build on 7.99.23 with clang.
> 
> Why don't you use __dead as defined by sys/cdefs.h?

I could not remember that we called it __dead on NetBSD.  Maybe Thomas
did not remember, either.  I will change it to __dead in my next round
of commits.

Dave

-- 
David Young
dyo...@pobox.comUrbana, IL(217) 721-9981