This is the mail archive of the newlib@sourceware.org mailing list for the newlib project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] libgloss/arm: Cast arguments to do_AngelSWI to int.

From: Nick Clifton <nickc at redhat dot com>
To: Paul Brook <paul at codesourcery dot com>
Cc: Kazu Hirata <kazu at codesourcery dot com>, newlib at sources dot redhat dot com
Date: Tue, 26 Dec 2006 09:27:50 +1300
Subject: Re: [patch] libgloss/arm: Cast arguments to do_AngelSWI to int.
References: <200612201047.kBKAllCm025497@sparrowhawk.codesourcery.com> <200612240151.40602.paul@codesourcery.com> <459029C3.1020004@redhat.com> <200612252012.50697.paul@codesourcery.com>

Hi Paul,

I'm not convinced by your argument. Aliasing int and void* via unions is a gcc extension. Casting a void* to an int is legal C.

Well, that shows how much I know. I had always thought that using unions was the correct, C way of doing things.

I withdraw my objections to Kazu's patch.

Cheers
  Nick

References:
- [patch] libgloss/arm: Cast arguments to do_AngelSWI to int.
  - From: Kazu Hirata
- Re: [patch] libgloss/arm: Cast arguments to do_AngelSWI to int.
  - From: Paul Brook
- Re: [patch] libgloss/arm: Cast arguments to do_AngelSWI to int.
  - From: Nick Clifton
- Re: [patch] libgloss/arm: Cast arguments to do_AngelSWI to int.
  - From: Paul Brook

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]