Patch #1039 2006-05-09 17:48

sethjackson

Editor Zoom
Download
1039-Editor_Zoom.patch (751 bytes)
Category
Application::Refinement
Status
Accepted
Close date
2006-05-15 20:58
Assigned to
 
Index: src/sdk/editormanager.cpp
===================================================================
--- src/sdk/editormanager.cpp    (revision 2438)
+++ src/sdk/editormanager.cpp    (working copy)
@@ -213,7 +213,7 @@
 
     CreateSearchLog();
     LoadAutoComplete();
-    m_zoom = Manager::Get()->GetConfigManager(_T("editor"))->ReadInt(_T("zoom"));
+    m_zoom = Manager::Get()->GetConfigManager(_T("editor"))->ReadInt(_T("/zoom"));
 }
 
 // class destructor
@@ -246,7 +246,7 @@
         delete m_pData;
         m_pData = NULL;
     }
-    Manager::Get()->GetConfigManager(_T("editor"))->Write(_T("zoom"), m_zoom);
+    Manager::Get()->GetConfigManager(_T("editor"))->Write(_T("/zoom"), m_zoom);
 }
 
 void EditorManager::CreateMenu(wxMenuBar* menuBar)
sethjackson 2006-05-09 17:49

Nit-picky me..... The "/" is not required AFAIK, but nonetheless....

thomasdenk 2006-05-15 16:07

Afraid we need that slash. The application changes the config path for the namespace editor in other places.

Thus, if we remove that leading slash, it may work, but it may as well fail to find the value on a random base.

The slash guarantees that it can always find the value.

thomasdenk 2006-05-15 20:58

My bad... the slash is actually *missing* and the patch adds it... (blushes).

Well, I'll start again, hem hem!

Yes, the slash is needed for the above reason, thank you :)