Fix some Clang warnings.

[nageru] / player.cpp

diff --git a/player.cpp b/player.cpp
index 5f6af62964685ba34c0e26728ae5517e56cc22bf..3a958917fff5639942bba4c8e713c562044bd2d5 100644 (file)
--- a/player.cpp
+++ b/player.cpp
@@ -89,7 +89,6 @@ void Player::thread_func(bool also_output_to_stream)
                // TODO: Lock to a rational multiple of the frame rate if possible.
                double speed = 0.5;
 
-               bool aborted = false;
                for (int frameno = 0; ; ++frameno) {  // Ends when the clip ends.
                        double out_pts = out_pts_origin + TIMEBASE * frameno / output_framerate;
                        steady_clock::time_point next_frame_start =
@@ -134,7 +133,7 @@ void Player::thread_func(bool also_output_to_stream)
                        if (in_pts_lower == in_pts_upper) {
                                destination->setFrame(stream_idx, in_pts_lower, /*interpolated=*/false);
                                if (video_stream != nullptr) {
-                                       video_stream->schedule_original_frame(lrint(out_pts), stream_idx, in_pts_lower);
+                                       video_stream->schedule_original_frame(pts, stream_idx, in_pts_lower);
                                }
                                continue;
                        }
@@ -147,7 +146,7 @@ void Player::thread_func(bool also_output_to_stream)
                                if (fabs(snap_pts_as_frameno - frameno) < 0.01) {
                                        destination->setFrame(stream_idx, snap_pts, /*interpolated=*/false);
                                        if (video_stream != nullptr) {
-                                               video_stream->schedule_original_frame(lrint(out_pts), stream_idx, snap_pts);
+                                               video_stream->schedule_original_frame(pts, stream_idx, snap_pts);
                                        }
                                        in_pts_origin += snap_pts - in_pts;
                                        snapped = true;
@@ -172,8 +171,8 @@ void Player::thread_func(bool also_output_to_stream)
                        } else {
                                // Calculate the interpolated frame. When it's done, the destination
                                // will be unblocked.
-                               destination->setFrame(stream_idx, lrint(out_pts), /*interpolated=*/true);
-                               video_stream->schedule_interpolated_frame(lrint(out_pts), stream_idx, in_pts_lower, in_pts_upper, alpha);
+                               destination->setFrame(stream_idx, pts, /*interpolated=*/true);
+                               video_stream->schedule_interpolated_frame(pts, stream_idx, in_pts_lower, in_pts_upper, alpha);
                        }
                }
 
@@ -181,7 +180,7 @@ void Player::thread_func(bool also_output_to_stream)
                        unique_lock<mutex> lock(queue_state_mu);
                        playing = false;
                }
-               if (done_callback != nullptr && !aborted) {
+               if (done_callback != nullptr) {
                        done_callback();
                }
        }