Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Random crash for Pokey #220

Closed
LoveBodhi opened this issue Oct 29, 2021 · 6 comments
Closed

Random crash for Pokey #220

LoveBodhi opened this issue Oct 29, 2021 · 6 comments
Assignees
Labels
vanilla bug Something isn't working. The old bug which is reproducing on original VB6-built version of game.

Comments

@LoveBodhi
Copy link
Contributor

LoveBodhi commented Oct 29, 2021

Describe the issue

Random crashes when stacked goomba (Pokey, SMB2, npc-247) freeze with iceball (when set noiceball=0). Sometimes can be frozen, sometime can't, sometime crash.

Example Case

NSMB Classic, World 2-1 (Desert Walk)

bug1

Recording

None

Vanilla Recording

None

@Wohlstand Wohlstand added the bug Something isn't working. Only modern bugs are exist in TheXTech only. Doesn't exist on old VB6 game. label Oct 29, 2021
@Wohlstand
Copy link
Owner

That's a very weird crash that happens because of the next case: (I investigated just now and found the reason)

  • When some of Pokey segments got frozen, it turns into an ice cube
  • Other pokey segments expect that the frozen part was one of past segments, now it's an ice cube.
  • It sets the Special value with -3
  • At Ice cube, the special value is treated as a content NPC ID
  • Ice cube tries to find the NPC ID with -3
  • Kaboom

Снимок экрана от 2021-10-29 04-19-13

Снимок экрана от 2021-10-29 04-21-47

That's hard to catch, but, I bet it's fully vanilla crash bug.

@Wohlstand Wohlstand added vanilla bug Something isn't working. The old bug which is reproducing on original VB6-built version of game. and removed bug Something isn't working. Only modern bugs are exist in TheXTech only. Doesn't exist on old VB6 game. labels Oct 29, 2021
Wohlstand added a commit that referenced this issue Oct 29, 2021
@Wohlstand Wohlstand self-assigned this Oct 29, 2021
@Wohlstand
Copy link
Owner

So, I fixed that crash just now, should work now.

@Wohlstand Wohlstand added the NEED A TEST Completed task that needs a test and confirmation by a reporter label Oct 29, 2021
@0lhi
Copy link
Collaborator

0lhi commented Oct 29, 2021

Test on 1.3.5.2 lead once to LoveBodhi's Error Message, and sometimes to this:

Scr_2021-10-29_14-35-27

I could reproduce the latter on 1.3.5.3:

Scr_2021-10-29_14-43-31

I could not reproduce the Error Message a second time, neither on 1.3.5.2 nor 1.3.5.3.

@Wohlstand
Copy link
Owner

Error will appear strictly if you will shoot on a specific moment, once the segment will be turned into ice cube at the same moment when other segment checks the bottom if it's pokey segment or not.

@Wohlstand
Copy link
Owner

The case of ice move is a vanilla mess and is not a topic of this issue, can be put as a separated issue for later work.
So, as the problem got solved, I'll close this.

@Wohlstand Wohlstand removed the NEED A TEST Completed task that needs a test and confirmation by a reporter label Oct 29, 2021
@Wohlstand
Copy link
Owner

Wohlstand commented Oct 29, 2021

P.S. to reproduce error message I had to repeat multiple attempts to freeze them, and had to restart level by menu. (I used the level test run that normal episode playing)

@Wohlstand Wohlstand added this to the Version 1.3.5.3 milestone Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vanilla bug Something isn't working. The old bug which is reproducing on original VB6-built version of game.
Projects
None yet
Development

No branches or pull requests

3 participants