corrected bug in wxTimeSpan::IsShorterThan() for equal time spans (ticket #9539)
authorVadim Zeitlin <vadim@wxwidgets.org>
Thu, 5 Jun 2008 23:51:04 +0000 (23:51 +0000)
committerVadim Zeitlin <vadim@wxwidgets.org>
Thu, 5 Jun 2008 23:51:04 +0000 (23:51 +0000)
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@53988 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

docs/changes.txt
include/wx/datetime.h

index 4c271ac1dbbaef603ef656177d94193c9c33de0b..c6df56c38baef6c5493a48c88563aa4c761c83c6 100644 (file)
@@ -272,6 +272,7 @@ All:
   now thread-safe if the standard library provided with your compiler is.
 - Added wxCmdLineParser::AddUsageText() (Marcin 'Malcom' Malich).
 - Fix reading/writing UTF-7-encoded text streams.
+- Corrected bug in wxTimeSpan::IsShorterThan() for equal time spans.
 
 All (Unix):
 
index 3fddea3f9a10f6025349fc2a839d3632d6200d80..45afb337484c6e371d54d9698ed91db954d9e249 100644 (file)
@@ -1433,7 +1433,7 @@ public:
         // compare two timestamps: works with the absolute values, i.e. 1
         // hour is shorter than -2 hours. Also, it will return false if the
         // timespans are equal in absolute value.
-    bool IsShorterThan(const wxTimeSpan& t) const { return !IsLongerThan(t); }
+    bool IsShorterThan(const wxTimeSpan& t) const;
 
     inline bool operator<(const wxTimeSpan &ts) const
     {
@@ -2203,6 +2203,11 @@ inline bool wxTimeSpan::IsLongerThan(const wxTimeSpan& ts) const
     return GetValue().Abs() > ts.GetValue().Abs();
 }
 
+inline bool wxTimeSpan::IsShorterThan(const wxTimeSpan& ts) const
+{
+    return GetValue().Abs() < ts.GetValue().Abs();
+}
+
 // ----------------------------------------------------------------------------
 // wxDateSpan
 // ----------------------------------------------------------------------------