Unverified Commit 22153d37 authored by YosuaMichael's avatar YosuaMichael Committed by GitHub
Browse files

Add more logging information for decoder (#6108)



Differential Revision: D36671123

fbshipit-source-id: eb99f293978ed150bca60073115971586f0fa02a
Co-authored-by: default avatarOleksandr Voietsa <ovoietsa@fb.com>
Co-authored-by: default avatarNicolas Hug <contact@nicolas-hug.com>
parent 03eda6e5
...@@ -488,7 +488,8 @@ int Decoder::getFrame(size_t workingTimeInMs) { ...@@ -488,7 +488,8 @@ int Decoder::getFrame(size_t workingTimeInMs) {
AVPacket* avPacket; AVPacket* avPacket;
avPacket = av_packet_alloc(); avPacket = av_packet_alloc();
if (avPacket == nullptr) { if (avPacket == nullptr) {
LOG(ERROR) << "decoder as not able to allocate the packet."; LOG(ERROR) << "uuid=" << params_.loggingUuid
<< " decoder as not able to allocate the packet.";
return AVERROR_BUFFER_TOO_SMALL; return AVERROR_BUFFER_TOO_SMALL;
} }
avPacket->data = nullptr; avPacket->data = nullptr;
...@@ -506,6 +507,7 @@ int Decoder::getFrame(size_t workingTimeInMs) { ...@@ -506,6 +507,7 @@ int Decoder::getFrame(size_t workingTimeInMs) {
bool decodedFrame = false; bool decodedFrame = false;
while (!interrupted_ && inRange_.any() && !decodedFrame) { while (!interrupted_ && inRange_.any() && !decodedFrame) {
if (watcher() == false) { if (watcher() == false) {
LOG(ERROR) << "uuid=" << params_.loggingUuid << " hit ETIMEDOUT";
result = ETIMEDOUT; result = ETIMEDOUT;
break; break;
} }
...@@ -522,7 +524,8 @@ int Decoder::getFrame(size_t workingTimeInMs) { ...@@ -522,7 +524,8 @@ int Decoder::getFrame(size_t workingTimeInMs) {
break; break;
} else if (result < 0) { } else if (result < 0) {
flushStreams(); flushStreams();
LOG(ERROR) << "Error detected: " << Util::generateErrorDesc(result); LOG(ERROR) << "uuid=" << params_.loggingUuid
<< " error detected: " << Util::generateErrorDesc(result);
break; break;
} }
...@@ -543,7 +546,8 @@ int Decoder::getFrame(size_t workingTimeInMs) { ...@@ -543,7 +546,8 @@ int Decoder::getFrame(size_t workingTimeInMs) {
// packet either got consumed completely or not at all // packet either got consumed completely or not at all
if ((result = processPacket( if ((result = processPacket(
stream, avPacket, &gotFrame, &hasMsg, params_.fastSeek)) < 0) { stream, avPacket, &gotFrame, &hasMsg, params_.fastSeek)) < 0) {
LOG(ERROR) << "processPacket failed with code: " << result; LOG(ERROR) << "uuid=" << params_.loggingUuid
<< " processPacket failed with code: " << result;
break; break;
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment