Review Request: New KIO::http_post and KIO::StoredHttpPost APIs that accept a QIODevice as input...

Dawit Alemayehu adawit at kde.org
Wed Feb 9 01:28:14 GMT 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100516/
-----------------------------------------------------------

(Updated Feb. 9, 2011, 1:28 a.m.)


Review request for kdelibs.


Changes
-------

Changed kio_http to use the size information when available and send the data to the server as soon as it gets it.
Changed the KIO-QNAM integration class to use the new KIO::http_post API.


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 (updated)
-----

  kio/kio/job_p.h daac895 
  kio/kio/jobclasses.h e9bd191 
  kioslave/http/http.h be08d7b 
  kioslave/http/http.cpp 187e3c7 
  kio/kio/accessmanager.cpp 722203f 
  kio/kio/job.h 632dfc8 
  kio/kio/job.cpp 7d4a849 

Diff: http://git.reviewboard.kde.org/r/100516/diff


Testing
-------

Used by changing kdewebkit to use the new API. 


Thanks,

Dawit

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110209/7f795043/attachment.htm>


More information about the kde-core-devel mailing list