D20572: RFC: Add .clang-format
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag added a comment.
In D20572#450547 <https://phabricator.kde.org/D20572#450547>, @apol wrote:
> We could give it a go, how do you test it?
One way would be
clang-format -style=file <filename>
Though most text editors have clang-format integration, which makes things much easier. For example, Qt Creator has clang-format support out-of-box, no need to install plugins, etc.
> Do you suggest having it in git hooks?
Well, this is an option.
> Can we use it only on the places that change?
Might be worth to look into http://clang.llvm.org/docs/ClangFormat.html#script-for-patch-reformatting.
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D20572
To: zzag, #kwin
Cc: apol, kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, mart