remove unused avioContextHolder_

Author: Daniel Flores

src/torchcodec/_core/Encoder.cpp

@@ -555,18 +555,10 @@ TensorFormat analyzeTensorFormat(const torch::Tensor& frames) {
 } // namespace
 
 VideoEncoder::~VideoEncoder() {
-  close_avio();
-}
-
-void VideoEncoder::close_avio() {
   if (avFormatContext_ && avFormatContext_->pb) {
     avio_flush(avFormatContext_->pb);
-
-    if (!avioContextHolder_) {
-      avio_close(avFormatContext_->pb);
-      // avoids closing again in destructor, which would segfault.
-      avFormatContext_->pb = nullptr;
-    }
+    avio_close(avFormatContext_->pb);
+    avFormatContext_->pb = nullptr;
   }
 }
 
@@ -779,7 +771,7 @@ void VideoEncoder::encode() {
       "Error in av_write_trailer: ",
       getFFMPEGErrorStringFromErrorCode(status));
 
-  close_avio();
+  // close_avio();
 }
 
 void VideoEncoder::encodeFrame(

src/torchcodec/_core/Encoder.h

@@ -68,17 +68,8 @@ class VideoEncoder {
       std::string_view fileName,
       const VideoStreamOptions& videoStreamOptions);
 
-  VideoEncoder(
-      const torch::Tensor& frames,
-      int frameRate,
-      std::string_view formatName,
-      std::unique_ptr<AVIOContextHolder> avioContextHolder,
-      const VideoStreamOptions& videoStreamOptions);
-
   void encode();
 
-  torch::Tensor encodeToTensor();
-
  private:
   void initializeEncoder(const VideoStreamOptions& videoStreamOptions);
   UniqueAVFrame convertTensorToAVFrame(
@@ -105,10 +96,7 @@ class VideoEncoder {
   int outHeight_ = -1;
   AVPixelFormat outPixelFormat_ = AV_PIX_FMT_NONE;
 
-  std::unique_ptr<AVIOContextHolder> avioContextHolder_;
-
   bool encodeWasCalled_ = false;
-  int64_t lastEncodedAVFramePts_ = 0;
 };
 } // namespace facebook::torchcodec