From 9585786c4ba9d41f5f23e0b39988bf0da899071c Mon Sep 17 00:00:00 2001 From: itsmattkc Date: Wed, 26 Feb 2020 14:07:47 +1100 Subject: [PATCH] exporter: removed unused and problematic function This array this code made was never actually used and in fact the code was potentially crash-prone. --- app/render/backend/exporter.cpp | 26 -------------------------- app/render/backend/exporter.h | 2 -- 2 files changed, 28 deletions(-) diff --git a/app/render/backend/exporter.cpp b/app/render/backend/exporter.cpp index cabaca780..cb064a1cd 100644 --- a/app/render/backend/exporter.cpp +++ b/app/render/backend/exporter.cpp @@ -236,32 +236,6 @@ void Exporter::VideoHashesComplete() TimeRangeList ranges; ranges.append(TimeRange(0, viewer_node_->Length())); - QMap::const_iterator i; - QMap::iterator j; - - // Copy time hash map - QMap time_hash_map = video_backend_->frame_cache()->time_hash_map(); - - // Check for any times that share duplicate hashes - for (i=time_hash_map.begin();i!=time_hash_map.end();i++) { - j = time_hash_map.begin(); - - while (j != time_hash_map.end()) { - if (i != j && i.value() == j.value()) { - // Remove the time range from this and - ranges.RemoveTimeRange(TimeRange(j.key(), j.key() + video_backend_->params().time_base())); - - QList times_with_this_hash = matched_frames_.value(i.value()); - times_with_this_hash.append(j.key()); - matched_frames_.insert(i.value(), times_with_this_hash); - - j = time_hash_map.erase(j); - } else { - j++; - } - } - } - // Set video backend to render mode but NOT hash or download video_backend_->SetOperatingMode(VideoRenderWorker::kRenderOnly); video_backend_->SetOnlySignalLastFrameRequested(false); diff --git a/app/render/backend/exporter.h b/app/render/backend/exporter.h index f5b111372..490a775f0 100644 --- a/app/render/backend/exporter.h +++ b/app/render/backend/exporter.h @@ -73,8 +73,6 @@ private: QHash cached_frames_; - QHash< QByteArray, QList > matched_frames_; - private slots: void FrameRendered(const rational &time, FramePtr value);