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.

Sign up for GitHub

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

Jump to bottom

Assertion failure in Maps::Tiles::setBoat() #7957

Closed
2 tasks done
oleg-derevenetz opened this issue Oct 18, 2023 · 0 comments · Fixed by #7959
Closed
2 tasks done

Assertion failure in Maps::Tiles::setBoat() #7957

oleg-derevenetz opened this issue Oct 18, 2023 · 0 comments · Fixed by #7959
Assignees
@ihhub
Labels
bug Something doesn't work crash Application crashes. The highest level of priority regression Something went wrong in previous commits
Milestone
1.0.10

Comments

@oleg-derevenetz
Copy link
Collaborator

Preliminary checks

Platform

Windows

Describe the bug

How to reproduce:

  1. Load the attached save file;
  2. End the turn.

Save file

Crash.zip

Additional info

Looks like a regression after #7907.
@oleg-derevenetz oleg-derevenetz added bug Something doesn't work crash Application crashes. The highest level of priority regression Something went wrong in previous commits labels Oct 18, 2023
@oleg-derevenetz oleg-derevenetz added this to the 1.0.10 milestone Oct 18, 2023
@ihhub ihhub self-assigned this Oct 18, 2023
ihhub added a commit that referenced this issue Oct 18, 2023
@ihhub
Fix summon boat spell assertion
a3b18f8 
close #7957
@ihhub ihhub mentioned this issue Oct 18, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ihhub ihhub

Labels
bug Something doesn't work crash Application crashes. The highest level of priority regression Something went wrong in previous commits
Projects
None yet
Milestone
1.0.10
Development

Successfully merging a pull request may close this issue.

Fix summon boat spell assertion ihhub/fheroes2
2 participants
@ihhub @oleg-derevenetz