diff options
author | Ivan Komissarov <ABBAPOH@gmail.com> | 2019-11-24 16:14:53 +0900 |
---|---|---|
committer | Ivan Komissarov <ABBAPOH@gmail.com> | 2020-01-07 12:58:12 +0000 |
commit | 861c81ab8603a01c23e97543837670503538119e (patch) | |
tree | 2329f5c0e80db5db3bbe4b155853168643589078 /.clang-tidy | |
parent | 375ba8b23f0bf2b8639e9d7196f4451149798464 (diff) | |
download | qbs-861c81ab8603a01c23e97543837670503538119e.tar.gz |
Fix 'performance-unnecessary-value-param' warnings
Use more pass-by-const-ref since there is no point in passing objects by
value in these cases
Also, treat this warning as error
Change-Id: I5382dcd56639f535cf240205cc5c3ac667c4ece3
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to '.clang-tidy')
-rw-r--r-- | .clang-tidy | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/.clang-tidy b/.clang-tidy index f5da48683..1002e18bb 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -1,6 +1,6 @@ --- Checks: '-*,bugprone-*,cppcoreguidelines-c-copy-assignment-signature,cppcoreguidelines-interfaces-global-init,cppcoreguidelines-pro-bounds-constant-array-index,cppcoreguidelines-pro-type-cstyle-cast,cppcoreguidelines-pro-type-member-init,cppcoreguidelines-slicing,fuchsia-virtual-inheritance,google-build-explicit-make-pair,google-build-namespaces,google-global-names-in-headers,google-objc-*,google-readability-casting,google-readability-function-size,google-readability-namespace-comments,google-runtime-operator,misc-definitions-in-headers,misc-misplaced-const,misc-new-delete-overloads,misc-non-copyable-objects,misc-redundant-expression,misc-static-assert,misc-throw-by-value-catch-by-reference,misc-unconventional-assign-operator,misc-uniqueptr-reset-release,misc-unused-*,modernize-avoid-bind,modernize-deprecated-headers,modernize-loop-convert,modernize-make-*,modernize-redundant-void-arg,modernize-replace-*,modernize-return-braced-init-list,modernize-shrink-to-fit,modernize-unary-static-assert,modernize-use-auto,modernize-use-bool-literals,modernize-use-emplace,modernize-use-equals-*,modernize-use-noexcept,modernize-use-override,modernize-use-transparent-functors,modernize-use-using,performance-*,readability-avoid-const-params-in-decls,readability-container-size-empty,readability-delete-null-pointer,readability-deleted-default,readability-function-size,readability-identifier-naming,readability-misleading-indentation,readability-misplaced-array-index,readability-non-const-parameter,readability-redundant-*,readability-simplify-boolean-expr,readability-static-definition-in-anonymous-namespace,readability-uniqueptr-delete-release' -WarningsAsErrors: 'bugprone-*,-bugprone-suspicious-enum-usage,-bugprone-macro-parentheses,-bugprone-narrowing-conversions,modernize-make-*,modernize-use-emplace,modernize-use-equals-default,modernize-use-override,performance-move-const-arg' +WarningsAsErrors: 'bugprone-*,-bugprone-suspicious-enum-usage,-bugprone-macro-parentheses,-bugprone-narrowing-conversions,modernize-make-*,modernize-use-emplace,modernize-use-equals-default,modernize-use-override,performance-move-const-arg,performance-unnecessary-value-param' HeaderFilterRegex: '' AnalyzeTemporaryDtors: false CheckOptions: |