Bug: RBY Screech has 0 PP (when maxed out to 3 PP Ups)

[BeyondTheHorizon]

Another Screech issue, this time with PP.

When I edit a Pokémon like say, Persian, and give it Screech, the move has 0 PP when I reopen the .pkm file.

053 - PERSIAN - 09A9.pk1

Edited May 19, 2017 by BeyondTheHorizon
Changed title as I found further information

[BeyondTheHorizon]

UPDATE: I've found that the bug only occurs when I give the move 3 PP Ups. If I give it 2 or less PP Ups, it retains its PP when the file is re-opened.

[BlackShark]

Not only Screech, this occurs for all the moves with 40 PP base (like Acid Armor), in Gen 1 as well as Gen 2.

[Kaphotics]

Right, because the MovePP/PPups storage only uses 1 byte for each move.

2 bits for PP up count (0-3)

6 bits for Current PP (0-63)

When PP is 64, an &= 0x3F sets it back to 0 as 7 bits are needed to store '64'.

Will need to see how the pkm actually stores 64 PP...

==============

http://bulbapedia.bulbagarden.net/wiki/Power_Point#Base_value_alteration

fix'd https://github.com/kwsch/PKHeX/commit/63b3c8b020509d6c1bcebd6d02756b76da576593

thanks for reporting!

[SadisticMystic]

It's for that very reason that RBY and GSC set a maximum of 7 PP that can be gained on a PP Up, so that 40-PP moves turn into 61, not 64.