]> git.saurik.com Git - wxWidgets.git/blobdiff - src/msw/textctrl.cpp
Solved bug which was breaking richedit ctrls for non-unicode builds on some compilers.
[wxWidgets.git] / src / msw / textctrl.cpp
index 6d55b5594910180ae1e010c8d51cb74b4ba4fa90..47e6c260bb87ae0e0deafe27a145635f200bd3a5 100644 (file)
 #   include <fstream>
 #endif
 
-#if wxUSE_RICHEDIT
+#if wxUSE_RICHEDIT && !defined(__GNUWIN32_OLD__)
     #include <richedit.h>
 #endif
 
+// ----------------------------------------------------------------------------
+// private classes
+// ----------------------------------------------------------------------------
+
+#if wxUSE_RICHEDIT
+
+// this module initializes RichEdit DLL if needed
+class wxRichEditModule : public wxModule
+{
+public:
+    virtual bool OnInit();
+    virtual void OnExit();
+
+    // get the version currently loaded, -1 if none
+    static int GetLoadedVersion() { return ms_verRichEdit; }
+
+    // load the richedit DLL of at least of required version
+    static bool Load(int version = 1);
+
+private:
+    // the handle to richedit DLL and the version of the DLL loaded
+    static HINSTANCE ms_hRichEdit;
+
+    // the DLL version loaded or -1 if none
+    static int ms_verRichEdit;
+
+    DECLARE_DYNAMIC_CLASS(wxRichEditModule)
+};
+
+HINSTANCE wxRichEditModule::ms_hRichEdit = (HINSTANCE)NULL;
+int       wxRichEditModule::ms_verRichEdit = -1;
+
+IMPLEMENT_DYNAMIC_CLASS(wxRichEditModule, wxModule)
+
+#endif // wxUSE_RICHEDIT
 
 // ----------------------------------------------------------------------------
 // event tables and other macros
