Use const'ref, change if(foo) delete foo => delete foo not necessary to check it...

[kdenlive] / src / docclipbase.h

diff --git a/src/docclipbase.h b/src/docclipbase.h
index 0e22c4dad85a311889bbc874d006f6e314a3f0b8..e4f90fceb480a993cd30de4005945edd627221e2 100644 (file)
--- a/src/docclipbase.h
+++ b/src/docclipbase.h
@@ -89,7 +89,7 @@ public:
     const GenTime & duration() const;
     const GenTime maxDuration() const;
     /** returns the duration of this clip */
-    void setDuration(GenTime dur);
+    void setDuration(const GenTime &dur);
 
     /** returns clip type (audio, text, image,...) */
     const CLIPTYPE & clipType() const;
@@ -271,8 +271,8 @@ public slots:
     GenTime findPreviousSnapMarker(const GenTime & currTime);
     GenTime hasSnapMarkers(const GenTime & time);
     QString deleteSnapMarker(const GenTime & time);
-    void editSnapMarker(const GenTime & time, QString comment);
-    void addSnapMarker(const CommentedTime marker);
+    void editSnapMarker(const GenTime & time, const QString &comment);
+    void addSnapMarker(const CommentedTime &marker);
     QList < GenTime > snapMarkers() const;
     QString markerComment(GenTime t) const;
     CommentedTime markerAt(GenTime t) const;