Skip to content

Fix crash on application quit#445

Merged
zccrs merged 1 commit intovioken:masterfrom
zccrs:master
Aug 1, 2024
Merged

Fix crash on application quit#445
zccrs merged 1 commit intovioken:masterfrom
zccrs:master

Conversation

@zccrs
Copy link
Copy Markdown
Contributor

@zccrs zccrs commented Aug 1, 2024

setParentItem(nullptr) maybe lead to
QQuickDeliveryAgentPrivate::clearFocusInScope, Qt wants emit QQuickItem::window()->focusObjectChanged signal, but the window of that item is nullptr now.

recurrence:
step 1: run tinywl-qtquick
step 2: run qtcreator in tinywl-qtquick
step 3: quit tinywl-qtquick by Ctrl+Q shortcut

setParentItem(nullptr) maybe lead to
QQuickDeliveryAgentPrivate::clearFocusInScope, Qt wants emit
QQuickItem::window()->focusObjectChanged signal, but the window of that
item is nullptr now.
@zccrs zccrs requested review from ZhongYic00 and wineee August 1, 2024 04:02
Copy link
Copy Markdown
Member

@wineee wineee left a comment

Choose a reason for hiding this comment

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

@zccrs zccrs merged commit e74d9d6 into vioken:master Aug 1, 2024
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.

2 participants