[FFmpeg-devel] [PATCH 45/48] doc/examples/vaapi_encode: use av_packet_alloc() to allocate packets
Michael Niedermayer
michael at niedermayer.cc
Thu Mar 11 15:34:56 EET 2021
On Wed, Mar 10, 2021 at 06:25:58PM -0300, James Almer wrote:
> On 3/10/2021 5:22 PM, Michael Niedermayer wrote:
> > On Sat, Mar 06, 2021 at 06:45:57PM -0300, James Almer wrote:
> > > On 3/6/2021 6:35 PM, Michael Niedermayer wrote:
> > > > On Fri, Mar 05, 2021 at 01:33:36PM -0300, James Almer wrote:
> > > > > Signed-off-by: James Almer <jamrial at gmail.com>
> > > > > ---
> > > > > doc/examples/vaapi_encode.c | 16 ++++++++--------
> > > > > 1 file changed, 8 insertions(+), 8 deletions(-)
> > > > >
> > > > > diff --git a/doc/examples/vaapi_encode.c b/doc/examples/vaapi_encode.c
> > > > > index 707939db37..66cb949cdc 100644
> > > > > --- a/doc/examples/vaapi_encode.c
> > > > > +++ b/doc/examples/vaapi_encode.c
> > > > > @@ -74,27 +74,27 @@ static int set_hwframe_ctx(AVCodecContext *ctx, AVBufferRef *hw_device_ctx)
> > > > > static int encode_write(AVCodecContext *avctx, AVFrame *frame, FILE *fout)
> > > > > {
> > > > > int ret = 0;
> > > > > - AVPacket enc_pkt;
> > > > > + AVPacket *enc_pkt;
> > > > > - av_init_packet(&enc_pkt);
> > > > > - enc_pkt.data = NULL;
> > > > > - enc_pkt.size = 0;
> > > > > + if (!(enc_pkt = av_packet_alloc()))
> > > > > + return AVERROR(ENOMEM);
> > > > > if ((ret = avcodec_send_frame(avctx, frame)) < 0) {
> > > > > fprintf(stderr, "Error code: %s\n", av_err2str(ret));
> > > > > goto end;
> > > > > }
> > > > > while (1) {
> > > > > - ret = avcodec_receive_packet(avctx, &enc_pkt);
> > > > > + ret = avcodec_receive_packet(avctx, enc_pkt);
> > > > > if (ret)
> > > > > break;
> > > > > - enc_pkt.stream_index = 0;
> > > > > - ret = fwrite(enc_pkt.data, enc_pkt.size, 1, fout);
> > > > > - av_packet_unref(&enc_pkt);
> > > > > + enc_pkt->stream_index = 0;
> > > > > + ret = fwrite(enc_pkt.data, enc_pkt->size, 1, fout);
> > > > > + av_packet_unref(enc_pkt);
> > > > > }
> > > > > end:
> > > > > + av_packet_free(&enc_pkt);
> > > > > ret = ((ret == AVERROR(EAGAIN)) ? 0 : -1);
> > > > > return ret;
> > > > > }
> > > >
> > > > breaks:
> > > > doc/examples/vaapi_encode.c: In function ‘encode_write’:
> > > > doc/examples/vaapi_encode.c:92:29: error: ‘enc_pkt’ is a pointer; did you mean to use ‘->’?
> > > > ret = fwrite(enc_pkt.data, enc_pkt->size, 1, fout);
> > > > ^
> > > > ->
> > > > ffbuild/common.mak:67: recipe for target 'doc/examples/vaapi_encode.o' failed
> > > > make: *** [doc/examples/vaapi_encode.o] Error 1
> > > > make: *** Waiting for unfinished jobs....
> > >
> > > Fixed locally. Thanks.
> >
> > do you have a public git tree for testing such minor changes ?
> > so i can retest the update set easily
> >
> > thanks
>
> Yes, pushed it to
>
> https://github.com/jamrial/FFmpeg/commits/avpacket
seems to work now, thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210311/782d1fb4/attachment.sig>
More information about the ffmpeg-devel
mailing list