[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [kdepim] calendarviews/agenda: Fix Decoration::weekDate(), it was a noop.
From:       Sergio Martins <iamsergio () gmail ! com>
Date:       2015-04-26 18:38:30
Message-ID: E1YmRRm-000829-Rw () scm ! kde ! org
[Download RAW message or body]

Git commit 1c7b43aa73dbec7368daf3beb0ce9ac8d7f5c8bf by Sergio Martins.
Committed on 26/04/2015 at 18:37.
Pushed by smartins into branch 'master'.

Fix Decoration::weekDate(), it was a noop.

QDate::addDays() is const, use it's return value instead.

M  +1    -2    calendarviews/agenda/calendardecoration.cpp

http://commits.kde.org/kdepim/1c7b43aa73dbec7368daf3beb0ce9ac8d7f5c8bf

diff --git a/calendarviews/agenda/calendardecoration.cpp b/calendarviews/agenda/calendardecoration.cpp
index d931a71..35f3242 100644
--- a/calendarviews/agenda/calendardecoration.cpp
+++ b/calendarviews/agenda/calendardecoration.cpp
@@ -270,8 +270,7 @@ Element::List Decoration::createYearElements(const QDate &)
 QDate Decoration::weekDate(const QDate &date)
 {
     QDate result = date;
-    result.addDays(date.dayOfWeek() - 1);
-    return result;
+    return result.addDays(date.dayOfWeek() - 1);
 }
 
 QDate Decoration::monthDate(const QDate &date)
[prev in list] [next in list] [prev in thread] [next in thread]