Angband 4.2.4

Collapse
X
 
  • Time
  • Show
Clear All
new posts
  • Pete Mack
    Prophet
    • Apr 2007
    • 6883

    The message only occurs when you cross the threshold. Going further over it is silent.

    Comment

    • malcontent
      Adept
      • Jul 2019
      • 166

      Originally posted by Pete Mack
      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!

      Comment

      • backwardsEric
        Knight
        • Aug 2019
        • 526

        Originally posted by Sphara
        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.

        Comment

        • PowerWyrm
          Prophet
          • Apr 2008
          • 2986

          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 https://github.com/draconisPW/PWMAngband (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!

          Comment

          • PowerWyrm
            Prophet
            • Apr 2008
            • 2986

            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.
            Last edited by PowerWyrm; June 8, 2022, 14:22.
            PWMAngband variant maintainer - check https://github.com/draconisPW/PWMAngband (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!

            Comment

            • backwardsEric
              Knight
              • Aug 2019
              • 526

              Originally posted by PowerWyrm
              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).

              Comment

              • PowerWyrm
                Prophet
                • Apr 2008
                • 2986

                Originally posted by backwardsEric
                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 https://github.com/draconisPW/PWMAngband (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!

                Comment

                • PowerWyrm
                  Prophet
                  • Apr 2008
                  • 2986

                  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 https://github.com/draconisPW/PWMAngband (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!

                  Comment

                  • PowerWyrm
                    Prophet
                    • Apr 2008
                    • 2986

                    Originally posted by backwardsEric
                    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 https://github.com/draconisPW/PWMAngband (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!

                    Comment

                    • backwardsEric
                      Knight
                      • Aug 2019
                      • 526

                      Originally posted by PowerWyrm
                      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.

                      Comment

                      • PowerWyrm
                        Prophet
                        • Apr 2008
                        • 2986

                        Originally posted by backwardsEric
                        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.
                        I'm using SDL Mixer 1.2 so it's probably an error in that version. I guess Mix_Init returns 0 not only when the format is not supported, but also when the format doesn't require a call to Mix_Init.
                        PWMAngband variant maintainer - check https://github.com/draconisPW/PWMAngband (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!

                        Comment

                        • PowerWyrm
                          Prophet
                          • Apr 2008
                          • 2986

                          Feather falling halves damage from lava. Shouldn't walking on lava identify the flag?
                          PWMAngband variant maintainer - check https://github.com/draconisPW/PWMAngband (or http://www.mangband.org/forum/viewforum.php?f=9) to learn more about this new variant!

                          Comment

                          • Grotug
                            Veteran
                            • Nov 2013
                            • 1634

                            Bug report: items sometimes falsely display a resistance they do not have. If you look in the character resistances sheet it shows the item does not have the resistance even though the item says it does.

                            In game example: I put on a bronze amulet which, due to my knowledge of the game, I recognized as an amulet of Weaponmastery, since it has +2 strength, a slay, and sustains Strength and Constitution and prevents paralysis. However, while I was wearing this amulet of weaponmastery, I had also just found a Jewelled ring. Since the ring did not have any abilities on it I cast "resist poison" spell which identified the ring as a ring of resist poison. 'Cool!' I think to myself. I then look again at my bronze amulet to see that it also says it resists poison. Hmm! Anyway, obviously a bug. This happened once before with an amulet of resist acid I believe. It said it also resisted cold, and so I thought I had an amulet of resistance, but I didn't, because I knew the rune of resist lightning and the amulet didn't resist lightning. Once I fully id'd the amulet (which I think was resist acid) it stopped showing that it resisted cold. I suspect that when I learn the rune of rDisenchantment and the "weaponmastery is fully ID'd that it will stop showing that it resists poison. (I'll report back).
                            Beginner's Guide to Angband 4.2.3 Part 1: https://www.youtube.com/watch?v=m9c9e2wMngM

                            Detailed account of my Ironman win here.

                            "My guess is that Grip and Fang have many more kills than Gothmog and Lungorthin." --Fizzix

                            Comment

                            • backwardsEric
                              Knight
                              • Aug 2019
                              • 526

                              Originally posted by Grotug
                              Bug report: items sometimes falsely display a resistance they do not have. If you look in the character resistances sheet it shows the item does not have the resistance even though the item says it does.
                              Could you post a compressed savefile for a character affected by the bug? Trying to reproduce it with the debugging commands (new druid character, advance to maximum level, get and equip amulet of weapon mastery and ring of resist poison, cast resist poison) didn't show resist poison for the amulet on the character sheet.

                              Comment

                              • Grotug
                                Veteran
                                • Nov 2013
                                • 1634

                                The character died a most *stupid* death (arkenstone, legendary stealth, x4 +21 xbow, fast diving)

                                I was playing hobbit ranger; not sure if race/class combo matters for this bug. I’m sorry, I should have made a copy of the save file. It’s a little tricky to reproduce because you also need to be wearing an unidentified ring of resist poison while wearing the unidentified amulet of weaponmastery. But the bug should trigger with any ring with unknown resistances (such as fire/cold ring) and an unID’d amulet. Although with a ring of cold/fire I guess you’ll need to be breathed on by cold or fire or maybe quaffing a rHeat or rCold potion would trigger the bug. At any rate I’ll try to reproduce it.

                                Note: the bug won’t trigger if the amulet of weaponmastery (or whatever amulet you wear that doesn’t have rCold, rFire or rPois) or the ring are fully ID’d. Both items need to be un ID’d).
                                Beginner's Guide to Angband 4.2.3 Part 1: https://www.youtube.com/watch?v=m9c9e2wMngM

                                Detailed account of my Ironman win here.

                                "My guess is that Grip and Fang have many more kills than Gothmog and Lungorthin." --Fizzix

                                Comment

                                Working...
                                😀
                                😂
                                🥰
                                😘
                                🤢
                                😎
                                😞
                                😡
                                👍
                                👎