[Kde-pim] Review Request: Fixed remaning formatting, began focusing on explicit constructors

Kevin Krammer krammer at kde.org
Fri Dec 14 14:12:59 GMT 2012


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

Ship it!


Ship It!

- Kevin Krammer


On Dec. 14, 2012, 2:04 p.m., Kyle Morris wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107721/
> -----------------------------------------------------------
> 
> (Updated Dec. 14, 2012, 2:04 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Description
> -------
> 
> There were still some fixes I thought should be accounted for, regarding the formatting, however I decided to switch things up a bit near the end and look at a few other fixes. Total of 33 fixes. 
> 
> 
> Diffs
> -----
> 
>   calendarsupport/attachmenthandler.h 82f73b2 
>   calendarsupport/groupware.cpp e3a9bb3 
>   calendarsupport/incidenceattachmentmodel.h 15338af 
>   calendarsupport/kcalmodel.h 4261499 
>   calendarsupport/mailclient.cpp 9b169f1 
>   calendarsupport/next/invitationhandler.h 565a8af 
>   calendarsupport/utils.h e1188ee 
>   calendarviews/eventviews/month/monthview.cpp f7ff367 
>   kaddressbook/contactselectionwidget.h 3830caa 
>   kaddressbook/globalcontactmodel.h b2d3882 
>   kaddressbook/printing/compactstyle.cpp 45ea91c 
>   kaddressbook/printing/printingwizard.h 3ec6351 
>   kaddressbook/printing/ringbinderstyle.h 8341ff6 
>   kaddressbook/printing/ringbinderstyle.cpp 2c60cee 
>   kaddressbook/startup.cpp 1c3ea85 
>   kaddressbook/xxport/vcard/vcard_xxport.cpp a44b958 
>   korganizer/calendarview.cpp 815314d 
>   korganizer/kohelper.h 5d421cd 
>   korganizer/views/journalview/kojournalview.cpp 0d54e89 
> 
> Diff: http://git.reviewboard.kde.org/r/107721/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Kyle Morris
> 
>

_______________________________________________
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