@@ -117,9 +152,6 @@ bool wxTextCtrl::Create(wxWindow *parent, wxWindowID id,
     if ( parent )
         parent->AddChild(this);
 
-    // set colours
-    SetupColours();
-
     // translate wxWin style flags to MSW ones, checking for consistency while
     // doing it
     long msStyle = ES_LEFT | WS_VISIBLE | WS_CHILD | WS_TABSTOP;
@@ -143,8 +175,6 @@ bool wxTextCtrl::Create(wxWindow *parent, wxWindowID id,
     if (m_windowStyle & wxTE_READONLY)
         msStyle |= ES_READONLY;
 
-    if (m_windowStyle & wxHSCROLL)
-        msStyle |= (WS_HSCROLL | ES_AUTOHSCROLL);
     if (m_windowStyle & wxTE_PASSWORD) // hidden input
         msStyle |= ES_PASSWORD;
 
@@ -164,7 +194,7 @@ bool wxTextCtrl::Create(wxWindow *parent, wxWindowID id,
         m_lDlgCode |= DLGC_WANTTAB;
 
     // do create the control - either an EDIT or RICHEDIT
-    const wxChar *windowClass = wxT("EDIT");
+    wxString windowClass = wxT("EDIT");
 
 #if wxUSE_RICHEDIT
     if ( m_windowStyle & wxTE_RICH )
@@ -176,10 +206,11 @@ bool wxTextCtrl::Create(wxWindow *parent, wxWindowID id,
         {
             // first try to load the RichEdit DLL (will do nothing if already
             // done)
-            if ( !wxTheApp->InitRichEdit() )
+            if ( !wxRichEditModule::Load() )
             {
                 wxLogError(_("Impossible to create a rich edit control, "
-                             "using simple text control instead."));
+                             "using simple text control instead. Please "
+                             "reinstall riched32.dll"));
 
                 s_errorGiven = TRUE;
             }
@@ -193,12 +224,31 @@ bool wxTextCtrl::Create(wxWindow *parent, wxWindowID id,
         {
             msStyle |= ES_AUTOVSCROLL;
             m_isRich = TRUE;
-            windowClass = wxT("RICHEDIT");
+
+            int ver = wxRichEditModule::GetLoadedVersion();
+            if ( ver == 1 )
+            {
+                windowClass = wxT("RICHEDIT");
+            }
+            else
+            {
+#ifndef RICHEDIT_CLASS
+                wxString RICHEDIT_CLASS;
+                RICHEDIT_CLASS.Printf(_T("RichEdit%d0"), ver);
+#if wxUSE_UNICODE
+                RICHEDIT_CLASS += _T('W');
+#else // ANSI
+                RICHEDIT_CLASS += _T('A');
+#endif // Unicode/ANSI
+#endif // !RICHEDIT_CLASS
+
+                windowClass = RICHEDIT_CLASS;
+            }
         }
     }
     else
         m_isRich = FALSE;
-#endif
+#endif // wxUSE_RICHEDIT
 
     bool want3D;
     WXDWORD exStyle = Determine3DEffects(WS_EX_CLIENTEDGE, &want3D);
@@ -212,7 +262,7 @@ bool wxTextCtrl::Create(wxWindow *parent, wxWindowID id,
     //     might be -1 in which case we should use the default values (and
     //     SetSize called below takes care of it)
     m_hWnd = (WXHWND)::CreateWindowEx(exStyle,
-                                      windowClass,
+                                      windowClass.c_str(),
                                       NULL,
                                       msStyle,
                                       0, 0, 0, 0,
@@ -261,6 +311,9 @@ bool wxTextCtrl::Create(wxWindow *parent, wxWindowID id,
     }
 #endif
 
+    // set colours
+    SetupColours();
+
     SetSize(pos.x, pos.y, size.x, size.y);
 
     return TRUE;
@@ -298,8 +351,13 @@ void wxTextCtrl::AdoptAttributesFromHWND()
 
 void wxTextCtrl::SetupColours()
 {
-    // FIXME why is bg colour not inherited from parent?
-    SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_WINDOW));
+    wxColour bkgndColour;
+    if (IsEditable() || (m_windowStyle & wxTE_MULTILINE))
+        bkgndColour = wxSystemSettings::GetSystemColour(wxSYS_COLOUR_WINDOW);
+    else
+        bkgndColour = wxSystemSettings::GetSystemColour(wxSYS_COLOUR_3DFACE);
+
+    SetBackgroundColour(bkgndColour);
     SetForegroundColour(GetParent()->GetForegroundColour());
 }
 
@@ -309,16 +367,57 @@ void wxTextCtrl::SetupColours()
 
 wxString wxTextCtrl::GetValue() const
 {
-    return wxGetWindowText(GetHWND());
+    // we can't use wxGetWindowText() (i.e. WM_GETTEXT internally) for
+    // retrieving more than 64Kb under Win9x
+#if wxUSE_RICHEDIT
+    if ( m_isRich )
+    {
+        int len = GetWindowTextLength(GetHwnd()) + 1;
+
+        wxString str;
+        wxChar *p = str.GetWriteBuf(len);
+
+        TEXTRANGE textRange;
+        textRange.chrg.cpMin = 0;
+        textRange.chrg.cpMax = -1;
+        textRange.lpstrText = p;
+
+        (void)SendMessage(GetHwnd(), EM_GETTEXTRANGE, 0, (LPARAM)&textRange);
+
+        // believe it or not, but EM_GETTEXTRANGE uses just CR ('\r') for the
+        // newlines which is neither Unix nor Windows style (Win95 with
+        // riched20.dll shows this behaviour) - convert it to something
+        // reasonable
+        for ( ; *p; p++ )
+        {
+            if ( *p == _T('\r') )
+                *p = _T('\n');
+        }
+
+        str.UngetWriteBuf();
+
+        return str;
+    }
+#endif // wxUSE_RICHEDIT
+
+    // WM_GETTEXT uses standard DOS CR+LF (\r\n) convention - convert to the
+    // same one as above for consitency
+    wxString str = wxGetWindowText(GetHWND());
+
+    return wxTextFile::Translate(str, wxTextFileType_Unix);
 }
 
 void wxTextCtrl::SetValue(const wxString& value)
 {
-    wxString valueDos = wxTextFile::Translate(value, wxTextFileType_Dos);
-
-    if ( valueDos != GetValue() )
+    // if the text is long enough, it's faster to just set it instead of first
+    // comparing it with the old one (chances are that it will be different
+    // anyhow, this comparison is there to avoid flicker for small single-line
+    // edit controls mostly)
+    if ( (value.length() > 0x400) || (value != GetValue()) )
     {
-        SetWindowText(GetHwnd(), valueDos);
+        wxString valueDos = wxTextFile::Translate(value, wxTextFileType_Dos);
+                               
+        SetWindowText(GetHwnd(), valueDos.c_str());
 
         AdjustSpaceLimit();
     }
@@ -420,8 +519,16 @@ bool wxTextCtrl::CanPaste() const
 
 void wxTextCtrl::SetEditable(bool editable)
 {
+    bool isEditable = IsEditable();
+
     HWND hWnd = GetHwnd();
     SendMessage(hWnd, EM_SETREADONLY, (WPARAM)!editable, (LPARAM)0L);
+
+    if (editable != isEditable)
+    {
+       SetupColours();
+       Refresh();
+    }
 }
 
 void wxTextCtrl::SetInsertionPoint(long pos)
@@ -748,11 +855,13 @@ bool wxTextCtrl::CanRedo() const
 // implemenation details
 // ----------------------------------------------------------------------------
 
+/*
 void wxTextCtrl::Command(wxCommandEvent & event)
 {
     SetValue(event.GetString());
     ProcessCommand (event);
 }
+*/
 
 void wxTextCtrl::OnDropFiles(wxDropFilesEvent& event)
 {
@@ -784,30 +893,19 @@ void wxTextCtrl::OnChar(wxKeyEvent& event)
             // ourselves the fact that we got here means that the user code
             // decided to skip processing of this TAB - probably to let it
             // do its default job.
-            //
-            // NB: Notice that Ctrl-Tab is handled elsewhere and Alt-Tab is
-            //     handled by Windows
             {
                 wxNavigationKeyEvent eventNav;
                 eventNav.SetDirection(!event.ShiftDown());
-                eventNav.SetWindowChange(FALSE);
+                eventNav.SetWindowChange(event.ControlDown());
                 eventNav.SetEventObject(this);
 
-                if ( GetEventHandler()->ProcessEvent(eventNav) )
+                if ( GetParent()->GetEventHandler()->ProcessEvent(eventNav) )
                     return;
             }
             break;
-
-        default:
-            event.Skip();
-            return;
     }
 
-    // don't just call event.Skip() because this will cause TABs and ENTERs
-    // be passed upwards and we don't always want this - instead process it
-    // right here
-
-    // FIXME
+    // no, we didn't process it
     event.Skip();
 }
 
@@ -829,22 +927,20 @@ bool wxTextCtrl::MSWCommand(WXUINT param, WXWORD WXUNUSED(id))
         case EN_CHANGE:
             {
                 wxCommandEvent event(wxEVT_COMMAND_TEXT_UPDATED, m_windowId);
-                wxString val(GetValue());
-                if ( !val.IsNull() )
-                    event.m_commandString = WXSTRINGCAST val;
                 event.SetEventObject( this );
-                ProcessCommand(event);
+                event.SetString( GetValue() );
+                ProcessCommand( event );
             }
             break;
 
-        case EN_ERRSPACE:
+        case EN_MAXTEXT:
             // the text size limit has been hit - increase it
             AdjustSpaceLimit();
             break;
 
             // the other notification messages are not processed
         case EN_UPDATE:
-        case EN_MAXTEXT:
+        case EN_ERRSPACE:
         case EN_HSCROLL:
         case EN_VSCROLL:
         default:
@@ -865,15 +961,25 @@ void wxTextCtrl::AdjustSpaceLimit()
         limit = len + 0x8000;    // 32Kb
 
 #if wxUSE_RICHEDIT
-        if ( m_isRich || limit > 0xffff )
-#else
-        if ( limit > 0xffff )
-#endif
-            ::SendMessage(GetHwnd(), EM_LIMITTEXT, 0, limit);
+        if ( m_isRich )
+        {
+            // as a nice side effect, this also allows passing limit > 64Kb
+            ::SendMessage(GetHwnd(), EM_EXLIMITTEXT, 0, limit);
+        }
         else
+#endif // wxUSE_RICHEDIT
+        {
+            if ( limit > 0xffff )
+            {
+                // this will set it to a platform-dependent maximum (much more
+                // than 64Kb under NT)
+                limit = 0;
+            }
+
             ::SendMessage(GetHwnd(), EM_LIMITTEXT, limit, 0);
+        }
     }
-#endif
+#endif // !Win16
 }
 
 bool wxTextCtrl::AcceptsFocus() const
@@ -953,3 +1059,68 @@ void wxTextCtrl::OnUpdateRedo(wxUpdateUIEvent& event)
     event.Enable( CanRedo() );
 }
 
