Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-14 Thread Albert Astals Cid
El dijous, 11 de juliol de 2019, a les 16:18:08 CEST, David Edmundson va escriure: > One topic discussed at the recent Plasma sprint was that we should run > a code formatting tool (clang-format) over all our repos to ease all > future review comments about whitespace. > > All new contributions

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-14 Thread Alexey Min
> There's one comment about constructor initialisers, which seems resolvable. If that solvable, then it's great. I'm in favour of this idea, maybe even extend to frameworks repos? > So if there's anything else, then yes, please be more specific. No, so far that was the only problem we've

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-14 Thread David Edmundson
>Aren't the problems described in task https://phabricator.kde.org/T11214 specific enough? There's one comment about constructor initialisers, which seems resolvable. So if there's anything else, then yes, please be more specific. If we find we we can't do this until we get some more features

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-13 Thread Alexey Min
Aren't the problems described in task https://phabricator.kde.org/T11214 specific enough? сб, 13 июл. 2019 г. в 14:18, Vlad Zagorodniy : > Can you be more specific about it? What parts of the kdelibs/Frameworks > coding style can't be implemented/enforced with clang-format? >

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-13 Thread Vlad Zagorodniy
On 7/13/19 1:01 AM, Allan Sandfeld Jensen wrote: I would argue against it. clang-format is extremely primitive and stupid, it can't implement KDE or Qt coding styles or anything close to i Can you be more specific about it? What parts of the kdelibs/Frameworks coding style can't be

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-12 Thread Allan Sandfeld Jensen
On Donnerstag, 11. Juli 2019 16:18:08 CEST David Edmundson wrote: > One topic discussed at the recent Plasma sprint was that we should run > a code formatting tool (clang-format) over all our repos to ease all > future review comments about whitespace. > > All new contributions simply have to run

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-12 Thread David Edmundson
> Presumably this would impose a common coding style throughout all KDE > repositories? Or would each project be able to specify its own clang-format > configuration? In past discussions about KDE coding style, it was accepted > that > while Frameworks and PIM libraries would adopt a common

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-12 Thread Martin Flöser
Am 2019-07-11 16:18, schrieb David Edmundson: One topic discussed at the recent Plasma sprint was that we should run a code formatting tool (clang-format) over all our repos to ease all future review comments about whitespace. All new contributions simply have to run the same tool and we get

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-11 Thread David Edmundson
On Thu, Jul 11, 2019 at 3:32 PM alcinos wrote: > > Hi, > > Just to mention that we have a .clang-format in Kdenlive as well > (https://invent.kde.org/kde/kdenlive/blob/master/.clang-format) Oh awesome, do you mind if I ask some questions? Did running it across the base cause any issues? Did it

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-11 Thread alcinos
Hi, Just to mention that we have a .clang-format in Kdenlive as well ( https://invent.kde.org/kde/kdenlive/blob/master/.clang-format) I designed it so that it would be consistent but match rather closely the dominant style in the code-base at the time I wrote it. Unfortunately, we don't

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-11 Thread Carl Schwan
Hi, I like the idea a lot and I would love to do the same for the websites. Probably using tools like [PHP Coding Standards Fixer](https://cs.symfony.com/). Cheers, Carl ‐‐‐ Original Message ‐‐‐ On Thursday 11 July 2019 16:18, David Edmundson wrote: > One topic discussed at the

Re: RFC: Running clang-format across all Plasma (and more?) repos

2019-07-11 Thread Hugo Pereira Da Costa
Hi David, On 7/11/19 4:18 PM, David Edmundson wrote: One topic discussed at the recent Plasma sprint was that we should run a code formatting tool (clang-format) over all our repos to ease all future review comments about whitespace. All new contributions simply have to run the same tool and

RFC: Running clang-format across all Plasma (and more?) repos

2019-07-11 Thread David Edmundson
One topic discussed at the recent Plasma sprint was that we should run a code formatting tool (clang-format) over all our repos to ease all future review comments about whitespace. All new contributions simply have to run the same tool and we get consistent code without having to comment on every