Bug 1878510 - Follow-up: Explicitly pass values through variadic functions

While MSVC has no issue with this and understands values should be passed,
clang doesn't like it and aborts.
This commit is contained in:
Moonchild 2024-09-27 13:15:06 +02:00 • committed by roytam1
commit 8cb68d8ea8
2 changed files with 4 additions and 4 deletions

View file

@ -712,7 +712,7 @@ TrackBuffersManager::SegmentParserLoop()
MSE_DEBUG("Re-creating demuxer, new start (%" PRId64 MSE_DEBUG("Re-creating demuxer, new start (%" PRId64
") is smaller than last parsed end time (%" PRId64 ")", ") is smaller than last parsed end time (%" PRId64 ")",
start, start,
mLastParsedEndTime); mLastParsedEndTime.value());
mFrameEndTimeBeforeRecreateDemuxer = Some(end); mFrameEndTimeBeforeRecreateDemuxer = Some(end);
ResetDemuxingState(); ResetDemuxingState();
return; return;
@ -826,7 +826,7 @@ TrackBuffersManager::CreateDemuxerforMIMEType()
MSE_DEBUG( MSE_DEBUG(
"CreateDemuxerFromMimeType: " "CreateDemuxerFromMimeType: "
"mFrameEndTimeBeforeRecreateDemuxer=%" PRId64, "mFrameEndTimeBeforeRecreateDemuxer=%" PRId64,
mFrameEndTimeBeforeRecreateDemuxer); mFrameEndTimeBeforeRecreateDemuxer.value());
} }
mInputDemuxer = new WebMDemuxer(mCurrentInputBuffer, true /* IsMediaSource*/, mInputDemuxer = new WebMDemuxer(mCurrentInputBuffer, true /* IsMediaSource*/,
mFrameEndTimeBeforeRecreateDemuxer); mFrameEndTimeBeforeRecreateDemuxer);

View file

@ -182,7 +182,7 @@ WebMDemuxer::WebMDemuxer(MediaResource* aResource,
if (aIsMediaSource && aFrameEndTimeBeforeRecreateDemuxer) { if (aIsMediaSource && aFrameEndTimeBeforeRecreateDemuxer) {
mVideoFrameEndTimeBeforeReset = aFrameEndTimeBeforeRecreateDemuxer; mVideoFrameEndTimeBeforeReset = aFrameEndTimeBeforeRecreateDemuxer;
WEBM_DEBUG("Set mVideoFrameEndTimeBeforeReset=%" PRId64, WEBM_DEBUG("Set mVideoFrameEndTimeBeforeReset=%" PRId64,
mVideoFrameEndTimeBeforeReset); mVideoFrameEndTimeBeforeReset.value());
} }
} }
@ -648,7 +648,7 @@ WebMDemuxer::GetNextPacket(TrackInfo::TrackType aType, MediaRawDataQueue *aSampl
next_tstamp += tstamp - mLastVideoFrameTime.refOr(0); next_tstamp += tstamp - mLastVideoFrameTime.refOr(0);
} else if (mVideoFrameEndTimeBeforeReset) { } else if (mVideoFrameEndTimeBeforeReset) {
WEBM_DEBUG("Setting next timestamp to be %" PRId64 " us", WEBM_DEBUG("Setting next timestamp to be %" PRId64 " us",
mVideoFrameEndTimeBeforeReset); mVideoFrameEndTimeBeforeReset.value());
next_tstamp = mVideoFrameEndTimeBeforeReset.ref(); next_tstamp = mVideoFrameEndTimeBeforeReset.ref();
} else { } else {
PushVideoPacket(holder); PushVideoPacket(holder);