]> git.sesse.net Git - nageru/blobdiff - futatabi/mainwindow.cpp
Fix an issue where the stop button would get grayed out if restarting playing.
[nageru] / futatabi / mainwindow.cpp
index bbaa5a417ab8c9c636d76f324f3b396e7ef236b4..d73f579a9e6faee5880c6ac2447004a1df6cc99c 100644 (file)
@@ -186,10 +186,10 @@ MainWindow::MainWindow()
        live_player.reset(new Player(ui->live_display, Player::HTTPD_STREAM_OUTPUT));
        live_player->set_done_callback([this] {
                post_to_main_thread([this] {
-                       live_player_clip_done();
+                       live_player_done();
                });
        });
-       live_player->set_progress_callback([this](const map<size_t, double> &progress, double time_remaining) {
+       live_player->set_progress_callback([this](const map<uint64_t, double> &progress, double time_remaining) {
                post_to_main_thread([this, progress, time_remaining] {
                        live_player_clip_progress(progress, time_remaining);
                });
@@ -431,6 +431,14 @@ void MainWindow::defer_timer_expired()
 
 void MainWindow::content_changed()
 {
+       // If we are playing, update the part of the playlist that's not playing yet.
+       vector<ClipWithID> clips;
+       for (unsigned row = 0; row < playlist_clips->size(); ++row) {
+               clips.emplace_back(*playlist_clips->clip_with_id(row));
+       }
+       live_player->splice_play(clips);
+
+       // Serialize the state.
        if (defer_timeout->isActive() &&
            (!currently_deferring_model_changes || deferred_change_id != current_change_id)) {
                // There's some deferred event waiting, but this event is unrelated.
@@ -487,13 +495,13 @@ void MainWindow::play_clicked()
                start_row = selected->selectedRows(0)[0].row();
        }
 
-       vector<Player::ClipWithRow> clips;
+       vector<ClipWithID> clips;
        for (unsigned row = start_row; row < playlist_clips->size(); ++row) {
-               clips.emplace_back(Player::ClipWithRow{ *playlist_clips->clip(row), row });
+               clips.emplace_back(*playlist_clips->clip_with_id(row));
        }
        live_player->play(clips);
        playlist_clips->set_progress({ { start_row, 0.0f } });
-       playlist_clips->set_currently_playing(start_row, 0.0f);
+       ui->playlist->selectionModel()->clear();
        playlist_selection_changed();
 
        ui->stop_btn->setEnabled(true);
@@ -504,43 +512,16 @@ void MainWindow::stop_clicked()
        Clip fake_clip;
        fake_clip.pts_in = 0;
        fake_clip.pts_out = 0;
-       size_t last_row = playlist_clips->size() - 1;
-       playlist_clips->set_currently_playing(last_row, 0.0f);
+       playlist_clips->set_progress({});
        live_player->play(fake_clip);
-}
-
-void MainWindow::live_player_clip_done()
-{
-       int row = playlist_clips->get_currently_playing();
-       if (row == -1 || row == int(playlist_clips->size()) - 1) {
-               set_output_status("paused");
-               playlist_clips->set_progress({});
-               playlist_clips->set_currently_playing(-1, 0.0f);
-       } else {
-               playlist_clips->set_progress({ { row + 1, 0.0f } });
-               playlist_clips->set_currently_playing(row + 1, 0.0f);
-       }
        ui->stop_btn->setEnabled(false);
 }
 
-pair<Clip, size_t> MainWindow::live_player_get_next_clip()
+void MainWindow::live_player_done()
 {
-       // playlist_clips can only be accessed on the main thread.
-       // Hopefully, we won't have to wait too long for this to come back.
-       //
-       // TODO: If MainWindow is in the process of being destroyed and waiting
-       // for Player to shut down, we could have a deadlock here.
-       promise<pair<Clip, size_t>> clip_promise;
-       future<pair<Clip, size_t>> clip = clip_promise.get_future();
-       post_to_main_thread([&clip_promise] {
-               int row = playlist_clips->get_currently_playing();
-               if (row != -1 && row < int(playlist_clips->size()) - 1) {
-                       clip_promise.set_value(make_pair(*playlist_clips->clip(row + 1), row + 1));
-               } else {
-                       clip_promise.set_value(make_pair(Clip(), 0));
-               }
-       });
-       return clip.get();
+       playlist_selection_changed();
+       playlist_clips->set_progress({});
+       ui->stop_btn->setEnabled(false);
 }
 
 static string format_duration(double t)
@@ -558,7 +539,7 @@ static string format_duration(double t)
        return buf;
 }
 
-void MainWindow::live_player_clip_progress(const map<size_t, double> &progress, double time_remaining)
+void MainWindow::live_player_clip_progress(const map<uint64_t, double> &progress, double time_remaining)
 {
        playlist_clips->set_progress(progress);
        set_output_status(format_duration(time_remaining) + " left");
@@ -843,9 +824,9 @@ void MainWindow::playlist_selection_changed()
        if (!any_selected) {
                set_output_status("paused");
        } else {
-               vector<Player::ClipWithRow> clips;
+               vector<ClipWithID> clips;
                for (size_t row = selected->selectedRows().front().row(); row < playlist_clips->size(); ++row) {
-                       clips.emplace_back(Player::ClipWithRow{ *playlist_clips->clip(row), row });
+                       clips.emplace_back(*playlist_clips->clip_with_id(row));
                }
                double remaining = compute_total_time(clips);
                set_output_status(format_duration(remaining) + " ready");