Skip to content

[FIX] overlay: Auto disconnect when the overlay widget is deleted#1412

Merged
kernc merged 1 commit intobiolab:masterfrom
ales-erjavec:overlay-fix
Jul 7, 2016
Merged

[FIX] overlay: Auto disconnect when the overlay widget is deleted#1412
kernc merged 1 commit intobiolab:masterfrom
ales-erjavec:overlay-fix

Conversation

@ales-erjavec
Copy link
Contributor

Register __on_destroyed as a Qt slot so it can be automatically
disconnected when the overlay widget is deleted/destroyed.
This fixes a possible 'RuntimeError C/C++ object was deleted ...' error
when the overlay is deleted before the target widget.

@astaric
Copy link
Member

astaric commented Jul 7, 2016

Travis is misbehaving again and did not automatically run the tests. Could you rebase and push again please?

Register `__on_destroyed` as a Qt slot so it can be automatically
disconnected when the overlay widget is deleted/destroyed.
This fixes a possible 'RuntimeError C/C++ object was deleted ...' error
when the overlay is deleted before the target widget.
@codecov-io
Copy link

codecov-io commented Jul 7, 2016

Current coverage is 87.95%

Merging #1412 into master will not change coverage

@@             master      #1412   diff @@
==========================================
  Files            77         77          
  Lines          7561       7561          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           6650       6650          
  Misses          911        911          
  Partials          0          0          

Sunburst

Powered by Codecov. Last updated by 2230b7c...6ec7128

@kernc kernc merged commit 38dfe71 into biolab:master Jul 7, 2016
@ales-erjavec ales-erjavec deleted the overlay-fix branch July 14, 2016 17: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.

4 participants