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

Renamed GOPanelView to GOGUIPanelView and moved to the gui directory #1636

Merged
merged 2 commits into from
Aug 25, 2023

Conversation

oleg68
Copy link
Contributor

@oleg68 oleg68 commented Aug 16, 2023

Working with #1035 I noticed that the GOPanelView class is close to other GUI classes.

So this PR renames GOPanelView to GOGUIPanelView and moved to the gui directory.

It is just refactoring. No GO behavior should be changed.

@larspalo
Copy link
Contributor

When testing this PR I've noticed a few crashes when closing GO, however it doesn't seem to be related to this PR at all (and I can make it happen with a current build too), but it's likely worth examining more closely.

CrashReportClose20230818.txt

@oleg68
Copy link
Contributor Author

oleg68 commented Aug 23, 2023

When testing this PR I've noticed a few crashes when closing GO, however it doesn't seem to be related to this PR at all (and I can make it happen with a current build too), but it's likely worth examining more closely.

It looks like some GUI element of the main toolbar is deleted twice: one times explicitly and one times as part as a child of the toolbar window.

Could you give me reproducing steps?

@larspalo
Copy link
Contributor

Could you give me reproducing steps?

Sure, it seems to happen every time both with a debug build and a release build on (Ubuntu 20.04) Linux.

  • Start GrandOrgue from a terminal either with
gdb ./GrandOrgue
run

or
pasuspender ./GrandOrgue
or
./GrandOrgue

  • Open File->Settings and change from using PortAudio Jack to other device ALSA (or the other way around: from ALSA device to Jack).
  • Ok and close Settings dialog
  • Close GrandOrgue from File menu and watch the message in terminal or gdb

Copy link
Contributor

@larspalo larspalo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As the crash I mentioned earlier isn't related to this PR, I approve.

@oleg68 oleg68 merged commit a2402e3 into GrandOrgue:master Aug 25, 2023
1 check passed
@oleg68 oleg68 deleted the refactor/gopanelview branch August 25, 2023 18:50
@oleg68
Copy link
Contributor Author

oleg68 commented Aug 26, 2023

  • Open File->Settings and change from using PortAudio Jack to other device ALSA (or the other way around: from ALSA device to Jack).
  • Ok and close Settings dialog
  • Close GrandOrgue from File menu and watch the message in terminal or gdb

I couldn't reproduce the crash: GO closes correctly.

Could you post the screenshot of File->Settings->Audio before the crash?

@larspalo
Copy link
Contributor

@oleg68 This is using a standard Ubuntu 20.04 and a local build. I've re-built both the release and debug version of GO from latest official source and it crashes at closing GO even without having done anything at all after startup, it really doesn't matter if I use PortAudio or RtAudio. It just crashes at closing. I also managed to get a crash while changing from PortAudio to RtAudio in the Settings dialog, but that's another thing, I guess.

Screenshots and crash reports from gdb:
GoCrash20230826.zip

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.

3 participants