#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)
Change History (9)
comment:1 by , 15 years ago
| Owner: | set to |
|---|
by , 15 years ago
| Attachment: | bs2-alignment.diff.gz added |
|---|
comment:2 by , 15 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 , 15 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 , 15 years ago
Just tested the BS2 demo on zodiac 68k and ARM mode, seems to work very well :)
great job !
comment:5 by , 15 years ago
| Status: | new → closed |
|---|
comment:6 by , 15 years ago
| Owner: | changed from to |
|---|
comment:7 by , 15 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 , 2 years ago
| Component: | → Engine: Sword2 |
|---|---|
| Game: | → Broken Sword 2 |
Updated patch against an October 29 CVS snapshot