fixed bug where an unindexed video would cache a blank first frame

app/decoder/ffmpeg/ffmpegdecoder.cpp

@@ -478,6 +478,10 @@ void FFmpegDecoder::Index()
 
   // Save index to file
   SaveFrameIndex();
+
+  // Reset state
+  avcodec_flush_buffers(codec_ctx_);
+  av_seek_frame(fmt_ctx_, avstream_->index, 0, AVSEEK_FLAG_BACKWARD);
 }
 
 QString FFmpegDecoder::GetIndexFilename()

app/node/input.cpp

@@ -77,7 +77,7 @@ QVariant NodeInput::get_value(const rational& time)
     // Retrieve the value
     if (!edges_.isEmpty()) {
       // A connection - use the output of the connected Node
-      value_ = edges_.first()->output()->get_value(time);
+      value_ = get_connected_output()->get_value(time);
     } else {
       // No connections - use the internal value
       // FIXME: Re-implement keyframing

app/node/input/media/media.cpp

@@ -132,16 +132,15 @@ QVariant MediaInput::Value(NodeOutput *output, const rational &time)
     }
 
     // Check if we need to get a frame or not
-    if (frame_ == nullptr || frame_native_ts_ != decoder_->GetTimestampFromTime(time)) {
+    if (frame_ == nullptr || frame_->native_timestamp() != decoder_->GetTimestampFromTime(time)) {
       // Get frame from Decoder
       frame_ = decoder_->Retrieve(time);
 
       if (frame_ == nullptr) {
+        qDebug() << "Received a null frame while time was" << time.toDouble();
         return 0;
       }
 
-      frame_native_ts_ = frame_->native_timestamp();
-
       if (color_service_ == nullptr) {
         // FIXME: Hardcoded values for testing
         color_service_ = std::make_shared<ColorService>("srgb", OCIO::ROLE_SCENE_LINEAR);

app/node/input/media/media.h

@@ -77,7 +77,6 @@ private:
   GLuint ocio_texture_;
 
   FramePtr frame_;
-  int64_t frame_native_ts_;
 
 };
 

app/node/param.cpp

@@ -287,6 +287,11 @@ const rational &NodeParam::LastRequestedTime()
   return time_;
 }
 
+bool NodeParam::ValueCachingEnabled()
+{
+  return value_caching_;
+}
+
 void NodeParam::SetValueCachingEnabled(bool enabled)
 {
   value_caching_ = enabled;

app/node/param.h

@@ -255,6 +255,7 @@ public:
    */
   const rational& LastRequestedTime();
 
+  bool ValueCachingEnabled();
   void SetValueCachingEnabled(bool enabled);
 
   virtual DataType data_type() = 0;

app/node/processor/renderer/renderer.cpp

@@ -359,7 +359,7 @@ void RendererProcessor::CacheNext()
 
   rational cache_frame = cache_queue_.takeFirst();
 
-  //qDebug() << "[RendererProcessor] Caching" << cache_frame.toDouble();
+  qDebug() << "[RendererProcessor] Caching" << cache_frame.toDouble();
 
   threads_.first()->Queue(NodeDependency(texture_input_->get_connected_output(), cache_frame), true);