dethrace-labs / dethrace

Reverse engineering the 1997 game "Carmageddon"
https://twitter.com/dethrace_labs
GNU General Public License v3.0
870 stars 46 forks source link

Big endian fixes - part 2 #405

Closed BSzili closed 1 month ago

BSzili commented 3 months ago

These were missing from the new platform code.

Link4Electronics commented 3 months ago

Hello, I cloned your main fork and tried to compile it on a PowerMac G5 running linux PPC64. During compilation it shows a lot of warnings like: warning: "BR_ENDIAN_BIG 0 : note: this is the location of the previous definition." It does compile either way, but when I try to run it gives this:


Caught SIGSEV


Stack trace: 0: BrPlaneEquation at /lib/BRender-v1.3.2/core/math/plane.c:35 1: PrepareFaceNormals at /lib/BRender-v1.3.2/core/v1db/prepmesh.c:355 2: BrModelUpdate at /lib/BRender-v1.3.2/core/v1db/prepmesh.c:1134 3: BrModelAdd at /lib/BRender-v1.3.2/core/v1db/regsupt.c:55 4: InitializeBRenderEnvironment at /src/DETHRACE/common/init.c:285 5: InitialiseApplication at /src/DETHRACE/common/init.c:377 6: InitialiseDeathRace at /src/DETHRACE/common/init.c:429 7: GameMain at /src/DETHRACE/common/main.c:122 8: original_main at /src/DETHRACE/pc-win95/win95sys.c:969 9: main at /src/DETHRACE/main.c:48 10: ?? ??:0

BSzili commented 3 months ago

Please don't try to build this yet for big endian, some of my are still local only.

BSzili commented 2 months ago

@madebr This one is good to merge, I meant the BRender repo that needs a couple header/build system changes for the BRENDIAN* macros to properly propagate. Sorry for the confusion.