[Kde-pim] Review Request 109637: remove the <TAB>s for kdepimlibs/kalarm

Guy Maurel guy-kde at maurel.de
Thu Mar 21 17:26:45 GMT 2013



> On March 21, 2013, 9:48 a.m., David Jarvie wrote:
> > Could you please replace each tab with 4 spaces instead of 8. That will keep the indentation consistent both within the file and within the library.

Well! The "standard" value for TAB is 8.
One the the next step is to correct all the intentations.
If it is possible for you "to wait", it will be corrected at that step.
Do you?


- Guy


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


On March 21, 2013, 8:16 a.m., Guy Maurel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109637/
> -----------------------------------------------------------
> 
> (Updated March 21, 2013, 8:16 a.m.)
> 
> 
> Review request for KDEPIM-Libraries, David Jarvie and Kevin Krammer.
> 
> 
> Description
> -------
> 
> According the discussion in Berlin KDEPIM-Meeting and http://techbase.kde.org/Policies/Kdepim_Coding_Style#Migration
> I put the <TAB>s away
> 
> 
> Diffs
> -----
> 
>   kalarmcal/kacalendar.cpp cbe1c5b 
> 
> Diff: http://git.reviewboard.kde.org/r/109637/diff/
> 
> 
> Testing
> -------
> 
> Tested for CMAKE_BUILD_TYPE "release" and"debug" with http://techbase.kde.org/Policies/Kdepim_Coding_Style#Check_the_objects
> 
> 
> 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