diff options
author | IIMarckus <iimarckus@gmail.com> | 2018-07-21 00:41:56 -0600 |
---|---|---|
committer | IIMarckus <iimarckus@gmail.com> | 2018-07-21 00:41:56 -0600 |
commit | 8aebc7ba56746b19d3338ab548ad0ab59d2012aa (patch) | |
tree | 8d254ad7c603e72f4d231f5587ba32df2397fc36 /docs/bugs_and_glitches.md | |
parent | 2d73d040d16a7836f935a7a6cb2a311aed2ca5a2 (diff) | |
parent | 57fc81d44e1f26b8895d37c9d847400851a8f8b4 (diff) |
Merge branch 'master' of hyperdriveguy/pokecrystal
Diffstat (limited to 'docs/bugs_and_glitches.md')
-rw-r--r-- | docs/bugs_and_glitches.md | 30 |
1 files changed, 29 insertions, 1 deletions
diff --git a/docs/bugs_and_glitches.md b/docs/bugs_and_glitches.md index 786c51ab3..875a2ac77 100644 --- a/docs/bugs_and_glitches.md +++ b/docs/bugs_and_glitches.md @@ -246,7 +246,35 @@ DefenseDownHit: ([Video](https://www.youtube.com/watch?v=uRYyzKRatFk)) -*To do:* Identify specific code causing this bug and fix it. +This is a bug with `BattleCommand_Counter` in [engine/battle/move_effects/counter.asm](/engine/battle/move_effects/counter.asm) and `BattleCommand_MirrorCoat` in [engine/battle/move_effects/mirror_coat.asm](/engine/battle/move_effects/mirror_coat.asm): + +```asm + ; BUG: Move should fail with all non-damaging battle actions + ld hl, wCurDamage + ld a, [hli] + or [hl] + ret z +``` + +**Fix:** + +```diff + ld hl, wCurDamage + ld a, [hli] + or [hl] +- ret z ++ jp z, .failed +``` + +Add this to the end of each file: + +```diff ++.failed ++ ld a, 1 ++ ld [wEffectFailed], a ++ and a ++ ret +``` ## A Disabled but PP Up–enhanced move may not trigger Struggle |