[Kde-pim] Review Request: Summary: Reduce the issues found by krazy2, directory: dav
Kevin Krammer
krammer at kde.org
Thu Apr 19 17:24:45 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104670/#review12663
-----------------------------------------------------------
Ship it!
Also ok. should also be in the general "fix coding style issues" commit
- Kevin Krammer
On April 19, 2012, 4:18 p.m., Guy Maurel wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104670/
> -----------------------------------------------------------
>
> (Updated April 19, 2012, 4:18 p.m.)
>
>
> Review request for KDEPIM and Kevin Krammer.
>
>
> Description
> -------
>
> Taking a look to
> http://www.englishbreakfastnetwork.org/krazy/reports/kde-4.x/kdepim-runtime/resources/index.html
>
> one can see: kdepim-runtime as still more than 600 issues.
>
> I propose to reduce this with this bugfix.
> Thanks to Kevin for the very usefull hints of tippos such as:
> old: if(condition){
> new: if ( condition ) {
>
>
> Diffs
> -----
>
> resources/dav/common/davcollectionmodifyjob.cpp 64cb534
> resources/dav/common/davprincipalsearchjob.cpp 1f79f3e
> resources/dav/resource/replaycache.h 65384bb
>
> Diff: http://git.reviewboard.kde.org/r/104670/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Guy Maurel
>
>
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
More information about the kde-pim
mailing list