Minor code improvements

MediaPlayer.cpp

@@ -543,7 +543,7 @@ bool MediaPlayer::isImage() const
     return media_.isimage;
 }
 
-std::string MediaPlayer::hardwareDecoderName()
+std::string MediaPlayer::hardwareDecoderName() const
 {
     return hardware_decoder_;
 }

MediaPlayer.h

@@ -188,6 +188,10 @@ public:
      * Seek to zero
      * */
     void rewind();
+    /**
+     * Get position time
+     * */
+    GstClockTime position();
     /**
      * go to a valid position in media timeline
      * pos in nanoseconds.
@@ -210,10 +214,6 @@ public:
     void setTimeline(const Timeline &tl);
 
     float currentTimelineFading();
-    /**
-     * Get position time
-     * */
-    GstClockTime position();
     /**
      * Get framerate of the media
      * */
@@ -245,7 +245,7 @@ public:
      * Get the name of the hardware decoder used
      * Empty string if none (i.e. software decoding)
      * */
-    std::string hardwareDecoderName();
+    std::string hardwareDecoderName() const;
     /**
      * Forces open using software decoding
      * (i.e. without hadrware decoding)

Mixer.cpp

@@ -400,7 +400,7 @@ void Mixer::insertSource(Source *s, View::Mode m)
         attach(s);
 
         // new state in history manager
-        Action::manager().store(s->name() + std::string(" source inserted"));
+        Action::manager().store(s->name() + std::string(": source inserted"));
 
         // if requested to show the source in a given view
         // (known to work for View::MIXING et TRANSITION: other views untested)

Mixer.h

@@ -40,7 +40,7 @@ public:
 
     // update session and all views
     void update();
-    inline float dt() const { return dt_;}
+    inline float dt() const { return dt_;} // in miliseconds
     inline int fps() const { return int(roundf(1000.f/dt__));}
 
     // draw session and current view