Re: [FFmpeg-devel] [PATCH] ffmpeg: block output == input for files

2018-08-28 Thread Gyan Doshi
On 29-08-2018 02:43 AM, Michael Niedermayer wrote: On Tue, Aug 28, 2018 at 08:31:51AM +0200, Marton Balint wrote: Instead of this, maybe we should add support to write lock the files when opening them for reading. Then ffmpeg can request this. That would be an useful option, and not just for

Re: [FFmpeg-devel] make work (live) libsrt

2018-08-28 Thread Tudor Suciu
Hello Marton, And should simply set SRTO_PACKETSIZE to our pkt_size parameter if we > don't want the libsrt default 1356. > At least for the specific case of mpegts I believe it's much better to have fixed size packets(188x*). It helps error recovery if we don't have to re-synchronize ts so (7*188

Re: [FFmpeg-devel] [PATCH] aacenc: remove unsupported PCE mappings

2018-08-28 Thread Rostislav Pehlivanov
On 28 August 2018 at 22:29, Rostislav Pehlivanov wrote: > Turns out those ones make the decoder unable to accurately determine the > layout, hence making some API users' guess, and sometimes those are correct > (ffmpeg.c), but most often (especially with side channels) they're wrong. > The remove

[FFmpeg-devel] [PATCH] configure: _deps: validate, reduce sensitivity

2018-08-28 Thread avih
I analyzed the linkage failures at: http://ffmpeg.org/pipermail/ffmpeg-devel/2018-August/233674.html After finding their immediate cause - expecting a specific output order from unique() - I realized that there might be a related issue where devs are expected to maintain _deps (e.g. avfilter_deps)

[FFmpeg-devel] [PATCH] aacenc: remove unsupported PCE mappings

2018-08-28 Thread Rostislav Pehlivanov
Turns out those ones make the decoder unable to accurately determine the layout, hence making some API users' guess, and sometimes those are correct (ffmpeg.c), but most often (especially with side channels) they're wrong. The removed ones need to be investigated into as to why the decoder rejects

Re: [FFmpeg-devel] make work (live) libsrt

2018-08-28 Thread Marton Balint
On Sat, 25 Aug 2018, Tudor Suciu wrote: Hello Marton, The new version takes into account your remarks, modifications. I checked the srt API, and we are doing this all wrong. The SRTO_PAYLOADSIZE socket parameter defines the maximum allowed packet size. So we should query this parameter, a

Re: [FFmpeg-devel] [PATCH] ffmpeg: block output == input for files

2018-08-28 Thread Michael Niedermayer
On Tue, Aug 28, 2018 at 08:31:51AM +0200, Marton Balint wrote: > > > On Tue, 28 Aug 2018, Gyan Doshi wrote: > > > > >With some regularity, we have users trying to update input files using > >ffmpeg, usually for the purposes of tagging, but occasionally for changing > >the encoding or something e

Re: [FFmpeg-devel] [PATCH] avformat/matroskadec: Parse encryption init info from streams.

2018-08-28 Thread Jacob Trimble
On Mon, Aug 20, 2018 at 11:39 AM Jacob Trimble wrote: > > On Thu, Aug 9, 2018 at 9:14 AM Jacob Trimble wrote: > > > > On Wed, Aug 1, 2018 at 1:46 PM Jacob Trimble wrote: > > > > > > On Mon, Jul 23, 2018 at 2:01 PM Jacob Trimble wrote: > > > > > > > > On Thu, Jul 12, 2018 at 5:05 PM Jacob Trimbl

[FFmpeg-devel] [PATCH] avfilter/vf_framerate: switch to activate

2018-08-28 Thread Marton Balint
Signed-off-by: Marton Balint --- libavfilter/vf_framerate.c | 119 - 1 file changed, 63 insertions(+), 56 deletions(-) diff --git a/libavfilter/vf_framerate.c b/libavfilter/vf_framerate.c index 3e2615be5e..fb65381923 100644 --- a/libavfilter/vf_framera

[FFmpeg-devel] [PATCH] avformat/utils: Don't parse encrypted packets.

2018-08-28 Thread Jacob Trimble
If a packet is full-sample encrypted, then packet data can't be parsed without decrypting it. So this skips the packet parsing for those packets. If the packet has sub-sample encryption, it is assumed that the headers are in the clear and the parser will only need that info. Signed-off-by: Jacob

[FFmpeg-devel] [PATCH] fate: add 10-bit test for hue video filter

2018-08-28 Thread Tobias Rapp
Signed-off-by: Tobias Rapp --- tests/fate/filter-video.mak | 3 +++ tests/ref/fate/filter-hue4 | 1 + 2 files changed, 4 insertions(+) create mode 100644 tests/ref/fate/filter-hue4 diff --git a/tests/fate/filter-video.mak b/tests/fate/filter-video.mak index edd51e1..c87a7ba 100644 --- a/tests/

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/mscc: fix several bugs

2018-08-28 Thread Carl Eugen Hoyos
> Am 28.08.2018 um 12:05 schrieb Paul B Mahol : > >> On 8/28/18, Carl Eugen Hoyos wrote: >> 2018-08-28 12:00 GMT+02:00, Paul B Mahol : On 8/28/18, Carl Eugen Hoyos wrote: 2018-08-26 18:05 GMT+02:00, Paul B Mahol : > switch (avctx->bits_per_coded_sample) { > -cas

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/mscc: fix several bugs

2018-08-28 Thread Paul B Mahol
On 8/28/18, Carl Eugen Hoyos wrote: > 2018-08-28 12:00 GMT+02:00, Paul B Mahol : >> On 8/28/18, Carl Eugen Hoyos wrote: >>> 2018-08-26 18:05 GMT+02:00, Paul B Mahol : >>> switch (avctx->bits_per_coded_sample) { -case 8: avctx->pix_fmt = AV_PIX_FMT_GRAY8; break; +case

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/mscc: fix several bugs

2018-08-28 Thread Carl Eugen Hoyos
2018-08-28 12:00 GMT+02:00, Paul B Mahol : > On 8/28/18, Carl Eugen Hoyos wrote: >> 2018-08-26 18:05 GMT+02:00, Paul B Mahol : >> >>> switch (avctx->bits_per_coded_sample) { >>> -case 8: avctx->pix_fmt = AV_PIX_FMT_GRAY8; break; >>> +case 8: avctx->pix_fmt = AV_PIX_FMT_PAL8; brea

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/mscc: fix several bugs

2018-08-28 Thread Paul B Mahol
On 8/28/18, Carl Eugen Hoyos wrote: > 2018-08-26 18:05 GMT+02:00, Paul B Mahol : > >> switch (avctx->bits_per_coded_sample) { >> -case 8: avctx->pix_fmt = AV_PIX_FMT_GRAY8; break; >> +case 8: avctx->pix_fmt = AV_PIX_FMT_PAL8; break; > > Do you have a sample that looks correct wit

[FFmpeg-devel] [PATCH v2] libavformat/mxfenc: add missing dnxhr mxf container essence ULs

2018-08-28 Thread Jason Stevens
Add missing dnxhr mxf container essence ULs to the mxf encoder. set proper mxf frame size for dnxhr. This fixes dnxhr mxf files being quarantined by Avid Media Composer. --- libavformat/mxfenc.c | 55 +++- 1 file changed, 54 insertions(+), 1 deletion(-) dif