変愚「[Fix] クロークの灼熱エゴアイテムでクラッシュする」のマージ #4214
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
#2998 の対処のコミット a1e75ec で光源の pval を 0 にする処理をなくした事により、 光源の元の pval + 1～3 の修正値になってしまっている。 自然生成される光源の寿命の最大値と暗視エゴの修正値の両方に pval を使い回しているのが そもそも問題であるので、松明と真鍮のランタンの pval は 0 にし、光源の寿命の最大値は FUEL_TORCH と FUEL_LAMP を参照するようにする。 厳密には自然生成された松明の寿命の最大値が 4000 から 2500 に下がることになるが、 ゲームバランスにはほぼ問題ないと思われる。
object_flags_lite 関数内の光源の灼熱エゴに関する処理で、光源の灼熱エゴ (LITE_AURA_FIRE) ではなく、クロークの灼熱エゴ (AURA_FIRE) のフラグを誤ってチェッ クしており、クロークに対して is_lite_requiring_fuel() を呼び出してしまうことで 例外が発生しクラッシュする。 正しいフラグに修正し、また念のため光源以外のアイテムでは early return するようにして おく。
やや冗長なコードをシンプルにし、関数名を実態に即したものに変更する。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.