search

wxWidgets / wxWidgets

Cross-Platform C++ GUI Library
https://www.wxwidgets.org/
5.75k stars 1.69k forks source link

wxRegConfig leaks #20240

Closed wxtrac closed 2 years ago

wxtrac commented 21 years ago

Issue migrated from trac ticket # 875

component: documentation | priority: normal

2003-03-20 13:31:39: arkanes created the issue

When using the static access to wxConfig(maps to wxRegConfig) on wxMSW (wxConfig::Get()), it doesn't get destroyed at app end and leaks a couple registry handles.

wxtrac commented 21 years ago

2003-03-24 19:23:35: @vadz commented

Well, no, it doesn't get destroyed -- who has ever said it would? If you had looked at the config sample before filing this bug report you'd have noticed that it deletes this object explicitly.

wxtrac commented 21 years ago

2003-03-24 23:35:56: arkanes commented

Reclassified as documentation bug - the wxConfigBase documentation says that the framework destroys it in wxApp::OnExit (No, I didn't look at the sample, just the wxConfig overview and class documentation). Patch attached.

2003-03-24 23:35:56: arkanes uploaded file wxconfig_doc.patch (1.1 KiB)

doc patch for wxConfigBase

wxtrac commented 21 years ago

2003-04-03 01:36:01: @vadz commented

Sorry, I didn't know/remember about this piece of disinformation in the docs, hence my surprize at your original report.

I've applied the patch to the CVS HEAD, thanks!