X Tutup
Skip to content

Fix MusicPlayer crashing when music ends on sector view while in hyperspace#5425

Merged
sturnclaw merged 1 commit intopioneerspacesim:masterfrom
WKFO:mpfix
Nov 24, 2022
Merged

Fix MusicPlayer crashing when music ends on sector view while in hyperspace#5425
sturnclaw merged 1 commit intopioneerspacesim:masterfrom
WKFO:mpfix

Conversation

@WKFO
Copy link
Contributor

@WKFO WKFO commented Nov 20, 2022

Reported on IRC. Just adding one more condition to check on track end.
Please excuse the commit description.

@Damjan94
Copy link
Contributor

This seems to have resolved the crash involving the music player change in System View when hyperspacing to an uninhabited system. When hyperspacing to a inhabited system, the game still crashes, music or not...

@sturnclaw sturnclaw merged commit c9bfd39 into pioneerspacesim:master Nov 24, 2022
@impaktor
Copy link
Member

When hyperspacing to a inhabited system, the game still crashes, music or not...

If it still crashes on current master, then please report that @Damjan94, now that we know the music crash bug is fixed.

@Damjan94
Copy link
Contributor

When hyperspacing to a inhabited system, the game still crashes, music or not...

If it still crashes on current master, then please report that @Damjan94, now that we know the music crash bug is fixed.

I haven't actually got to play the game yet, but it shouldn't crash anymore...

@sturnclaw
Copy link
Member

The issue was tracked in #5424, and this PR addressed one half of the mentioned issue. The other crash condition (an actual segfault) was addressed separately in another PR. No further issue reporting is required as long as no new crash bugs have been found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup