Opened 7 years ago

Closed 6 years ago

#9089 closed build error (fixed)

Generic wxDataViewCtrl leaks

Reported by: troelsk Owned by: roebling
Priority: normal Milestone:
Component: GUI-generic Version:
Keywords: Cc: troelsk, roebling
Blocked By: Blocking:
Patch: yes

Description

This should fix the rest of the leaks in generic.

Probably the only leak left is wxDataViewEditorCtrlEvtHandler in common.
(Uncommenting PopEventHandler calls produces mysterious crash, wxMSW. Another crash occurs if pressing Alt+F4 while editing a cell).

Attachments (2)

dataview-leak.patch download (4.6 KB) - added by troelsk 7 years ago.
dataview-leak-menu.patch download (915 bytes) - added by troelsk 7 years ago.
wxMenu leak

Download all attachments as: .zip

Change History (10)

Changed 7 years ago by troelsk

comment:1 Changed 7 years ago by troelsk

This patch wraps up this topic
http://thread.gmane.org/gmane.comp.lib.wxwidgets.devel/96372
(except for the editing leak, the missing PopEventHandler)

comment:2 Changed 7 years ago by troelsk

  • Type set to build error

Looks like the new "Many blocks, all 40 bytes
long" leak is from here:

[datavgen.cpp ln 3184]
ret = new wxDataViewTreeNode( parent ) ;

Apparantly it is never freed. I don't how (where) to fix this.

http://article.gmane.org/gmane.comp.lib.wxwidgets.devel/101594

comment:3 Changed 7 years ago by robind

  • Status changed from closed to reopened

comment:4 Changed 7 years ago by robind

  • Owner set to roebling
  • Status changed from reopened to assigned

comment:5 Changed 7 years ago by troelsk

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with r53679.

Changed 7 years ago by troelsk

wxMenu leak

comment:6 Changed 7 years ago by troelsk

Added patch fixing a wxMenu memory leak in the sample. Not so important.

comment:7 Changed 6 years ago by troelsk

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:8 Changed 6 years ago by vadz

  • Resolution set to fixed
  • Status changed from reopened to closed

Applied as r54969, thanks.

P.S. Please don't reopen old reports, open the new ones. Trac item numbers are cheap...

Note: See TracTickets for help on using tickets.