Review Request: Fix DAV jobs bug

Sergio Luis Martins iamsergio at gmail.com
Sun Feb 20 21:58:19 GMT 2011


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



kio/kio/davjob.cpp
<http://git.reviewboard.kde.org/r/100694/#comment1309>

    Use static_cast<qint64>() instead of C-style cast.


- Sergio Luis


On Feb. 20, 2011, 9:48 p.m., Grégory Oestreicher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100694/
> -----------------------------------------------------------
> 
> (Updated Feb. 20, 2011, 9:48 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> Following commit 01db8ec6e9934a13ed8025949dab9dd43010bf53 the DAV jobs created with the DavJob::dav* utility functions ceased to work because they do not provide the body data length. This patch corrects this issue by using the same mechanisms as those used for HTTP POST requests.
> 
> 
> Diffs
> -----
> 
>   kio/kio/davjob.cpp 3b2a8da 
>   kioslave/http/http.h e8fe695 
>   kioslave/http/http.cpp 70960d6 
> 
> Diff: http://git.reviewboard.kde.org/r/100694/diff
> 
> 
> Testing
> -------
> 
> Tested using the Akonadi Dav groupware resource and DaviCal as a backend server. No issues so far.
> 
> 
> Thanks,
> 
> Grégory
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110220/79555a0c/attachment.htm>


More information about the kde-core-devel mailing list