[PATCH 5/5] Add abort(3) to the end of FatalError in case one of DDX functions fails to abort
Matt Turner
mattst88 at gmail.com
Wed May 12 13:40:54 PDT 2010
On Wed, May 12, 2010 at 4:33 PM, Mikhail Gusarov
<dottedmag at dottedmag.net> wrote:
> Also it makes gcc happier.
>
> Signed-off-by: Mikhail Gusarov <dottedmag at dottedmag.net>
> ---
> os/log.c | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/os/log.c b/os/log.c
> index 0781659..a482616 100644
> --- a/os/log.c
> +++ b/os/log.c
> @@ -545,6 +545,10 @@ FatalError(const char *f, ...)
> AbortServer();
> } else
> OsAbort();
> + /*
> + * OsVendorFatalError, AbortServer or OsAbort failed to abort.
> + */
> + abort();
> /*NOTREACHED*/
> }
>
> --
> 1.7.1
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>
I think this would be a good place to use gcc-4.5's __builtin_unreachable.
Matt
More information about the xorg-devel
mailing list