Opened 19 years ago

Closed 19 years ago

Last modified 6 years ago

#8478 closed patch

BS2: Alignment fix, I hope

Reported by: eriktorbjorn Owned by: eriktorbjorn
Priority: normal Component: Engine: Sword2
Version: Keywords:
Cc: Game: Broken Sword 2

Description

This patch is a rather extensive rewrite to get rid of the reported alignment issues in the Broken Sword 2 engine.

I'm currently in the process of testing BS2 with it, and have played my way to the warehouses in Marseille without problem. However, it would be nice if someone could try it on big-endian, because there is a good chance I've introduced endian regressions.

And of course, it would be good to know if it actually fixes the aforementioned alignment issues.

Ticket imported from: #1341495. Ticket imported from: patches/583.

Attachments (1)

bs2-alignment.diff.gz (59.5 KB ) - added by eriktorbjorn 19 years ago.
Updated patch against an October 29 CVS snapshot

Download all attachments as: .zip

Change History (9)

comment:1 by eriktorbjorn, 19 years ago

Owner: set to SF/chrilith

by eriktorbjorn, 19 years ago

Attachment: bs2-alignment.diff.gz added

Updated patch against an October 29 CVS snapshot

comment:2 by eriktorbjorn, 19 years ago

Uploaded an updated patch. The first one wouldn't compile on big-endian.

I've now reached the native village without noticing any regressions.

comment:3 by SF/damienguard, 19 years ago

Tried the first few scenes, works just fine on 10.4.2. Also tested load/save and cutscenes all without problem.

comment:4 by SF/chrilith, 19 years ago

Just tested the BS2 demo on zodiac 68k and ARM mode, seems to work very well :)

great job !

comment:5 by eriktorbjorn, 19 years ago

Status: newclosed

comment:6 by eriktorbjorn, 19 years ago

Owner: changed from SF/chrilith to eriktorbjorn

comment:7 by eriktorbjorn, 19 years ago

I'm committing this, then, since the initial reports indicate that I haven't completely broken things. It won't be included in 0.8.0, but I imagine there'll be plenty of time to deal with any remaining regressions before 0.9.0.

comment:8 by digitall, 6 years ago

Component: Engine: Sword2
Game: Broken Sword 2
Note: See TracTickets for help on using tickets.