[Kde-pim] Review Request 110220: Change the One-Line-If-Statement(s)

Guy Maurel guy-kde at maurel.de
Sun Jul 28 14:46:50 BST 2013


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

(Updated July 28, 2013, 1:46 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDEPIM-Libraries and Volker Krause.


Description
-------

According the discussion in Berlin KDEPIM-Meeting and http://techbase.kde.org/Policies/Kdepim_Coding_Style#Migration
I change the One-Line-If such as:
if ( a > b ) c = 123;
to a block:
if ( a > b ) {
   c = 123;
}


Diffs
-----

  akonadi/item.h 9aa4012 
  akonadi/socialutils/socialfeeditem.cpp 722536c 

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


Testing
-------

Tested for CMAKE_BUILD_TYPE "release" and "debug" with http://techbase.kde.org/Policies/Kdepim_Coding_Style#Check_the_assembler_files


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