[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: XTL list <firstname.lastname@example.org>
- Subject: QUESTIONS 2
- From: Angus Leeming <email@example.com>
- Date: Wed, 26 Apr 2000 12:58:54 +0100
- Delivered-To: mailing list firstname.lastname@example.org
- Mailing-List: contact email@example.com; run by ezmlm
Before I ask the questions, I'll sweeten you up by saying that I've got
GIOP_format working happily on the Alpha.
Josť> Yes. You're correct. req_align is certainly not "reader friendly" code. :-)
Josť> It might not even be the fastest possible, but as it is critical for
Josť> performance we should be very careful when changing it.
I have had to change req_align() to do so :-(
However, I think that the logic behind the old version was incorrect.
(Certainly, it didn't work on the Alpha!) The new version is no bigger, so
performance shouldn't change.
Ok, the questions:
There is a variable "bool endianess" in GIOP_format. What is its purpose?
I THINK that it exists so that data written by one machine can be read by
another. Is this correct?
If so, then "long" should be cast to "long long" for storage (in
GIOP_format) ALWAYS, because long is 64 bits on some machines and 32 bits on
If this isn't its purpose, then endianess is redundant I think. (And
consequently, all those calls to bswap_16() etc are unnecessary.)
Another, related question:
Is GIOP_format designed to import/export between different machines whilst
XDR_format is designed for one machine only?
If XDR_format is designed for possibly MANY machines then similar endianess
functionality should be put in XDR_format. It doesn't exist currently.
Furthermore, since the length of a word differs between machines, then the code
should use 64 bits as its storage size ALWAYS. Same reasoning as above. Sounds
horrible, doesn't it!!!
If XDR_format is designed for ONE machine only, then we can discard all that
bswap_32() etc stuff. The internal translations don't need it. This means that
those global functions can be sucked back into the class.
I'm willing to implement the fixes, but need the answers to do so...