Re: [Flightgear-devel] new multiplayer patch

2005-09-22 Thread AJ MacLeod
On Wednesday 21 September 2005 19:54, Oliver Schroeder wrote: The problem lies in XDR_encode_double() and XDR_decode_double(). Making all arguments const did the trick. You can find my updated version here: http://www.o-schroeder.de/fg_server/tinyxdr.tgz please test it. I can confirm that it

Re: [Flightgear-devel] new multiplayer patch

2005-09-22 Thread Erik Hofman
Oliver Schroeder wrote: The problem lies in XDR_encode_double() and XDR_decode_double(). Making all arguments const did the trick. You can find my updated version here: http://www.o-schroeder.de/fg_server/tinyxdr.tgz It's committed. Erik ___

RE: [Flightgear-devel] new multiplayer patch

2005-09-22 Thread Vivian Meazza
Erik Hofman Oliver Schroeder wrote: The problem lies in XDR_encode_double() and XDR_decode_double(). Making all arguments const did the trick. You can find my updated version here: http://www.o-schroeder.de/fg_server/tinyxdr.tgz It's committed. Erik That seems to break

Re: [Flightgear-devel] new multiplayer patch

2005-09-22 Thread Erik Hofman
Vivian Meazza wrote: swap_test.cpp:12: error: invalid conversion from `uint32_t*' to `unsigned int*' Ok, this is fixed. Erik ___ Flightgear-devel mailing list Flightgear-devel@flightgear.org

Re: [Flightgear-devel] new multiplayer patch

2005-09-21 Thread Erik Hofman
AJ MacLeod wrote: On Tuesday 20 September 2005 14:57, Erik Hofman wrote: Hmm, that should be correct. What does the utility output if you include byteswap.h instead off our own functions? Swapping #include byteswap.h instead of tiny_xdr.hpp I get exactly the same result here.. That's odd.

Re: [Flightgear-devel] new multiplayer patch

2005-09-21 Thread Erik Hofman
Erik Hofman wrote: That's odd. This *should* indicate that it still would work with our own definition of these functions ??!? I've updated the code slightly to swap two 32-bit bytes on 32-bit hardware. Could you test it again? Erik ___

Re: [Flightgear-devel] new multiplayer patch

2005-09-21 Thread AJ MacLeod
On Wednesday 21 September 2005 10:46, Erik Hofman wrote: That's odd. This *should* indicate that it still would work with our own definition of these functions ??!? I should note here that I'm a really only a simple sysadmin and not a programmer, so I'm saying nothing as to what should work

Re: [Flightgear-devel] new multiplayer patch

2005-09-21 Thread AJ MacLeod
Sorry about that - I was too quick to test and didn't read the cvs log carefully enough :-) Here's the output of your swap test UI32: (normal) 1234567 UI32: (swapped) 67452301 UI64: (normal) 123456789abcdef UI64: (swapped) efcdab8967452301 AJ ___

Re: [Flightgear-devel] new multiplayer patch

2005-09-21 Thread Erik Hofman
AJ MacLeod wrote: Certainly... Unfortunately it appears that the problem (for whatever reason) is still there... http://www.adeptopensource.co.uk/personal/fg/server_map_screenshot.png Hmm, this looks like a signed/unsigned mismatch. Erik ___

Re: [Flightgear-devel] new multiplayer patch

2005-09-21 Thread Arnt Karlsen
On Wed, 21 Sep 2005 13:21:27 +0200, Erik wrote in message [EMAIL PROTECTED]: AJ MacLeod wrote: Certainly... Unfortunately it appears that the problem (for whatever reason) is still there... http://www.adeptopensource.co.uk/personal/fg/server_map_screenshot.png Hmm, this looks

Re: [Flightgear-devel] new multiplayer patch

2005-09-21 Thread Oliver Schroeder
Am Mittwoch 21 September 2005 10:23 schrieb Erik Hofman: AJ MacLeod wrote: Swapping #include byteswap.h instead of tiny_xdr.hpp I get exactly the same result here.. That's odd. This *should* indicate that it still would work with our own definition of these functions ??!? I'm lost here

Re: [Flightgear-devel] new multiplayer patch

2005-09-20 Thread Oliver Schroeder
Hi Erik, Erik Hofman wrote: Secondly I've updated the code to use the ulEndianSwap() function instead of defining our own (my system doesn't even define byteswap.h). There's something really odd with byteswapping. Using your modified version does not work. Using my own byteswap functions

Re: [Flightgear-devel] new multiplayer patch

2005-09-20 Thread Erik Hofman
Oliver Schroeder wrote: There's something really odd with byteswapping. Using your modified version does not work. Using my own byteswap functions does not work. Using byteswap.h (ie. remove both bswap-functions and #include byteswap.h instead) does work. So far I did not find out what it is.

Re: [Flightgear-devel] new multiplayer patch

2005-09-20 Thread AJ MacLeod
On Tuesday 20 September 2005 10:36, Erik Hofman wrote: I've added a byte swap test utility in the MultiPlayer directory so you can easily test if it really makes a difference. On my machine all (should) work(s) fine. Are you sure? It doesn't here since yesterday (Linux x86. ) FG runs OK, and

Re: [Flightgear-devel] new multiplayer patch

2005-09-20 Thread Erik Hofman
AJ MacLeod wrote: The byte swap test utility you added today gives this; UI32: (normal) ffaaccee UI32: (swapped) eeccaaff UI64: (normal) bbaaddffaaccee UI64: (swapped) eeccaaffddaabb00 Hmm, that should be correct. What does the utility output if you include byteswap.h instead off our own

Re: [Flightgear-devel] new multiplayer patch

2005-09-20 Thread AJ MacLeod
On Tuesday 20 September 2005 14:57, Erik Hofman wrote: Hmm, that should be correct. What does the utility output if you include byteswap.h instead off our own functions? Swapping #include byteswap.h instead of tiny_xdr.hpp I get exactly the same result here.. AJ

[Flightgear-devel] new multiplayer patch

2005-09-18 Thread Oliver Schroeder
Hi list. I have prepared a new patch for multiplayer, which fixes endianess issues with multiplayer code. It's basically identical to the patch I sent before my vacation, but contains minor fixes. Multiplayer should now be working under all unix-like environments and windows native. The basic

Re: [Flightgear-devel] new multiplayer patch

2005-09-18 Thread Erik Hofman
Oliver Schroeder wrote: Hi list. I have prepared a new patch for multiplayer, which fixes endianess issues with multiplayer code. It's basically identical to the patch I sent before my vacation, but contains minor fixes. Multiplayer should now be working under all unix-like environments and