Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PARALLACTION: Add custom Adlib driver for BRA. #281

Merged
merged 1 commit into from Sep 27, 2012

Conversation

@fuzzie
Copy link
Member

commented Sep 22, 2012

This isn't quite perfect, but it's a vast improvement over the current situation. :-)

Mostly based on https://github.com/peres/BRA_adlib_driver/ so many thanks to peres.

PARALLACTION: Add custom Adlib driver for BRA.
Thanks to peres for working out how it works in the original engine.
Also, fix the length of MIDI events so it works properly.

@ghost ghost assigned peres Sep 22, 2012

@digitall

This comment has been minimized.

Copy link
Member

commented Sep 22, 2012

Have tested with my copy of BRA/DOS/English and the music is infinitely improved. Though it may not be exactly correct, it matches that heard in playtest videos (probably DOSBox) i.e. https://www.youtube.com/watch?v=PCXW789PIy4

@lordhoto

This comment has been minimized.

Copy link
Member

commented Sep 23, 2012

I guess this is fine to merge.

fuzzie added a commit that referenced this pull request Sep 27, 2012
Merge pull request #281 from fuzzie/parallaction-adlib
PARALLACTION: Add custom Adlib driver for BRA.

@fuzzie fuzzie merged commit 73cb72e into scummvm:master Sep 27, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.