Skip to content

Issue 1029 - Crash on QTreeViewItem#setBackgroundColor#1032

Merged
klayoutmatthias merged 2 commits intomasterfrom
issue-1029
Mar 16, 2022
Merged

Issue 1029 - Crash on QTreeViewItem#setBackgroundColor#1032
klayoutmatthias merged 2 commits intomasterfrom
issue-1029

Conversation

@klayoutmatthias
Copy link
Collaborator

No description provided.

The problem happened because in the described case the
tl::Variant used as a intermediate container holds the
Python QBrush object and when it gets deleted, the
QBrush object is deleted too.
@klayoutmatthias klayoutmatthias linked an issue Mar 16, 2022 that may be closed by this pull request
@klayoutmatthias klayoutmatthias merged commit 51cf918 into master Mar 16, 2022
klayoutmatthias added a commit that referenced this pull request Mar 16, 2022
* Fixed issue #1029

The problem happened because in the described case the
tl::Variant used as a intermediate container holds the
Python QBrush object and when it gets deleted, the
QBrush object is deleted too.

* Added tests
@klayoutmatthias klayoutmatthias deleted the issue-1029 branch February 16, 2023 21:01
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.

Problem with Qt binding

1 participant