News:

The new Release 25.03 is out! You can download binaries for Windows and many major Linux distros here .

Main Menu

sdk\editormanager.cpp

Started by sethjackson, March 05, 2006, 10:30:58 PM

Previous topic - Next topic

sethjackson

Why does there need to be a wxNotebookSizer? The wx manual says it is depreciated. Besides we use wxFNB not wxNotebook.  :? Patch below gets rid of it....

Code (diff) Select

Index: src/sdk/editormanager.cpp
===================================================================
--- src/sdk/editormanager.cpp (revision 2141)
+++ src/sdk/editormanager.cpp (working copy)
@@ -208,11 +208,6 @@

     CreateSearchLog();
     LoadAutoComplete();
-
-#if !wxCHECK_VERSION(2, 5, 0)
-    /*wxNotebookSizer* nbs =*/
-    new wxNotebookSizer(m_pNotebook);
-#endif
}

// class destructor



EDIT:

BTW I posted it to the tracker here.

https://developer.berlios.de/patch/index.php?func=detailpatch&patch_id=897&group_id=5358


thomas

"We should forget about small efficiencies, say about 97% of the time: Premature quotation is the root of public humiliation."

sethjackson

Quote from: thomas on March 05, 2006, 10:52:02 PM
wxWidgets 2.4.2

And who uses that? You cant compile startherepage.cpp with 2.4.2 because of wxLaunchDefaultBrowser() so...... Do we really need to be backwards compatible with 2.4.2?