On Wed, Mar 2, 2022 at 8:22 AM Andreas Unterweger <dustsi...@gmail.com>
wrote:
> > On 23.02.2022 15:58 Andreas Unterweger <dustsi...@gmail.com> wrote:
> >
> >
> > The last encoded frame is now fetched on EOF. It was previously left in
> the encoder and caused a "1 frame left in queue" warning.
> >
> > Signed-off-by: Andreas Unterweger <dustsi...@gmail.com>
> > ---
> > doc/examples/transcode_aac.c | 22 +++++++++++-----------
> > 1 file changed, 11 insertions(+), 11 deletions(-)
> >
> > diff --git a/doc/examples/transcode_aac.c b/doc/examples/transcode_aac.c
> > index 1cf6317e27..a09690fe45 100644
> > --- a/doc/examples/transcode_aac.c
> > +++ b/doc/examples/transcode_aac.c
> > @@ -381,6 +381,8 @@ static int decode_audio_frame(AVFrame *frame,
> > if (error < 0)
> > return error;
> >
> > + *data_present = 0;
> > + *finished = 0;
> > /* Read one audio frame from the input file into a temporary
> packet. */
> > if ((error = av_read_frame(input_format_context, input_packet)) <
> 0) {
> > /* If we are at the end of the file, flush the decoder below.
> */
> > @@ -559,7 +561,7 @@ static int read_decode_convert_and_store(AVAudioFifo
> *fifo,
> > AVFrame *input_frame = NULL;
> > /* Temporary storage for the converted input samples. */
> > uint8_t **converted_input_samples = NULL;
> > - int data_present = 0;
> > + int data_present;
> > int ret = AVERROR_EXIT;
> >
> > /* Initialize temporary storage for one input frame. */
> > @@ -679,18 +681,17 @@ static int encode_audio_frame(AVFrame *frame,
> > frame->pts = pts;
> > pts += frame->nb_samples;
> > }
> > -
> > +
> > + *data_present = 0;
> > /* Send the audio frame stored in the temporary packet to the
> encoder.
> > * The output audio stream encoder is used to do this. */
> > error = avcodec_send_frame(output_codec_context, frame);
> > - /* The encoder signals that it has nothing more to encode. */
> > - if (error == AVERROR_EOF) {
> > - error = 0;
> > - goto cleanup;
> > - } else if (error < 0) {
> > - fprintf(stderr, "Could not send packet for encoding (error
> '%s')\n",
> > - av_err2str(error));
> > - goto cleanup;
> > + /* Check for errors, but proceed with fetching encoded samples if
> the
> > + * encoder signals that it has nothing more to encode. */
> > + if (error < 0 && error != AVERROR_EOF) {
> > + fprintf(stderr, "Could not send packet for encoding (error
> '%s')\n",
> > + av_err2str(error));
> > + goto cleanup;
> > }
> >
> > /* Receive one encoded frame from the encoder. */
> > @@ -861,7 +862,6 @@ int main(int argc, char **argv)
> > int data_written;
> > /* Flush the encoder as it may have delayed frames. */
> > do {
> > - data_written = 0;
>
why this is removed ?
> > if (encode_audio_frame(NULL, output_format_context,
> > output_codec_context,
> &data_written))
> > goto cleanup;
> > --
> > 2.30.2
> >
>
> Ping
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
