https://bugs.kde.org/show_bug.cgi?id=468401

--- Comment #3 from Petr Pavlu <petr.pa...@dagobah.cz> ---
(In reply to Paul Floyd from comment #1)
> Thanks Petr this looks like a good start. Do you think that this will also
> need TypenameMacros for the tool 'namespace' macros (MC_ etc.)?

Missed these, added in v2 of the patch.

> Any thoughts on adding this to git hooks?

Not sure, it might be hard to tell clang-format what it should exactly do from
a Git hook and it has some potential to annoy people that don't want to use the
tool.

I would start simply and only add a configuration file in the root directory.
This gives folks that want to use this tool an easy way to do so.

> I think it would also be a good idea to add a section to README_DEVELOPERS
> (difficult to be totally comprehensive, but things like which package
> provides clang-format, how to use it, IDE integration ...)

Makes sense, v2 adds some basic information in README_DEVELOPERS and a
reference to the upstream documentation for integration with text editors and
IDEs.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to