Don't expect to always be able to allocate the pixel buffer

Summary: When reading compressed images, we don't always know the content of the compressed data, and we shouldn't warn about it. Reviewed By: jstraub Differential Revision: D52140732 fbshipit-source-id: 3e42d08534502ef330443cd8926fc5fa79c2cb25
facebookresearch · Dec 14, 2023 · 2b905e6 · 2b905e6
1 parent 2a41e76
commit 2b905e6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/csrc/reader/MultiVRSReader.cpp b/csrc/reader/MultiVRSReader.cpp
@@ -210,8 +210,7 @@ bool OssMultiVRSReader::MultiVideoRecordFormatStreamPlayer::setBlock(
         record.reader->read(data);
         block.structuredArray = false;
       } else {
-        std::shared_ptr<utils::PixelFrame> frame =
-            make_shared<utils::PixelFrame>(contentBlock.image());
+        std::shared_ptr<utils::PixelFrame> frame = make_shared<utils::PixelFrame>();
         bool frameValid = readFrame(*frame, record, contentBlock);
         if (XR_VERIFY(frameValid)) {
           block.structuredArray = true;

diff --git a/csrc/reader/VRSReader.cpp b/csrc/reader/VRSReader.cpp
@@ -298,7 +298,7 @@ bool OssVRSReader::setBlock(
         record.reader->read(data);
         block.structuredArray = false;
       } else {
-        shared_ptr<utils::PixelFrame> frame = make_shared<utils::PixelFrame>(contentBlock.image());
+        shared_ptr<utils::PixelFrame> frame = make_shared<utils::PixelFrame>();
         bool frameValid = readFrame(*frame, record, contentBlock);
         if (XR_VERIFY(frameValid)) {
           block.structuredArray = true;