From 36801445118dee936d94fc5ee0194599191334bb Mon Sep 17 00:00:00 2001 From: "FeRD (Frank Dana)" Date: Sun, 22 Mar 2020 12:33:59 -0400 Subject: [PATCH] FrameMapper: Eliminate is_open member variable It was being used uninitialized because it's never actually set, therefore it serves no purpose. --- include/FrameMapper.h | 1 - src/FrameMapper.cpp | 6 +++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/include/FrameMapper.h b/include/FrameMapper.h index 8be4ec1b..ff4ccd71 100644 --- a/include/FrameMapper.h +++ b/include/FrameMapper.h @@ -138,7 +138,6 @@ namespace openshot */ class FrameMapper : public ReaderBase { private: - bool is_open; bool field_toggle; // Internal odd / even toggle (used when building the mapping) Fraction original; // The original frame rate Fraction target; // The target frame rate diff --git a/src/FrameMapper.cpp b/src/FrameMapper.cpp index 7c4d04bb..e80f355c 100644 --- a/src/FrameMapper.cpp +++ b/src/FrameMapper.cpp @@ -61,9 +61,9 @@ FrameMapper::FrameMapper(ReaderBase *reader, Fraction target, PulldownType targe // Destructor FrameMapper::~FrameMapper() { - if (is_open) - // Auto Close if not already - Close(); + + // Auto Close if not already + Close(); reader = NULL; }