OSX memory leak (PR #26215)

[Stefan Csomor]

see #26208

---

You can view, comment on, or merge this pull request online at:

  https://github.com/wxWidgets/wxWidgets/pull/26215

Commit Summary

• ea8bb27 remove gc aware code

File Changes

(2 files)

• M src/osx/cocoa/menu.mm (7)
• M src/osx/cocoa/window.mm (6)

Patch Links:

• https://github.com/wxWidgets/wxWidgets/pull/26215.patch
• https://github.com/wxWidgets/wxWidgets/pull/26215.diff

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26215@github.com>

[VZ]

vadz left a comment (wxWidgets/wxWidgets#26215)

Thanks Stefan! Two questions:

1. Does this really fix the leak, i.e. do we not need to release m_osxClipView finally?
2. Can this be backported to 3.2 as is?

—
Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26215/c3937497334@github.com>

[Stefan Csomor]

csomor left a comment (wxWidgets/wxWidgets#26215)

@vadz this is a work in progress, I'll have to think through the different cycles, it's not correct as of now,

—
Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26215/c3937526018@github.com>