diff options
author | Miklos Vajna <vmiklos@collabora.co.uk> | 2017-11-20 09:04:51 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2017-11-20 13:41:40 +0100 |
commit | 13de75274b727428355eefd55176277a5f891c47 (patch) | |
tree | 739dd3c29c2cf9d76d601326a403b1b6618e8930 /include/svl | |
parent | ofz#4334 Null-dereference READ (diff) | |
download | core-13de75274b727428355eefd55176277a5f891c47.tar.gz core-13de75274b727428355eefd55176277a5f891c47.zip |
clang-format: enforce coding style via Jenkins
- factor out common code to a shared module, and quote path to the
clang-format binary, just in case.
- add a new check-last-commit script that is the CI equivalent of the
exiting git pre-commit hook, but this one handles lack of clang-format
as an error, not as a warning.
- $LODE_HOME/opt/bin is supposed to be in PATH already, so not
mentioning LODE_HOME in ClangFormat::find() explicitly.
- if both COMPILER_PLUGINS and LODE_HOME is set, invoke
solenv/clang-format/check-last-commit as part of 'make check'
To test these changes as part of CI, fix a single style violation in an
already committed, non-blacklisted file.
This depends on the lode.git commit
496123bcae28e06c6d6aeda39a5afd1e1fb1fd98 (utils_Linux: install
clang-format in the Jenkins case, 2017-11-16), otherwise erroring out on
a not installed clang-format as part of the build would be a problem.
Change-Id: Ib3110826194ff78a7f1bed1c3796147e92ccb3ba
Reviewed-on: https://gerrit.libreoffice.org/44939
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Diffstat (limited to 'include/svl')
-rw-r--r-- | include/svl/typedwhich.hxx | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/svl/typedwhich.hxx b/include/svl/typedwhich.hxx index cb22d48ebdc3..d3f2bedec671 100644 --- a/include/svl/typedwhich.hxx +++ b/include/svl/typedwhich.hxx @@ -24,6 +24,7 @@ public: { } constexpr operator sal_uInt16() const { return mnWhich; } + private: sal_uInt16 const mnWhich; }; |