From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Fri, 13 Apr 2007 23:36:42 +0000 (+0000)
Subject: fix wxTextCtrl::SetSelection(-1, -1) to behave as documented (patch 1700000)
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/b5a3a81b9ce50bbb1ab7961c41559dd1309fde10

fix wxTextCtrl::SetSelection(-1, -1) to behave as documented (patch 1700000)


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

diff --git a/docs/changes.txt b/docs/changes.txt
index adcb0e1b99..e19965a134 100644
--- a/docs/changes.txt
+++ b/docs/changes.txt
@@ -157,6 +157,10 @@ wxGTK:
 - Fix wxNO_BORDER style for wxRadioBox (David Hart)
 - Fix wxTextCtrl::GetLineText() for empty lines (Marcin Wojdyr)
 
+wxUniv:
+
+- Fix wxTextCtrl::SetSelection(-1, -1) to behave as documented (Anders Larsen)
+
 
 2.8.3
 -----
diff --git a/src/univ/textctrl.cpp b/src/univ/textctrl.cpp
index afba680e8d..487fb2fca0 100644
--- a/src/univ/textctrl.cpp
+++ b/src/univ/textctrl.cpp
@@ -1444,8 +1444,12 @@ wxString wxTextCtrl::GetSelectionText() const
 void wxTextCtrl::SetSelection(wxTextPos from, wxTextPos to)
 {
     // selecting till -1 is the same as selecting to the end
-    if ( to == -1 && from != -1 )
+    if ( to == -1 )
     {
+        // and selecting (-1, -1) range is the same as selecting everything, by
+        // convention
+        if ( from == -1 )
+            from = 0;
         to = GetLastPosition();
     }