Skip to content

tk blitting to destroyed canvases should be a noop, not a segfault. - #25104

Merged
jklymak merged 1 commit into
matplotlib:mainfrom
anntzer:tkc
Jan 30, 2023
Merged

tk blitting to destroyed canvases should be a noop, not a segfault.#25104
jklymak merged 1 commit into
matplotlib:mainfrom
anntzer:tkc

Conversation

@anntzer

@anntzer anntzer commented Jan 28, 2023

Copy link
Copy Markdown
Contributor

Choosing to noop (rather than erroring) is consistent with other backends.

Closes #16580, closes #23809.

PR Summary

PR Checklist

Documentation and Tests

  • Has pytest style unit tests (and pytest passes)
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • New plotting related features are documented with examples.

Release Notes

  • New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

Choosing to noop (rather than erroring) is consistent with other
backends.
Comment thread lib/matplotlib/tests/test_backend_tk.py
@jklymak
jklymak merged commit 1ad6cbf into matplotlib:main Jan 30, 2023
@anntzer
anntzer deleted the tkc branch January 30, 2023 22:58
@QuLogic QuLogic added this to the v3.8.0 milestone Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: blitting after closing second tkinter embed causes silent crash Segmentation fault when blitting onto closed figure (TkAgg)

4 participants