swaroop: various logging fixes.

5 files changed, 18 insertions, 2 deletions

diff --git a/src/wx/controls.cc b/src/wx/controls.cc
index 3df62fd31..01b689c9e 100644
--- a/src/wx/controls.cc
+++ b/src/wx/controls.cc
@@ -207,6 +207,12 @@ Controls::slider_moved (bool page)
 	}
 	_viewer->seek (t, false);
 	update_position_label ();
+
+	log (
+		wxString::Format(
+			"playback-seeked %s", t.timecode(_film->video_frame_rate()).c_str()
+			)
+		);
 }
 
 void
diff --git a/src/wx/controls.h b/src/wx/controls.h
index fd6181d91..7ffe0c4ea 100644
--- a/src/wx/controls.h
+++ b/src/wx/controls.h
@@ -51,6 +51,7 @@ public:
 		bool editor_controls = true
 		);
 
+	virtual void log (wxString) {}
 	virtual void set_film (boost::shared_ptr<Film> film);
 	boost::shared_ptr<Film> film () const;
 	void back_frame ();
diff --git a/src/wx/film_viewer.cc b/src/wx/film_viewer.cc
index 90f8c9fd2..5c1b59ad6 100644
--- a/src/wx/film_viewer.cc
+++ b/src/wx/film_viewer.cc
@@ -603,7 +603,6 @@ FilmViewer::seek (DCPTime t, bool accurate)
 	}
 
 	PositionChanged ();
-	Seeked (position());
 }
 
 void
diff --git a/src/wx/film_viewer.h b/src/wx/film_viewer.h
index 73c923c26..123172ad2 100644
--- a/src/wx/film_viewer.h
+++ b/src/wx/film_viewer.h
@@ -92,7 +92,6 @@ public:
 	boost::signals2::signal<void ()> PositionChanged;
 	boost::signals2::signal<void (DCPTime)> Started;
 	boost::signals2::signal<void (DCPTime)> Stopped;
-	boost::signals2::signal<void (DCPTime)> Seeked;
 	/** While playing back we reached the end of the film (emitted from GUI thread) */
 	boost::signals2::signal<void ()> Finished;
 
diff --git a/src/wx/swaroop_controls.cc b/src/wx/swaroop_controls.cc
index 62d9d0300..f93cc66f1 100644
--- a/src/wx/swaroop_controls.cc
+++ b/src/wx/swaroop_controls.cc
@@ -262,6 +262,15 @@ SwaroopControls::log (wxString s)
 	strftime (buffer, 64, "%c", t);
 	wxString ts = std_to_wx(string(buffer)) + N_(": ");
 	_log->SetValue(_log->GetValue() + ts + s + "\n");
+
+	optional<boost::filesystem::path> log = Config::instance()->player_log_file();
+	if (!log) {
+		return;
+	}
+
+	FILE* f = fopen_boost (*log, "a");
+	fprintf (f, "%s%s\n", wx_to_std(ts).c_str(), wx_to_std(s).c_str());
+	fclose (f);
 }
 
 void
@@ -385,6 +394,8 @@ SwaroopControls::spl_selection_changed ()
 		return;
 	}
 
+	log (wxString::Format("load-playlist %s", std_to_wx(_playlists[selected].name()).data()));
+
 	wxProgressDialog dialog (_("DCP-o-matic"), "Loading playlist and KDMs");
 
 	BOOST_FOREACH (SPLEntry const & i, _playlists[selected].get()) {