Message ID | CANy4d5BdOq=FWo7RGRa6Yg2kCAQ4vUfvzpx57fYDxYo7aqVQ8Q@mail.gmail.com |
---|---|
State | New |
Headers | show |
On Mon, Nov 13, 2017 at 14:13:15 +0900, Natsuki Kai wrote: > Oh sorry, I don't understand how to do well yet... [...] > - q->async_fifo = av_fifo_alloc((1 + q->async_depth) * > + q->async_fifo = av_fifo_alloc((q->async_depth) * > (sizeof(AVPacket) + > sizeof(mfxSyncPoint*) + sizeof(mfxBitstream*))); Your mail tool or send chain is inserting linebreaks into the patch. It will therefore not work anymore on the receiving side (as it is not only visually inspected on the list, but also tested). You can either send the patch with "git send-email", or attach it as a separate file. Moritz
Hi Moritz, Thank you for explaining in details. It really helped me. It seems gmail or other mail tool shouldn't be used, but only "git send-email" is accepted. I posted patch mail from git command. http://ffmpeg.org/pipermail/ffmpeg-devel/2017-November/219805.html So please discuss in that thread. P.S. I've been confused because "git send-email" doesn't work on windows PC... Finally, I gave up working on windows and posted on Mac. On Mon, Nov 13, 2017 at 9:54 PM, Moritz Barsnick <barsnick@gmx.net> wrote: > On Mon, Nov 13, 2017 at 14:13:15 +0900, Natsuki Kai wrote: >> Oh sorry, I don't understand how to do well yet... > [...] >> - q->async_fifo = av_fifo_alloc((1 + q->async_depth) * >> + q->async_fifo = av_fifo_alloc((q->async_depth) * >> (sizeof(AVPacket) + >> sizeof(mfxSyncPoint*) + sizeof(mfxBitstream*))); > > Your mail tool or send chain is inserting linebreaks into the patch. It > will therefore not work anymore on the receiving side (as it is not > only visually inspected on the list, but also tested). You can either > send the patch with "git send-email", or attach it as a separate file. > > Moritz > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
2017-11-14 17:02 GMT+01:00 Natsuki Kai <n.kai.cj.github@gmail.com>: > Hi Moritz, > > Thank you for explaining in details. It really helped me. > It seems gmail or other mail tool shouldn't be used, but only "git > send-email" is accepted. Just send your patch - made with git format-patch - as attachment. This should work with all mail clients. Carl Eugen
I sent it as another thread, but yes it's ok. Please check the attachment patch file. Thank you. On Wed, Nov 15, 2017 at 7:55 AM, Carl Eugen Hoyos <ceffmpeg@gmail.com> wrote: > 2017-11-14 17:02 GMT+01:00 Natsuki Kai <n.kai.cj.github@gmail.com>: >> Hi Moritz, >> >> Thank you for explaining in details. It really helped me. >> It seems gmail or other mail tool shouldn't be used, but only "git >> send-email" is accepted. > > Just send your patch - made with git format-patch - as > attachment. This should work with all mail clients. > > Carl Eugen > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index 2bc19f5241..7d73c64dca 100644 --- a/libavcodec/qsvenc.c +++ b/libavcodec/qsvenc.c @@ -803,7 +803,7 @@ int ff_qsv_enc_init(AVCodecContext *avctx, QSVEncContext *q) q->param.AsyncDepth = q->async_depth; - q->async_fifo = av_fifo_alloc((1 + q->async_depth) * + q->async_fifo = av_fifo_alloc((q->async_depth) * (sizeof(AVPacket) + sizeof(mfxSyncPoint*) + sizeof(mfxBitstream*)));