From 4e1bebf0f5e06ecdd99c41771e7b4ee19c0cdcf3 Mon Sep 17 00:00:00 2001 From: Jan Holthuis Date: Tue, 17 Nov 2020 20:02:14 +0100 Subject: Pass non-trivial/large function args by reference Fixes a bunch of `-Wclazy-function-args-by-ref` warnings. See https://github.com/KDE/clazy/blob/master/docs/checks/README-function-args-by-ref.md for details. Most of these changes are automated, using the clazy fixit by setting the `CLAZY_EXPORT_FIXES` before compilation and then running: $ mkdir /tmp/patch $ find ../src -iname "*.yaml" -exec mv {} /tmp/patch \; $ clang-apply-replacements /tmp/patch --- src/encoder/encoderrecordingsettings.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/encoder/encoderrecordingsettings.h') diff --git a/src/encoder/encoderrecordingsettings.h b/src/encoder/encoderrecordingsettings.h index 6038d9e00c..47e1c38e1f 100644 --- a/src/encoder/encoderrecordingsettings.h +++ b/src/encoder/encoderrecordingsettings.h @@ -28,7 +28,7 @@ class EncoderRecordingSettings : public EncoderSettings { // Selects the option by its index. If it is a single-element option, // index 0 means disabled and 1 enabled. - virtual void setGroupOption(QString groupCode, int optionIndex) { + virtual void setGroupOption(const QString& groupCode, int optionIndex) { Q_UNUSED(groupCode); Q_UNUSED(optionIndex); DEBUG_ASSERT(!"unimplemented"); -- cgit v1.2.3