>On 13 October 2011 16:53, Ralf Jung <[email protected]> wrote: > Hi, > >> No tabs, 4 spaces instead. >> >> http://techbase.kde.org/Policies/Kdelibs_Coding_Style > Almost the complete main.cpp is using tabs currently (except for > KCMUserAccount::decodeImgDrop, which uses 2 spaces) so I used it for the two > lines I added. Is that okay, or am I supposed to use 4 spaces nevertheless?
It's up to the maintainer of the code to accept your code (and he probably won't care about the coding style this time since every line but the last two you added were _already_ using tabs and 2 spaces), but why not use the astyle script to format the files, so everything is correct? Upload a new version of your patch but with only style changes. > > Kind regards, > Ralf >
