Angband.oook.cz
Angband.oook.cz
AboutVariantsLadderForumCompetitionComicScreenshotsFunniesLinks

Go Back   Angband Forums > Angband > Vanilla

Reply
 
Thread Tools Display Modes
Old May 10, 2022, 04:51   #121
Pete Mack
Prophet
 
Join Date: Apr 2007
Location: Seattle, WA
Posts: 6,681
Donated: $40
Pete Mack is on a distinguished road
The message only occurs when you cross the threshold. Going further over it is silent.
Pete Mack is offline   Reply With Quote
Old May 10, 2022, 17:00   #122
malcontent
Adept
 
Join Date: Jul 2019
Location: Seattle, WA USA
Posts: 137
malcontent is on a distinguished road
Quote:
Originally Posted by Pete Mack View Post
The message only occurs when you cross the threshold. Going further over it is silent.
Ah, yes, my Knight's Shield of resistance had already sent me past the threshold. Thanks!
malcontent is offline   Reply With Quote
Old May 16, 2022, 13:32   #123
backwardsEric
Swordsman
 
Join Date: Aug 2019
Posts: 296
backwardsEric is on a distinguished road
Quote:
Originally Posted by Sphara View Post
Hi Nick.
Trap immunity (Wormtongue boots) doesn't apply when disarming chest traps. Summoning trap went off on my face. Intended or not?
If you had directly opened it (or, to use wobbly's phrase, booted it) the trap immunity would have bypassed the trap. To me, that's another argument, besides how it works for floor traps, for applying it when disarming.
backwardsEric is offline   Reply With Quote
Old June 7, 2022, 15:04   #124
PowerWyrm
Prophet
 
PowerWyrm's Avatar
 
Join Date: Apr 2008
Posts: 2,928
PowerWyrm is on a distinguished road
Item ignoring setup is bugged, the tags are incorrect. Commit 3126eb4 missed a I2A -> all_letters_nohjkl change in tag_options_item() and added a pointless menu.selections in do_cmd_options_item() since the menu is managed by a menu_iter.

(expected: a-b-c-d-e-f-g-i-m-n-o-p-q-r-s-t-u-(blank)-Q-E-T; displayed: a-b-c-d-e-f-g-i-m-n-o-p-q-r-s-t-u-(v)-w-x-y)
__________________
PWMAngband variant maintainer - check http://powerwyrm.monsite-orange.fr (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!
PowerWyrm is offline   Reply With Quote
Old June 8, 2022, 14:07   #125
PowerWyrm
Prophet
 
PowerWyrm's Avatar
 
Join Date: Apr 2008
Posts: 2,928
PowerWyrm is on a distinguished road
Commit a318f61 removed the call to open_audio_hook(), surely this disabled sound completely no?

Also if you use ogg files and ogg format is supported but not mp3 format, code in open_audio_sdl() will now check for mp3 and reject sound instead of loading the ogg files.
__________________
PWMAngband variant maintainer - check http://powerwyrm.monsite-orange.fr (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!

Last edited by PowerWyrm; June 8, 2022 at 14:22.
PowerWyrm is offline   Reply With Quote
Old June 8, 2022, 14:53   #126
backwardsEric
Swordsman
 
Join Date: Aug 2019
Posts: 296
backwardsEric is on a distinguished road
Quote:
Originally Posted by PowerWyrm View Post
Commit a318f61 removed the call to open_audio_hook(), surely this disabled sound completely no?

Also if you use ogg files and ogg format is supported but not mp3 format, code in open_audio_sdl() will now check for mp3 and reject sound instead of loading the ogg files.
There were two calls to open_audio_hook(), one after the other, in sound-core.c. That change you referenced only removed one of them. As for the sound format support, the added call to Mix_init() was to check that the SDL Mixer library has MP3 support because that's what Vanilla's and FAangband's sound files use. If a variant uses other formats, then it's likely best to 'formats = ...;' line to reflect the formats that the variant expects the SDL mixer to support or to use 'formats = 0;' to disable checking for any specific format (at the risk of triggering file descriptor leaks, see https://github.com/angband/angband/issues/5313 , if the sound system later tries to load a format not supported by the SDL mixer).
backwardsEric is offline   Reply With Quote
Old June 9, 2022, 08:34   #127
PowerWyrm
Prophet
 
PowerWyrm's Avatar
 
Join Date: Apr 2008
Posts: 2,928
PowerWyrm is on a distinguished road
Quote:
Originally Posted by backwardsEric View Post
There were two calls to open_audio_hook(), one after the other, in sound-core.c. That change you referenced only removed one of them.
Check the change, that's not the case. First line is to check if the hook (pointer) exists:

Code:
if (!hooks.open_audio_hook) return 1;
Second line actually calls the hook and checks the return value:

Code:
if (!hooks.open_audio_hook()) return 1;
These are not the same calls. Removing the second line actually removes the call to open_audio_hook() and only leaves the pointer check.
__________________
PWMAngband variant maintainer - check http://powerwyrm.monsite-orange.fr (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!
PowerWyrm is offline   Reply With Quote
Old June 9, 2022, 08:39   #128
PowerWyrm
Prophet
 
PowerWyrm's Avatar
 
Join Date: Apr 2008
Posts: 2,928
PowerWyrm is on a distinguished road
Effect READ_MINDS returns false if no monster with a mind is found, which is not the same as any other detection/mapping effect which always return true. This has a weird side effect of preventing putting the effect on any object, because if there's no monster with a mind around, the object will not be consumed.
__________________
PWMAngband variant maintainer - check http://powerwyrm.monsite-orange.fr (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!
PowerWyrm is offline   Reply With Quote
Old June 9, 2022, 13:25   #129
PowerWyrm
Prophet
 
PowerWyrm's Avatar
 
Join Date: Apr 2008
Posts: 2,928
PowerWyrm is on a distinguished road
Quote:
Originally Posted by backwardsEric View Post
As for the sound format support, the added call to Mix_init() was to check that the SDL Mixer library has MP3 support because that's what Vanilla's and FAangband's sound files use.
The problem is that reading MP3 using the SDL Mixer library doesn't require calling Mix_init. In fact, on my WinXP VM, calling Mix_init(MIX_INIT_MP3) returns 0 (Does not support the required music formats) while I can play with sounds using MP3 files perfectly.

If there are file descriptor leaks, it must be another non related issue. For example, load_sample_sdl() calls Mix_LoadMUS() and stores the return value in the "loaded" boolean. Here we can't tell samples not loaded from samples where loading caused an error, maybe the boolean should be changed to an enum (not loaded, loaded, error). In case an error occured and the game tries to reload the sample, the "error" state should tell that the sample cannot be loaded and prevent calling Mix_LoadMUS() again.
__________________
PWMAngband variant maintainer - check http://powerwyrm.monsite-orange.fr (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!
PowerWyrm is offline   Reply With Quote
Old June 10, 2022, 12:52   #130
backwardsEric
Swordsman
 
Join Date: Aug 2019
Posts: 296
backwardsEric is on a distinguished road
Quote:
Originally Posted by PowerWyrm View Post
The problem is that reading MP3 using the SDL Mixer library doesn't require calling Mix_init. In fact, on my WinXP VM, calling Mix_init(MIX_INIT_MP3) returns 0 (Does not support the required music formats) while I can play with sounds using MP3 files perfectly.
Out of curiosity, is that because that environment has a default external command configured to play the MP3 file (or perhaps it's related to this part of the changes list for SDL Mixer 2.0.3: "Fixed regression where Mix_Init() would return 0 for available music formats")? In any case, trying to Mix_Init() to workaround the resource leaks doesn't seem useful.
backwardsEric is offline   Reply With Quote
Reply


Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)
 
Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT +1. The time now is 23:12.


Powered by vBulletin® Version 3.8.11
Copyright ©2000 - 2022, vBulletin Solutions Inc.