fergalmoran/flameshot
1
0
Fork 0
mirror of https://github.com/fergalmoran/flameshot.git synced 2026-01-04 08:07:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

fixed segfault

Browse Source
This commit is contained in:
Jeremy Borgman
2025-06-01 11:52:53 -05:00
parent b4b00ec3fa
commit a41137707c
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/widgets/capture/capturewidget.cpp
View File

@@ -205,6 +205,13 @@ CaptureWidget::CaptureWidget(const CaptureRequest& req,
// Init color picker // Init color picker
m_colorPicker = new ColorPicker(this); m_colorPicker = new ColorPicker(this);
// Init notification widget
m_notifierBox = new NotifierBox(this);
initPanel();
// TODO: Make it more clear why this has moved. In Qt6 some timing related to
// constructors / connect signals has changed so if initPanel is called
// after the connect a SEGFAULT occurs
connect(m_colorPicker, connect(m_colorPicker,
&ColorPicker::colorSelected, &ColorPicker::colorSelected,
this, this,
@@ -220,8 +227,7 @@ CaptureWidget::CaptureWidget(const CaptureRequest& req,
this, this,
&CaptureWidget::onToolSizeChanged); &CaptureWidget::onToolSizeChanged);
// Init notification widget
m_notifierBox = new NotifierBox(this);
m_notifierBox->hide(); m_notifierBox->hide();
connect(m_notifierBox, &NotifierBox::hidden, this, [this]() { connect(m_notifierBox, &NotifierBox::hidden, this, [this]() {
// Show cursor if it was hidden while adjusting tool size // Show cursor if it was hidden while adjusting tool size
@@ -231,7 +237,8 @@ CaptureWidget::CaptureWidget(const CaptureRequest& req,
onToolSizeSettled(m_context.toolSize); onToolSizeSettled(m_context.toolSize);
}); });
initPanel();
m_config.checkAndHandleError(); m_config.checkAndHandleError();
if (m_config.hasError()) { if (m_config.hasError()) {