#1481 closed defect
BS1: .pal files have to be in the current directory
Reported by: | eriktorbjorn | Owned by: | SF/ender |
---|---|---|---|
Priority: | normal | Component: | Engine: Sword1 |
Version: | Keywords: | ||
Cc: | Game: | Broken Sword 1 |
Description
I'm sure this is a known bug, but just to make reasonably sure it isn't forgotten: The cutscene player for Broken Sword I requires the .pal files to be in the current directory, which is not good.
Assigning to Ender, since as far as I know he's the only one who knows what the official cutscene packages are going to look like.
Ticket imported from: #902154. Ticket imported from: bugs/1481.
Attachments (2)
Change History (17)
comment:1 by , 21 years ago
Owner: | set to |
---|
comment:2 by , 21 years ago
Owner: | changed from | to
---|
comment:3 by , 21 years ago
by , 21 years ago
Attachment: | bs-binary-palette.diff added |
---|
Patch against a February 22 branch snapshot
comment:4 by , 21 years ago
I don't have the final .pal files, so I can't do the conversion. But I used the attached tool to convert the first two .pal files of the experimental BS2 cutscenes, and it seemed to work fine for them.
The patch also includes the corresponding changes to the BS1 cutscene player, but these are completely untested. I haven't even tried to compile them. Caveat emptor.
comment:5 by , 21 years ago
Owner: | changed from | to
---|
comment:7 by , 21 years ago
By the way, in this patch / converter the 'cnt' parameter is stored as a 16-bit integer. It should be trivial to change this to a byte, since it's never larger than 256.
comment:8 by , 21 years ago
I don't have any BS1 .pal files at all, anyway. There ARE no final BS1 .pal files :)
If you have some feel free to share *g*
comment:10 by , 21 years ago
Cutscenes I have, .pal 8-bit palette files I don't. I assumed you had made some since you logged this bug :o)
comment:11 by , 21 years ago
No, on the few occasions where I've tested the BS1 cutscene code at all I've used files from the preliminary BS2 cutscene pack. That's when I noticed that the BS2 looks for .pal files in the game data directory, whereas BS1 looks in the current directory.
comment:12 by , 21 years ago
Status: | new → closed |
---|
comment:13 by , 21 years ago
Closing this, I'll commit the appropriate code when I update the cutscene packs on Monday/Tuesday.
comment:14 by , 21 years ago
Looks to me as if it was only committed to the branch. What about the trunk?
comment:15 by , 21 years ago
Was meaning to backport it, but I was a bit rushed for time. Feel free to do so, otherwise I'll try and look at it this weekend.
This is also true for BS2, and there is indeed a FIXME in there (or at least in the BS2 code) about this.
The .pal files need to be converted to a binary format so we can use the File class. When this happens it'll look in "all the usual places". If you want to do the conversion and bounce me over the new .pal files, that'd be cool. :)