[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-devel
Subject: Re: Code in peace or How to lint a complete repository?
From: Nicolás_Alvarez <nicolas.alvarez () gmail ! com>
Date: 2018-02-28 19:44:07
Message-ID: CANPC-tuaq_fXyEK-HveARA_FPmoLFqU9seRxTLdzGdcUVuTOVw () mail ! gmail ! com
[Download RAW message or body]
2018-02-28 6:57 GMT-03:00 Michael Heidelbach <ottwolt@gmail.com>:
> Hi!
>
> Now that I saw https://phabricator.kde.org/D10905, I want to do something
> similar for baloo and baloo-widgets.
>
> I'd really like to that once and for all. Because applying coding style
> on-the-fly obscures real changes introduced.
>
> 1. Would it be reasonable?
> 2. Who would be willing to review such a huge diff?
> 3. How to apply KDE coding style most effectively?
>
> (The astyle command given in Kdelibs Coding Style does not produce
> completely correct results:
> [this](const QUrl &url) => [this](const QUrl & url) instead of
> [this](const QUrl& url)
> Also the operators-at-beginning-of-line rule is not respected )
>
> 4. What about arcanist: any ready-made .arclint available?
>
> Cheers,
>
> Michael
Just one suggestion: Provide the settings and version number of the
tool you used, both in the review and in the commit message, so that
other people can reproduce the exact same result.
--
Nicolás
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic