Update a comment after the previous commit.

[nageru] / futatabi / player.h

diff --git a/futatabi/player.h b/futatabi/player.h
index 22de0b396cd98be13d0f3dba1049882f616e24a4..41dcfb884ae10ab7f98a72786181eacddc435eea 100644 (file)
--- a/futatabi/player.h
+++ b/futatabi/player.h
@@ -32,8 +32,8 @@ public:
        Player(JPEGFrameView *destination, StreamOutput stream_output, AVFormatContext *file_avctx = nullptr);
        ~Player();
 
-       void play_clip(const Clip &clip, size_t clip_idx, unsigned stream_idx);
-       void override_angle(unsigned stream_idx);  // For the current clip only.
+       void play(const std::vector<Clip> &clips);
+       void override_angle(unsigned stream_idx);  // Assumes one-clip playlist only.
 
        // Not thread-safe to set concurrently with playing.
        // Will be called back from the player thread.
@@ -42,12 +42,7 @@ public:
 
        // Not thread-safe to set concurrently with playing.
        // Will be called back from the player thread.
-       // The second parameter is the clip's position in the play list.
-       using next_clip_callback_func = std::function<std::pair<Clip, size_t>()>;
-       void set_next_clip_callback(next_clip_callback_func cb) { next_clip_callback = cb; }
-
-       // Not thread-safe to set concurrently with playing.
-       // Will be called back from the player thread.
+       // The keys in the given map are indexes in the vector given to play().
        using progress_callback_func = std::function<void(const std::map<size_t, double> &progress)>;
        void set_progress_callback(progress_callback_func cb) { progress_callback = cb; }
 
@@ -70,22 +65,28 @@ private:
 
        JPEGFrameView *destination;
        done_callback_func done_callback;
-       next_clip_callback_func next_clip_callback;
        progress_callback_func progress_callback;
 
-       std::mutex mu;
-       Clip current_clip;  // Under mu. Can have pts_in = -1 for no clip.
-       size_t current_clip_idx;  // Under mu.
-       unsigned current_stream_idx;  // Under mu.
-
        std::mutex queue_state_mu;
        std::condition_variable new_clip_changed;
+       std::vector<Clip> queued_clip_list;   // Under queue_state_mu.
        bool new_clip_ready = false;  // Under queue_state_mu.
        bool playing = false;  // Under queue_state_mu.
        int override_stream_idx = -1;  // Under queue_state_mu.
+       int64_t last_pts_played = -1;  // Under queue_state_mu. Used by previews only.
 
        std::unique_ptr<VideoStream> video_stream;  // Can be nullptr.
 
+       std::atomic<int64_t> metric_dropped_interpolated_frame{0};
+       std::atomic<int64_t> metric_dropped_unconditional_frame{0};
+       std::atomic<int64_t> metric_faded_frame{0};
+       std::atomic<int64_t> metric_faded_snapped_frame{0};
+       std::atomic<int64_t> metric_original_frame{0};
+       std::atomic<int64_t> metric_original_snapped_frame{0};
+       std::atomic<int64_t> metric_refresh_frame{0};
+       std::atomic<int64_t> metric_interpolated_frame{0};
+       std::atomic<int64_t> metric_interpolated_faded_frame{0};
+
        // under queue_state_mu. Part of this instead of VideoStream so that we own
        // its lock and can sleep on it.
        size_t num_queued_frames = 0;