> On Feb. 8, 2011, 10:05 a.m., David Faure wrote: > > kio/kio/job.cpp, line 1639 > > <http://git.reviewboard.kde.org/r/100516/diff/2/?file=8636#file8636line1639> > > > > Shouldn't there be a kio_http patch to go along with this? Otherwise > > the post data size is not used. Or was that just "for the future"?
Yes and I was going to do that in a separate patch. However, there is no reason to separate those ; so I will update the diff to include the kio_http. - Dawit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100516/#review1305 ----------------------------------------------------------- On Feb. 7, 2011, 6:46 p.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100516/ > ----------------------------------------------------------- > > (Updated Feb. 7, 2011, 6:46 p.m.) > > > Review request for kdelibs. > > > Summary > ------- > > The attached patch is the first portion a set of patches to make uploading > data through HTTP more efficient without affecting the existing > implementation. Right now the amount of memory consumed when uploading large > files through http or webdav is really not acceptable because only a > QByteArray based API is available. That means if you want to upload a file of > say 50 or 100 MB to a server, then you have to read the entire thing first > before you can call KIO::http_post! > > > This addresses bug 34578. > http://bugs.kde.org/show_bug.cgi?id=34578 > > > Diffs > ----- > > kio/kio/job.h 632dfc8 > kio/kio/job.cpp 7d4a849 > kio/kio/job_p.h daac895 > kio/kio/jobclasses.h e9bd191 > > Diff: http://git.reviewboard.kde.org/r/100516/diff > > > Testing > ------- > > Used by changing kdewebkit to use the new API. > > > Thanks, > > Dawit > >
