Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: syssrc/sys/kern



On Thu, 31 Oct 2002, Christos Zoulas wrote:

>
> Module Name:  syssrc
> Committed By: christos
> Date:         Thu Oct 31 02:40:42 UTC 2002
>
> Modified Files:
>       syssrc/sys/kern: exec_macho.c
>
> Log Message:
> XXX: be32toh on powerpc does not cast to int32_t so passing u_long to
> be32toh produces an unsigned long result, causing a printf argument
> mismatch. This is the wrong fix, but I am not going to change the
> powerpc macros; fix the powerpc macros and revert my change.

(not specifically directted at Christos)

Uhm, why are we passing "long" variables to be32toh? If these things are
going to be 32-bit quantities, we should make them 32-bit quantities. Not
longs.

For instance, what happens when we have 64-bit powerpc, and want to run
tthe macho code?

Take care,

Bill




Home | Main Index | Thread Index | Old Index