summaryrefslogtreecommitdiffstats
path: root/src/mixer/playerinfo.h
diff options
context:
space:
mode:
authorJan Holthuis <jan.holthuis@ruhr-uni-bochum.de>2020-11-17 20:02:14 +0100
committerJan Holthuis <jan.holthuis@ruhr-uni-bochum.de>2020-11-19 19:46:54 +0100
commit4e1bebf0f5e06ecdd99c41771e7b4ee19c0cdcf3 (patch)
treed0e98cee9eef416282049092ea93b0b2b6eb417d /src/mixer/playerinfo.h
parent133ea1e3d6547964f014144eee15590ea449c7af (diff)
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
Diffstat (limited to 'src/mixer/playerinfo.h')
-rw-r--r--src/mixer/playerinfo.h16
1 files changed, 8 insertions, 8 deletions
diff --git a/src/mixer/playerinfo.h b/src/mixer/playerinfo.h
index 68cef8d636..7d6cb574c7 100644
--- a/src/mixer/playerinfo.h
+++ b/src/mixer/playerinfo.h
@@ -42,18 +42,18 @@ class PlayerInfo : public QObject {
signals:
void currentPlayingDeckChanged(int deck);
void currentPlayingTrackChanged(TrackPointer pTrack);
- void trackLoaded(QString group, TrackPointer pTrack);
- void trackUnloaded(QString group, TrackPointer pTrack);
+ void trackLoaded(const QString& group, TrackPointer pTrack);
+ void trackUnloaded(const QString& group, TrackPointer pTrack);
private:
class DeckControls {
public:
- DeckControls(QString& group)
- : m_play(group, "play"),
- m_pregain(group, "pregain"),
- m_volume(group, "volume"),
- m_orientation(group, "orientation") {
- }
+ DeckControls(const QString& group)
+ : m_play(group, "play"),
+ m_pregain(group, "pregain"),
+ m_volume(group, "volume"),
+ m_orientation(group, "orientation") {
+ }
ControlProxy m_play;
ControlProxy m_pregain;