> On March 13, 2014, 12:40 p.m., David Faure wrote: > > kioslave/ftp/ftp.cpp, line 2260 > > <https://git.reviewboard.kde.org/r/116783/diff/2/?file=253641#file253641line2260> > > > > It's a bit confusing to have two methods called ftpSize. Maybe call > > this one ftpSizeCmd() or something? (check existing method names for > > consistency)
Ok. I will rename it ftpSendSizeCmd for consistency and adjust the spaces to 2 for this branches and correct it properly for the frameworks branch. - Dawit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116783/#review52887 ----------------------------------------------------------- On March 13, 2014, 12:33 p.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116783/ > ----------------------------------------------------------- > > (Updated March 13, 2014, 12:33 p.m.) > > > Review request for kdelibs and David Faure. > > > Repository: kdelibs > > > Description > ------- > > I completely missed ftpFileExists issued a SIZE command to determine the > existence of a file when addressing bug# 326292. See > https://git.reviewboard.kde.org/r/116524/ review request. The attached patch > addresses that oversight to insure renaming files work properly on the > android ftp server listed in the bug report. > > > Diffs > ----- > > kioslave/ftp/ftp.h cbcd096 > kioslave/ftp/ftp.cpp b9d90e6 > > Diff: https://git.reviewboard.kde.org/r/116783/diff/ > > > Testing > ------- > > Rerun all the tests run for 116524. > > > Thanks, > > Dawit Alemayehu > >
