Improve logging of the cause when a movie can't be opened

BlueBrain · Sep 13, 2018 · a10f402 · a10f402
1 parent 575f651
commit a10f402
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/tide/core/data/FFMPEGVideoStream.cpp b/tide/core/data/FFMPEGVideoStream.cpp
@@ -358,6 +358,8 @@ void FFMPEGVideoStream::_generateSeekingParameters()
  const auto den = int64_t(timeBase.num) * AV_TIME_BASE;
  duration = av_rescale(_avFormatContext.duration, num, den);
  }
+ if (duration <= 0)
+ throw std::runtime_error("cannot determine movie duration");
 
  // Estimate number of frames if unavailable
  _numFrames = _videoStream->nb_frames;

diff --git a/tide/core/scene/MovieContent.cpp b/tide/core/scene/MovieContent.cpp
@@ -43,6 +43,8 @@
 
 #include "data/FFMPEGMovie.h"
 
+#include "utils/log.h"
+
 BOOST_CLASS_EXPORT_IMPLEMENT(MovieContent)
 
 IMPLEMENT_SERIALIZE_FOR_XML(MovieContent)
@@ -67,8 +69,9 @@ bool MovieContent::readMetadata()
  _frameDuration = movie.getFrameDuration();
  return true;
  }
- catch (const std::runtime_error&)
+ catch (const std::runtime_error& e)
  {
+ put_log(LOG_DEBUG, LOG_AV, e.what());
  return false;
  }
 }