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

Set sound for reinforce wall #40

Merged
merged 5 commits into from Aug 17, 2018
Merged

Set sound for reinforce wall #40

merged 5 commits into from Aug 17, 2018

Conversation

Loobinex
Copy link
Owner

The game tried to play the non-existing sound file 172 when imps would
reinforce walls. This gave errors, and made no sound.
Configured sound 63 to be used instead.

The game tried to play the non-existing sound file 172 when imps would
reinforce walls. This gave errors, and made no sound.
Configured sound 63 to be used instead.
@Loobinex
Copy link
Owner Author

Loobinex commented Aug 14, 2018

@spartahawk and @OrsonMaxwell try this build, have imps reinforce some walls and let me know if you prefer it with this sound, or quit like it was.
If you prefer it without sound, I can easily remove the line entirely and get rid of the pointless log messages.

If you have another sound in mind, feel free to share. (Use DKSound from the DKTools to browse the sounds).

The game tried to play sound files 80-82 that do not exist in the
sound.dat
@Loobinex
Copy link
Owner Author

Loobinex commented Aug 14, 2018

I also made a new sound.dat, that includes sounds 80-82 for chickens hatching. So this pull is enough to consider #39 fixed.

@Loobinex
Copy link
Owner Author

Received positive feedback on this from @OrsonMaxwell , and a message that no feedback is coming on this from @spartahawk , so I'm merging this as well.

@Loobinex Loobinex merged commit 1e0bc31 into master Aug 17, 2018
@Loobinex Loobinex deleted the ReinforceSound branch August 17, 2018 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant