Skip to content

Commit cdabcb0

Browse files
author
Molly Xu
committed
fix fallthrough error
1 parent 3494331 commit cdabcb0

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

src/torchcodec/_core/Metadata.cpp

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -94,16 +94,8 @@ std::optional<int64_t> StreamMetadata::getNumFrames(SeekMode seekMode) const {
9494

9595
std::optional<double> StreamMetadata::getAverageFps(SeekMode seekMode) const {
9696
switch (seekMode) {
97-
case SeekMode::exact:
98-
if (endStreamPtsSecondsFromContent.value() !=
99-
beginStreamPtsSecondsFromContent.value()) {
100-
return static_cast<double>(
101-
getNumFrames(seekMode).value() /
102-
(endStreamPtsSecondsFromContent.value() -
103-
beginStreamPtsSecondsFromContent.value()));
104-
}
10597
case SeekMode::custom_frame_mappings:
106-
case SeekMode::approximate:
98+
case SeekMode::exact:
10799
if (getNumFrames(seekMode).has_value() &&
108100
beginStreamPtsSecondsFromContent.has_value() &&
109101
endStreamPtsSecondsFromContent.has_value() &&
@@ -115,6 +107,8 @@ std::optional<double> StreamMetadata::getAverageFps(SeekMode seekMode) const {
115107
beginStreamPtsSecondsFromContent.value()));
116108
}
117109
return averageFpsFromHeader;
110+
case SeekMode::approximate:
111+
return averageFpsFromHeader;
118112
default:
119113
TORCH_CHECK(false, "Unknown SeekMode");
120114
}

0 commit comments

Comments
 (0)