[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/19] Fix call of function with qualified void return type
From: |
Marin Ramesa |
Subject: |
Re: [PATCH 03/19] Fix call of function with qualified void return type |
Date: |
Thu, 21 Nov 2013 08:26:05 +0100 |
On 21.11.2013 00:28:17, Guillem Jover wrote:
> On Wed, 2013-11-20 at 21:33:54 +0100, Marin Ramesa wrote:
> > On 20.11.2013 21:18:14, Guillem Jover wrote:
> > > > diff --git a/i386/i386/setjmp.h b/i386/i386/setjmp.h
> > > > index 667eecf..08626e3 100644
> > > > --- a/i386/i386/setjmp.h
> > > > +++ b/i386/i386/setjmp.h
> > > > @@ -36,7 +36,7 @@ typedef struct jmp_buf {
> > > > extern int _setjmp(jmp_buf_t*);
> > > >
> > > > #ifdef __GNUC__
> > > > -extern __volatile__ void _longjmp(jmp_buf_t*, int);
> > > > +extern /*__volatile__*/ void _longjmp(jmp_buf_t*, int);
> > > > #endif
> > > >
> > > > #endif /* _I386_SETJMP_H_ */
> > >
> > > Why not just remove it?
> >
> > Yes. It might be better to remove it. _longjmp is called only once.
>
> Just to be clear, I meant removing __volatile__ (instead of
> commenting it out), not removing the declaration.
Yes, it's been removed. Thanks.
[PATCH 04/19] ddb/db_examine.c: fix signedness, Marin Ramesa, 2013/11/20
[PATCH 05/19] ddb/db_expr.h: remove (db_addr_t *) argument name, Marin Ramesa, 2013/11/20
- Re: [PATCH 05/19] ddb/db_expr.h: remove (db_addr_t *) argument name, Guillem Jover, 2013/11/20
- Re: [PATCH 05/19] ddb/db_expr.h: remove (db_addr_t *) argument name, Marin Ramesa, 2013/11/20
- Re: [PATCH 05/19] ddb/db_expr.h: remove (db_addr_t *) argument name, Samuel Thibault, 2013/11/20
- Re: [PATCH 05/19] ddb/db_expr.h: remove (db_addr_t *) argument name, Marin Ramesa, 2013/11/20
- Re: [PATCH 05/19] ddb/db_expr.h: remove (db_addr_t *) argument name, Samuel Thibault, 2013/11/20
- Re: [PATCH 05/19] ddb/db_expr.h: remove (db_addr_t *) argument name, Marin Ramesa, 2013/11/20
[PATCH 06/19] ddb: fix implicit declaration of function, Marin Ramesa, 2013/11/20
[PATCH 07/19] ddb/db_run.c: remove set but unused variable, Marin Ramesa, 2013/11/20