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.