Log a warning when we kill a client that is not keeping up.

[nageru] / futatabi / frame_on_disk.h

diff --git a/futatabi/frame_on_disk.h b/futatabi/frame_on_disk.h
index 7807c52b4a69cdb2a08819bb10636f9e59bdc44e..35f375bb78ef37a6522b351cea981e4a0643bf4e 100644 (file)
--- a/futatabi/frame_on_disk.h
+++ b/futatabi/frame_on_disk.h
@@ -1,21 +1,22 @@
 #ifndef _FRAME_ON_DISK_H
 #define _FRAME_ON_DISK_H 1
 
+#include "defs.h"
+
 #include <algorithm>
 #include <mutex>
+#include <stdint.h>
 #include <string>
 #include <vector>
 
-#include <stdint.h>
-
-#include "defs.h"
-
 extern std::mutex frame_mu;
 struct FrameOnDisk {
-        int64_t pts = -1;  // -1 means empty.
-        off_t offset;
-        unsigned filename_idx;
-        uint32_t size;  // Not using size_t saves a few bytes; we can have so many frames.
+       int64_t pts = -1;  // -1 means empty.
+       off_t offset;
+       unsigned filename_idx;
+       uint32_t size;  // Not using size_t saves a few bytes; we can have so many frames. TODO: Not anymore due to audio_size.
+       uint32_t audio_size;
+       // Unfortunately, 32 bits wasted in padding here.
 };
 extern std::vector<FrameOnDisk> frames[MAX_STREAMS];  // Under frame_mu.
 extern std::vector<std::string> frame_filenames;  // Under frame_mu.
@@ -25,7 +26,8 @@ static bool inline operator==(const FrameOnDisk &a, const FrameOnDisk &b)
        return a.pts == b.pts &&
                a.offset == b.offset &&
                a.filename_idx == b.filename_idx &&
-               a.size == b.size;
+               a.size == b.size &&
+               a.audio_size == b.audio_size;
 }
 
 // A helper class to read frames from disk. It caches the file descriptor
@@ -33,11 +35,18 @@ static bool inline operator==(const FrameOnDisk &a, const FrameOnDisk &b)
 // the sequential reads. (For this reason, each display has a private
 // FrameReader. Thus, we can easily keep multiple open file descriptors around
 // for a single .frames file.)
+//
+// Thread-compatible, but not thread-safe.
 class FrameReader {
 public:
        FrameReader();
        ~FrameReader();
-       std::string read_frame(FrameOnDisk frame);
+
+       struct Frame {
+               std::string video;
+               std::string audio;
+       };
+       Frame read_frame(FrameOnDisk frame, bool read_video, bool read_audio);
 
 private:
        int fd = -1;
@@ -49,14 +58,14 @@ inline std::vector<FrameOnDisk>::iterator
 find_last_frame_before(std::vector<FrameOnDisk> &frames, int64_t pts_origin)
 {
        return std::lower_bound(frames.begin(), frames.end(), pts_origin,
-               [](const FrameOnDisk &frame, int64_t pts) { return frame.pts < pts; });
+                               [](const FrameOnDisk &frame, int64_t pts) { return frame.pts < pts; });
 }
 
 inline std::vector<FrameOnDisk>::iterator
 find_first_frame_at_or_after(std::vector<FrameOnDisk> &frames, int64_t pts_origin)
 {
        return std::upper_bound(frames.begin(), frames.end(), pts_origin - 1,
-               [](int64_t pts, const FrameOnDisk &frame) { return pts < frame.pts; });
+                               [](int64_t pts, const FrameOnDisk &frame) { return pts < frame.pts; });
 }
 
 #endif  // !defined(_FRAME_ON_DISK_H)