]> git.saurik.com Git - wxWidgets.git/commitdiff
Fix handling of wxTextEntry hints in wxGTK.
authorVadim Zeitlin <vadim@wxwidgets.org>
Sun, 9 Sep 2012 20:28:22 +0000 (20:28 +0000)
committerVadim Zeitlin <vadim@wxwidgets.org>
Sun, 9 Sep 2012 20:28:22 +0000 (20:28 +0000)
Recent changes (r71308) optimizing away setting the text of the text entry if
it didn't really change completely broke the generic text hints implementation
as e.g. clearing the hint didn't change anything because the entry was already
logically empty anyhow.

Fix this by simply comparing the new value with the real value of the entry
returned by DoGetValue() and not GetValue() that takes hints into account.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@72458 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/common/textentrycmn.cpp
src/gtk/textentry.cpp

index 971f5626688ed50faab56148b1f5a62469327d04..3e5c6117e07d68eb736508b14cc743decc9d2cdf 100644 (file)
@@ -224,7 +224,7 @@ void wxTextEntryBase::AppendText(const wxString& text)
 
 void wxTextEntryBase::DoSetValue(const wxString& value, int flags)
 {
-    if ( value != GetValue() )
+    if ( value != DoGetValue() )
     {
         EventsSuppressor noeventsIf(this, !(flags & SetValue_SendEvent));
 
index 1354841812ad59f911dfe9d9f50551c124229937..a1dad35bc6e93c45f85bf9f2a1a9b1515d74420a 100644 (file)
@@ -155,7 +155,7 @@ void wxTextEntry::WriteText(const wxString& value)
 
 void wxTextEntry::DoSetValue(const wxString& value, int flags)
 {
-    if (value != GetValue())
+    if (value != DoGetValue())
     {
         // use Remove() rather than SelectAll() to avoid unnecessary clipboard
         // operations, and prevent triggering an apparent bug in GTK which