User forums > General (but related to Code::Blocks)

Make DirectWrite the default value for editor technology (MSW). => broke build

(1/1)

killerbot:
-    cmbTechnology->SetSelection(cfg->ReadInt(wxT("/technology"), 0));
+    cmbTechnology->SetSelection(cfg->ReadInt(wxT("/technology"), 1);

a ")" is missing -==> @ wh11204

Could you add it, I am at a non edit location at the moment ;-)

stahta01:
This is what I think the correct fix is. But, I have not tested it.


--- Code: ---Index: src/sdk/cbeditor.cpp
===================================================================
--- src/sdk/cbeditor.cpp (revision 13584)
+++ src/sdk/cbeditor.cpp (working copy)
@@ -1441,7 +1441,7 @@
     if (!control)
         return;
 #if defined(__WXMSW__) && wxCHECK_VERSION(3, 1, 0)
-    const int technology = config->ReadInt(wxT("/technology"), 0);
+    const int technology = config->ReadInt(wxT("/technology"), 1);
     if (technology == 1)
         control->SetTechnology(wxSCI_TECHNOLOGY_DIRECTWRITE);
     else

--- End code ---

stahta01:
Your place appears to be here

--- Code: ---Index: src/src/editorconfigurationdlg.cpp
===================================================================
--- src/src/editorconfigurationdlg.cpp (revision 13584)
+++ src/src/editorconfigurationdlg.cpp (working copy)
@@ -163,7 +163,7 @@
     wxChoice *cmbFontQuality = XRCCTRL(*this, "cmbFontQuality", wxChoice);
 
 #if defined(__WXMSW__) && wxCHECK_VERSION(3, 1, 0)
-    cmbTechnology->SetSelection(cfg->ReadInt(wxT("/technology"), 0));
+    cmbTechnology->SetSelection(cfg->ReadInt(wxT("/technology"), 1));
     cmbFontQuality->SetSelection(cfg->ReadInt(wxT("/font_quality"), 0));
 #else
     cmbTechnology->SetSelection(0);

--- End code ---

I was wondering if both places needed changed or just one of the places. If one, I thought I picked the correct one.
Edit: I think it would be best to change both places, now.

Tim S.

Miguel Gimenez:
Sorry, I was on Linux so compilation obviously was OK.

EDIT: fixed in trunk. It is wonderful how a bracket can disappear just changing a '0' to a '1'  :o

Navigation

[0] Message Index