Make HDR side data unique pointer

CrendKing · Jun 30, 2021 · 567f713 · 567f713 · ssssssbbb · Jul 2, 2021
1 parent a690e21
commit 567f713
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/avisynth_filter/src/frame_handler.cpp b/avisynth_filter/src/frame_handler.cpp
@@ -62,7 +62,7 @@ auto FrameHandler::AddInputSample(IMediaSample *inputSample) -> HRESULT {
 
     const PVideoFrame frame = Format::CreateFrame(_filter._inputVideoFormat, sampleBuffer);
 
-    std::shared_ptr<HDRSideData> hdrSideData = std::make_shared<HDRSideData>();
+    std::unique_ptr<HDRSideData> hdrSideData = std::make_unique<HDRSideData>();
     {
         if (const ATL::CComQIPtr<IMediaSideData> inputSampleSideData(inputSample); inputSampleSideData != nullptr) {
             hdrSideData->ReadFrom(inputSampleSideData);
@@ -89,7 +89,7 @@ auto FrameHandler::AddInputSample(IMediaSample *inputSample) -> HRESULT {
 
         _sourceFrames.emplace(std::piecewise_construct,
                               std::forward_as_tuple(_nextSourceFrameNb),
-                              std::forward_as_tuple(frame, inputSampleStartTime, hdrSideData));
+                              std::forward_as_tuple(frame, inputSampleStartTime, std::move(hdrSideData)));
     }
     _newSourceFrameCv.notify_all();
 

diff --git a/avisynth_filter/src/frame_handler.h b/avisynth_filter/src/frame_handler.h
@@ -34,7 +34,7 @@ class FrameHandler {
     struct SourceFrameInfo {
         PVideoFrame frame;
         REFERENCE_TIME startTime;
-        std::shared_ptr<HDRSideData> hdrSideData;
+        std::unique_ptr<HDRSideData> hdrSideData;
     };
 
     static auto RefreshFrameRatesTemplate(int sampleNb, int &checkpointSampleNb, DWORD &checkpointTime, int &currentFrameRate) -> void;

diff --git a/vapoursynth_filter/src/frame_handler.cpp b/vapoursynth_filter/src/frame_handler.cpp
@@ -66,7 +66,7 @@ auto FrameHandler::AddInputSample(IMediaSample *inputSample) -> HRESULT {
     AVSF_VS_API->propSetInt(frameProps, "_SARDen", _filter._inputVideoFormat.pixelAspectRatioDen, paReplace);
     AVSF_VS_API->propSetInt(frameProps, VS_PROP_NAME_SOURCE_FRAME_NB, _nextSourceFrameNb, paReplace);
 
-    std::shared_ptr<HDRSideData> hdrSideData = std::make_shared<HDRSideData>();
+    std::unique_ptr<HDRSideData> hdrSideData = std::make_unique<HDRSideData>();
     {
         if (const ATL::CComQIPtr<IMediaSideData> inputSampleSideData(inputSample); inputSampleSideData != nullptr) {
             hdrSideData->ReadFrom(inputSampleSideData);
@@ -93,7 +93,7 @@ auto FrameHandler::AddInputSample(IMediaSample *inputSample) -> HRESULT {
 
         _sourceFrames.emplace(std::piecewise_construct,
                               std::forward_as_tuple(_nextSourceFrameNb),
-                              std::forward_as_tuple(frame, inputSampleStartTime, hdrSideData));
+                              std::forward_as_tuple(frame, inputSampleStartTime, std::move(hdrSideData)));
     }
 
     Environment::GetInstance().Log(L"Stored source frame: %6i at %10lli ~ %10lli duration(literal) %10lli, last_used %6i, extra_buffer %6i",

diff --git a/vapoursynth_filter/src/frame_handler.h b/vapoursynth_filter/src/frame_handler.h
@@ -36,7 +36,7 @@ class FrameHandler {
 
         VSFrameRef *frame;
         REFERENCE_TIME startTime;
-        std::shared_ptr<HDRSideData> hdrSideData;
+        std::unique_ptr<HDRSideData> hdrSideData;
     };
 
     static auto VS_CC VpsGetFrameCallback(void *userData, const VSFrameRef *f, int n, VSNodeRef *node, const char *errorMsg) -> void;