...
 
Commits (2)
  • Adrian Chaves's avatar
    Fix bad usage of QDateTime.addDays() · 1b30734e
    Adrian Chaves authored
    The issue was detected by the compiler. addDays() was returning a value that
    was being ignored.
    
    According to the Qt documentation, addDays() does not modify the QDateTime
    instance where it is called, but instead it returns the QDateTime instance
    resulting from the specified change.
    1b30734e
  • Luca Giambonini's avatar
    Merge branch 'ignored-return-value' into 'master' · 6f41b048
    Luca Giambonini authored
    Fix bad usage of QDateTime.addDays()
    
    See merge request chakra/tools/akabei/akabeicore!3
    6f41b048
......@@ -79,14 +79,14 @@ void AkabeiPolkitSQLiteConnectionTest::simpleTest()
const QByteArray md5sum = "balbablsdf323434523534";
const QUrl url("http://foo.com/bar.html");
QDateTime buildDate;
buildDate.addDays(100);
buildDate = buildDate.addDays(100);
const QString arch("i686");
const QString packager("John Doe");
const int flags = 1;
const QUrl screenshot("http://foo.com/bar.jpg");
const Akabei::Package::InstallReason reason = Akabei::Package::ExplicitlyInstalledReason;
QDateTime installDate;
installDate.addDays(200);
installDate = installDate.addDays(200);
try {
// That's gonna be long.
......
......@@ -79,14 +79,14 @@ void AkabeiSQLiteTest::test()
const QByteArray md5sum = "balbablsdf323434523534";
const QUrl url("http://foo.com/bar.html");
QDateTime buildDate;
buildDate.addDays(100);
buildDate = buildDate.addDays(100);
const QString arch("i686");
const QString packager("John Doe");
const int flags = 1;
const QUrl screenshot("http://foo.com/bar.jpg");
const Akabei::Package::InstallReason reason = Akabei::Package::ExplicitlyInstalledReason;
QDateTime installDate;
installDate.addDays(200);
installDate = installDate.addDays(200);
try {
// That's gonna be long.
......