+// ----------------------------------------------------------------------------
+// wxRichEditModule
+// ----------------------------------------------------------------------------
+
+#if wxUSE_RICHEDIT
+
+bool wxRichEditModule::OnInit()
+{
+    // don't do anything - we will load it when needed
+    return TRUE;
+}
+
+void wxRichEditModule::OnExit()
+{
+    if ( ms_hRichEdit )
+    {
+        FreeLibrary(ms_hRichEdit);
+    }
+}
+
+/* static */
+bool wxRichEditModule::Load(int version)
+{
+    wxCHECK_MSG( version >= 1 && version <= 3, FALSE,
+                 _T("incorrect richedit control version requested") );
+
+    if ( version <= ms_verRichEdit )
+    {
+        // we've already got this or better
+        return TRUE;
+    }
+
+    if ( ms_hRichEdit )
+    {
+        ::FreeLibrary(ms_hRichEdit);
+    }
+
+    // always try load riched20.dll first - like this we won't have to reload
+    // it later if we're first asked for RE 1 and then for RE 2 or 3
+    wxString dllname = _T("riched20.dll");
+    ms_hRichEdit = ::LoadLibrary(dllname);
+    ms_verRichEdit = 2; // no way to tell if it's 2 or 3, assume 2
+
+    if ( !ms_hRichEdit && (version == 1) )
+    {
+        // fall back to RE 1
+        dllname = _T("riched32.dll");
+        ms_hRichEdit = ::LoadLibrary(dllname);
+        ms_verRichEdit = 1;
+    }
+
+    if ( !ms_hRichEdit )
+    {
+        wxLogSysError(_("Could not load Rich Edit DLL '%s'"), dllname.c_str());
+
+        ms_verRichEdit = -1;
+
+        return FALSE;
+    }
+
+    return TRUE;
+}
+
+#endif // wxUSE_RICHEDIT
+