--===============2139744663600283668== Content-Type: multipart/alternative; boundary="===============5767768928734185566==" --===============5767768928734185566== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126659/#review90750 ----------------------------------------------------------- Ship it! Ship It! - David Faure On Jan. 7, 2016, 12:23 p.m., Wolfgang Bauer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126659/ > ----------------------------------------------------------- > > (Updated Jan. 7, 2016, 12:23 p.m.) > > > Review request for KDE Frameworks, kdelibs and David Faure. > > > Bugs: 354597 > https://bugs.kde.org/show_bug.cgi?id=354597 > > > Repository: kio > > > Description > ------- > > - QDate() treats the year literally (i.e. 90 is really year 90, not 1990), so subtracting 1900 is wrong. > - Use QDate::currentDate() instead of QDateTime::currentDateTime(), we only need the current date anyway > - Initialize day, month, and year to the current date instead of 0. In the case when no year is mentioned in the server's reply (the year is implicit), it wasn't set to the current year at all, so the result was either 0 or -1. > > > Diffs > ----- > > src/ioslaves/ftp/ftp.cpp 2179179 > > Diff: https://git.reviewboard.kde.org/r/126659/diff/ > > > Testing > ------- > > Connected to an FTP server with dolphin (15.12.0). The modification times/dates are now shown correctly. > > > Thanks, > > Wolfgang Bauer > > --===============5767768928734185566== MIME-Version: 1.0 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: 7bit
This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126659/ |
Ship it!
Ship It!
- David Faure
On January 7th, 2016, 12:23 p.m. UTC, Wolfgang Bauer wrote:
Review request for KDE Frameworks, kdelibs and David Faure.
By Wolfgang Bauer.
Updated Jan. 7, 2016, 12:23 p.m.
Bugs:
354597
Repository:
kio
Description
Testing
Diffs
|