[Kde-pim] Review Request: Summary: some more spaces, some less spaces, some more blocks, for kabc

Kevin Krammer krammer at kde.org
Thu Aug 9 19:03:45 BST 2012


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


Good work!
Only three minor nitpicks left :)


kabc/distributionlist.cpp
<http://git.reviewboard.kde.org/r/105879/#comment13413>

    I think this line needs 2 more space intentation, i.e. a.fullEmail should be under a.fullEmail in the line above, like before



kabc/scripts/addressee.src.cpp
<http://git.reviewboard.kde.org/r/105879/#comment13415>

    missing {} around block



kabc/vcardparser/vcardparser.cpp
<http://git.reviewboard.kde.org/r/105879/#comment13416>

    {} around block


- Kevin Krammer


On Aug. 8, 2012, 6:03 p.m., Guy Maurel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105879/
> -----------------------------------------------------------
> 
> (Updated Aug. 8, 2012, 6:03 p.m.)
> 
> 
> Review request for KDEPIM-Libraries and Kevin Krammer.
> 
> 
> Description
> -------
> 
> same as kdepimlibs/akonadi
> 
> 
> Diffs
> -----
> 
>   kabc/address.cpp 112c59e 
>   kabc/addressbook.cpp 8b43a25 
>   kabc/addresseedialog.cpp a254c8e 
>   kabc/addresseehelper.cpp c422850 
>   kabc/addresseelist.cpp 1c3c107 
>   kabc/addresslineedit.cpp 16568b2 
>   kabc/distributionlist.cpp b36368d 
>   kabc/distributionlistdialog.cpp fd3a027 
>   kabc/emailselectdialog.cpp f413365 
>   kabc/errorhandler.cpp eb92233 
>   kabc/formats/binaryformat.cpp e1fdf31 
>   kabc/ldif.cpp 19addd9 
>   kabc/ldifconverter.cpp ea28614 
>   kabc/locknull.cpp eff1aa5 
>   kabc/phonenumber.cpp 9a7d42f 
>   kabc/picture.cpp 0a51c8b 
>   kabc/plugins/dir/resourcedir.cpp 0635023 
>   kabc/plugins/dir/resourcedirconfig.cpp 5c7e556 
>   kabc/plugins/file/resourcefile.cpp 77a3511 
>   kabc/plugins/file/resourcefileconfig.cpp 90105d4 
>   kabc/plugins/ldapkio/resourceldapkio.cpp 8346a6a 
>   kabc/plugins/ldapkio/resourceldapkioconfig.cpp 729fe77 
>   kabc/plugins/net/resourcenetconfig.cpp fe0f0f5 
>   kabc/resource.cpp 5cc5204 
>   kabc/resourcecached.cpp 9a6f993 
>   kabc/scripts/addressee.src.h 90a858c 
>   kabc/scripts/addressee.src.cpp 85b7826 
>   kabc/scripts/field.src.cpp e1048fb 
>   kabc/stdaddressbook.h fe2f69b 
>   kabc/stdaddressbook.cpp 31a9eef 
>   kabc/tests/addresstest.cpp 2567b9c 
>   kabc/tests/phonenumbertest.cpp b941b99 
>   kabc/tests/testaddresseelist.cpp 62c49d2 
>   kabc/tests/testkabc.cpp db17095 
>   kabc/tests/testkabcdlg.cpp f70089b 
>   kabc/vcardformat.cpp 321184f 
>   kabc/vcardparser/testread2.cpp f93e857 
>   kabc/vcardparser/testroundtrip.cpp e13945b 
>   kabc/vcardparser/testwrite.cpp f725bd2 
>   kabc/vcardparser/vcard.cpp d8bd13c 
>   kabc/vcardparser/vcardline.cpp 567ece6 
>   kabc/vcardparser/vcardparser.cpp 44c860e 
>   kabc/vcardtool.cpp ed32ba8 
> 
> Diff: http://git.reviewboard.kde.org/r/105879/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