Opened 12 years ago

Closed 12 years ago

Last modified 11 months ago

#8747 closed patch

SWORD 1: inverse stereo for traffic in rue Jarry

Reported by: SF/richiefs Owned by: eriktorbjorn
Priority: normal Component: Engine: Sword1
Keywords: Cc:
Game: Broken Sword 1

Description

Here is a patch that implements the FR #1266599

Ticket imported from: #1833038. Ticket imported from: patches/852.

Attachments (3)

sword1-traffic-jarry.diff (819 bytes ) - added by SF/richiefs 12 years ago.
sword1-traffic-jarry.diff
sword1-traffic-jarry-ressources.diff (1008 bytes ) - added by SF/richiefs 12 years ago.
sword1-traffic-jarry-ressources.diff
fxlist.diff (717 bytes ) - added by eriktorbjorn 12 years ago.
Alternative patch

Download all attachments as: .zip

Change History (17)

by SF/richiefs, 12 years ago

Attachment: sword1-traffic-jarry.diff added

sword1-traffic-jarry.diff

comment:1 by SF/richiefs, 12 years ago

Link to the FR:
http://sourceforge.net/tracker/index.php?func=detail&aid=1739042&group_id=37116&atid=418823

comment:2 by SF/richiefs, 12 years ago

Good link...:
http://sourceforge.net/tracker/index.php?func=detail&aid=1266599&group_id=37116&atid=418823

comment:3 by fingolfin, 12 years ago

Owner: set to eriktorbjorn

comment:4 by fingolfin, 12 years ago

Torbjörn, could you take a look at this, if you have the time? Thanks!

comment:5 by eriktorbjorn, 12 years ago

It would probably make sense to use the FX_ constants for the sounds, as defined in swordres.h instead of hard-coding the values. But if it's simply that the "left-to-right" and "right-to-left" sound effects havebeen mixed up, wouldn't it be simpler to just fix _fxList[] in swordres.cpp? Or are they used elsewhere for something that's visible on screen?

by SF/richiefs, 12 years ago

sword1-traffic-jarry-ressources.diff

comment:6 by SF/richiefs, 12 years ago

I updated the patch. It doesn't have anymore hard coded values.
I have no idea if these samples are called elsewere.
File Added: sword1-traffic-jarry-ressources.diff

comment:7 by fingolfin, 12 years ago

Owner: changed from eriktorbjorn to lavosspawn

comment:8 by fingolfin, 12 years ago

Torbjörn, how about doing as you said and fix _fxList[], and we'll find out if that's wrong when somebody complains?

Maybe Robert also has something to add?

by eriktorbjorn, 12 years ago

Attachment: fxlist.diff added

Alternative patch

comment:9 by eriktorbjorn, 12 years ago

Here's what the alternative patch would look like. I have assumed that the symbolic name in _fxList[] is correct - for instance, "Truck passes, left to right" is called FX_HVYVEHR, because the truck is travelling to the "R"ight - and changed the #defines instead.

If it doesn't break any other scene, I think it's a much more elegant way of fixing the glitch.
File Added: fxlist.diff

comment:10 by fingolfin, 12 years ago

Go ahead and commit it, I'd say. And close both this and the FR.

comment:11 by fingolfin, 12 years ago

Owner: changed from lavosspawn to eriktorbjorn

comment:12 by eriktorbjorn, 12 years ago

I've committed my patch to the trunk. (Not to the branch - it's a feature request, not a bug.)

comment:13 by eriktorbjorn, 12 years ago

Status: newclosed

comment:14 by digitall, 11 months ago

Component: Engine: Sword1
Game: Broken Sword 1
Note: See TracTickets for help on using tickets.