Skip to content

Commit 0108506

Browse files
committed
source* -> src*
1 parent 721c315 commit 0108506

File tree

3 files changed

+24
-25
lines changed

3 files changed

+24
-25
lines changed

src/torchcodec/_core/FFMPEGCommon.cpp

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -161,9 +161,9 @@ void setChannelLayout(
161161
}
162162

163163
SwrContext* createSwrContext(
164-
AVSampleFormat sourceSampleFormat,
164+
AVSampleFormat srcSampleFormat,
165165
AVSampleFormat desiredSampleFormat,
166-
int sourceSampleRate,
166+
int srcSampleRate,
167167
int desiredSampleRate,
168168
const UniqueAVFrame& srcAVFrame,
169169
int desiredNumChannels) {
@@ -178,8 +178,8 @@ SwrContext* createSwrContext(
178178
desiredSampleFormat,
179179
desiredSampleRate,
180180
&srcAVFrame->ch_layout,
181-
sourceSampleFormat,
182-
sourceSampleRate,
181+
srcSampleFormat,
182+
srcSampleRate,
183183
0,
184184
nullptr);
185185

@@ -196,8 +196,8 @@ SwrContext* createSwrContext(
196196
desiredSampleFormat,
197197
desiredSampleRate,
198198
srcAVFrame->channel_layout,
199-
sourceSampleFormat,
200-
sourceSampleRate,
199+
srcSampleFormat,
200+
srcSampleRate,
201201
0,
202202
nullptr);
203203
#endif
@@ -228,8 +228,8 @@ UniqueAVFrame convertAudioAVFrameSamples(
228228
convertedAVFrame->format = static_cast<int>(desiredSampleFormat);
229229

230230
convertedAVFrame->sample_rate = desiredSampleRate;
231-
int sourceSampleRate = srcAVFrame->sample_rate;
232-
if (sourceSampleRate != desiredSampleRate) {
231+
int srcSampleRate = srcAVFrame->sample_rate;
232+
if (srcSampleRate != desiredSampleRate) {
233233
// Note that this is an upper bound on the number of output samples.
234234
// `swr_convert()` will likely not fill convertedAVFrame with that many
235235
// samples if sample rate conversion is needed. It will buffer the last few
@@ -239,10 +239,9 @@ UniqueAVFrame convertAudioAVFrameSamples(
239239
// output samples, but empirically `av_rescale_rnd()` seems to provide a
240240
// tighter bound.
241241
convertedAVFrame->nb_samples = av_rescale_rnd(
242-
swr_get_delay(swrContext.get(), sourceSampleRate) +
243-
srcAVFrame->nb_samples,
242+
swr_get_delay(swrContext.get(), srcSampleRate) + srcAVFrame->nb_samples,
244243
desiredSampleRate,
245-
sourceSampleRate,
244+
srcSampleRate,
246245
AV_ROUND_UP);
247246
} else {
248247
convertedAVFrame->nb_samples = srcAVFrame->nb_samples;

src/torchcodec/_core/FFMPEGCommon.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,9 +161,9 @@ void setChannelLayout(
161161
int desiredNumChannels);
162162

163163
SwrContext* createSwrContext(
164-
AVSampleFormat sourceSampleFormat,
164+
AVSampleFormat srcSampleFormat,
165165
AVSampleFormat desiredSampleFormat,
166-
int sourceSampleRate,
166+
int srcSampleRate,
167167
int desiredSampleRate,
168168
const UniqueAVFrame& srcAVFrame,
169169
int desiredNumChannels);

src/torchcodec/_core/SingleStreamDecoder.cpp

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1169,40 +1169,40 @@ FrameOutput SingleStreamDecoder::convertAVFrameToFrameOutput(
11691169
void SingleStreamDecoder::convertAudioAVFrameToFrameOutputOnCPU(
11701170
UniqueAVFrame& srcAVFrame,
11711171
FrameOutput& frameOutput) {
1172-
AVSampleFormat sourceSampleFormat =
1172+
AVSampleFormat srcSampleFormat =
11731173
static_cast<AVSampleFormat>(srcAVFrame->format);
11741174
AVSampleFormat desiredSampleFormat = AV_SAMPLE_FMT_FLTP;
11751175

11761176
StreamInfo& streamInfo = streamInfos_[activeStreamIndex_];
1177-
int sourceSampleRate = srcAVFrame->sample_rate;
1177+
int srcSampleRate = srcAVFrame->sample_rate;
11781178
int desiredSampleRate =
1179-
streamInfo.audioStreamOptions.sampleRate.value_or(sourceSampleRate);
1179+
streamInfo.audioStreamOptions.sampleRate.value_or(srcSampleRate);
11801180

1181-
int sourceNumChannels = getNumChannels(streamInfo.codecContext);
1181+
int srcNumChannels = getNumChannels(streamInfo.codecContext);
11821182
TORCH_CHECK(
1183-
sourceNumChannels == getNumChannels(srcAVFrame),
1183+
srcNumChannels == getNumChannels(srcAVFrame),
11841184
"The frame has ",
11851185
getNumChannels(srcAVFrame),
11861186
" channels, expected ",
1187-
sourceNumChannels,
1187+
srcNumChannels,
11881188
". If you are hitting this, it may be because you are using "
11891189
"a buggy FFmpeg version. FFmpeg4 is known to fail here in some "
11901190
"valid scenarios. Try to upgrade FFmpeg?");
11911191
int desiredNumChannels =
1192-
streamInfo.audioStreamOptions.numChannels.value_or(sourceNumChannels);
1192+
streamInfo.audioStreamOptions.numChannels.value_or(srcNumChannels);
11931193

11941194
bool mustConvert =
1195-
(sourceSampleFormat != desiredSampleFormat ||
1196-
sourceSampleRate != desiredSampleRate ||
1197-
sourceNumChannels != desiredNumChannels);
1195+
(srcSampleFormat != desiredSampleFormat ||
1196+
srcSampleRate != desiredSampleRate ||
1197+
srcNumChannels != desiredNumChannels);
11981198

11991199
UniqueAVFrame convertedAVFrame;
12001200
if (mustConvert) {
12011201
if (!streamInfo.swrContext) {
12021202
streamInfo.swrContext.reset(createSwrContext(
1203-
sourceSampleFormat,
1203+
srcSampleFormat,
12041204
desiredSampleFormat,
1205-
sourceSampleRate,
1205+
srcSampleRate,
12061206
desiredSampleRate,
12071207
srcAVFrame,
12081208
desiredNumChannels));

0 commit comments

Comments
 (0)