Re: [FFmpeg-devel] [PATCH] Fixing HW accelerated decoders hanging or throwing error in h263dec

2019-08-07 Thread Stefan Schoenefeld
Hi Phil, Unfortunately I cannot share the sample we use internally, but it should be reproducible with any h.263/mpeg-4 encoded file & hw accelerated decoding. Thanks Stefan -Original Message- From: ffmpeg-devel On Behalf Of Philip Langdale Sent: Saturday, August 3, 2019 0

[FFmpeg-devel] [PATCH] Fixing HW accelerated decoders hanging or throwing error in h263dec

2019-08-02 Thread Stefan Schoenefeld
if the codec implements update_thread_context(), which h263dec does not. The patch does not address this and I'm not sure any action needs to be taken here at all. Thanks, Stefan [1] This is depending on whether or not the hw decoder sets the HWACCEL_CAPS_ASYNC_SAFE flag >F

[FFmpeg-devel] [PATCH] dashdec: Fix reading values from SegmentTimeline inside, Period

2019-06-28 Thread Stefan _
Hi, attached patch fixes a small oversight in dashdec. YouTube uses DASH manifests structured like this for live recordings, seeking is currently broken in those cases. From b0eceb6bbe0c931d8c67a22980816bf3f8dd0bbe Mon Sep 17 00:00:00 2001 From: sfan5 Date: Sat, 29 Jun 2019 00:51:28 +0200

[FFmpeg-devel] Patch: to add missing padding to nearest WORD boundary AVI

2018-07-17 Thread stefan . becker555
0001-add-missing-padding-to-nearest-WORD-boundary-AVI.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avcodec/mediacodec_wrapper: fix false positives in swdec blacklist

2018-03-06 Thread Stefan _
Hi, attached patch fixes an issue with the previous mediacodec patch. From b6a8721679483900b4f824504fdb1f7944ec268f Mon Sep 17 00:00:00 2001 From: sfan5 Date: Tue, 6 Mar 2018 23:14:09 +0100 Subject: [PATCH] avcodec/mediacodec_wrapper: fix false positives in swdec blacklist

[FFmpeg-devel] [PATCH] avcodec/mediacodec_wrapper: blacklist more software decoders

2018-03-06 Thread Stefan _
Hi, for hwdec on Android the wrapper attempts to blacklist known software decoders also provided through the mediacodec API. The list isn't complete however and ffmpeg might e.g. prefer slower "external" HEVC software decoding over its own software decoder. This patch completes the list of

Re: [FFmpeg-devel] [PATCH v2] libavformat/tls: pass numeric hostnames to tls_connect_cbs()

2018-03-01 Thread Stefan _
bump, this is a minor patch > Hadn't seen that one, it does indeed describe this exact problem. > I've included it in the commit message, new patch attached. > > On 20.02.2018 at 22:30 Carl Eugen Hoyos wrote: >> 2018-02-20 19:40 GMT+01:00 Stefan _ <sf...@live.de>: >

Re: [FFmpeg-devel] [PATCH v2] libavformat/tls: pass numeric hostnames to tls_connect_cbs()

2018-02-21 Thread Stefan _
Hadn't seen that one, it does indeed describe this exact problem. I've included it in the commit message, new patch attached. On 20.02.2018 at 22:30 Carl Eugen Hoyos wrote: > 2018-02-20 19:40 GMT+01:00 Stefan _ <sf...@live.de>: > >> attached patch fixes a small issue with

[FFmpeg-devel] [PATCH] libavformat/tls: pass numeric hostnames to tls_connect_cbs()

2018-02-20 Thread Stefan _
Hi, attached patch fixes a small issue with the libtls backend. reported here: https://github.com/shinchiro/mpv-winbuild-cmake/issues/61 From 4bba90874839cf98ac27756c4a3137be4c84c669 Mon Sep 17 00:00:00 2001 From: sfan5 Date: Tue, 20 Feb 2018 19:18:13 +0100 Subject: [PATCH]

[FFmpeg-devel] [PATCH v3] lavf/mpegts: add supplementary audio descriptor

2018-02-20 Thread Stefan Pöschel
The supplementary audio descriptor is defined in ETSI EN 300 468 and provides more details regarding accessibility audio tracks, especially the normative annex J contains a detailed description of its use. Its language code (if present) overrides the language code of an also present ISO 639

Re: [FFmpeg-devel] [PATCH v2] lavf/mpegts: add supplementary audio descriptor

2018-02-20 Thread Stefan Pöschel
in a couple days along with my AV_DISPOSITION_DEPENDENT >> if no one objects. > > LGTM too otherwise. Great, thanks! Regards, Stefan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH v2] lavf/mpegts: add supplementary audio descriptor

2018-02-16 Thread Stefan Pöschel
The supplementary audio descriptor is defined in ETSI EN 300 468 and provides more details regarding accessibility audio tracks, especially the normative annex J contains a detailed description of its use. Its language code (if present) overrides the language code of an also present ISO 639

Re: [FFmpeg-devel] [PATCH] lavf/mpegts: add supplementary audio descriptor

2018-02-16 Thread Stefan Pöschel
Am 16.02.2018 um 20:01 schrieb Aman Gupta: >> +if(flags & 0x01) { > > Please add a space after "if" here. > >> +if(language[0]) > > Same, missing space. OK, I will add the spaces. And also add the respective field names as comments, as you did for the mix_type.

Re: [FFmpeg-devel] [PATCH] lavf/mpegts: add supplementary audio descriptor

2018-02-16 Thread Stefan Pöschel
p with > a good name for a new flag. Great! Regards, Stefan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavf/mpegts: add supplementary audio descriptor

2018-02-15 Thread Stefan Pöschel
e else, as I don't have deep knowledge in FFmpeg's internals and any implications of adding a new field (which seems to be necessary here). Regards, Stefan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] lavf/mpegts: add supplementary audio descriptor

2018-02-15 Thread Stefan Pöschel
The supplementary audio descriptor is defined in ETSI EN 300 468 and provides more details regarding accessibility audio tracks, especially the normative annex J contains a detailed description of its use. Its language code (if present) overrides the language code of an also present ISO 639

[FFmpeg-devel] [PATCH] dashdec: Support SegmentTimeline inside Period

2018-02-06 Thread Stefan _
Hi, attached patch adds support for yet another small detail of the standard that YouTube happens to use for DVR functionality of livestreams. Ideally dashdec would be fully standards compliant implementation, but that doesn't look like it's planned or happening anytime soon. From

Re: [FFmpeg-devel] [PATCH] dashdec: Make use of frame rate specified in Representation

2018-01-20 Thread Stefan _
bump On 15.01.2018 at 16:58 Stefan _ wrote: > Hi, > > attached patch fixes an annoyance when playing DASH videos from e.g. > YouTube: > > "mov,mp4,m4a,3gp,3g2,mj2: Stream #0: not enough frames to estimate rate; > con

[FFmpeg-devel] [PATCH] dashdec: Make use of frame rate specified in Representation

2018-01-15 Thread Stefan _
Hi, attached patch fixes an annoyance when playing DASH videos from e.g. YouTube: "mov,mp4,m4a,3gp,3g2,mj2: Stream #0: not enough frames to estimate rate; consider increasing probesize" From e0210059fb420ef2e6c6b0d89c7e733b99f78ee5 Mon Sep 17 00:00:00 2001 From: sfan5 Date:

[FFmpeg-devel] [PATCH 4/6] dashdec: Correct seeking behaviour

2018-01-07 Thread Stefan _
>From bdf3557a400620fce66ca0237f1ff172c227609b Mon Sep 17 00:00:00 2001 From: sfan5 Date: Fri, 5 Jan 2018 00:51:32 +0100 Subject: [PATCH 4/6] dashdec: Correct seeking behaviour dash_read_seek() is called only once to issue a seek of *all* streams to the specified timestamp. But

[FFmpeg-devel] [PATCH 5/6] dashdec: Avoid trying to read any segments beyond the last

2018-01-07 Thread Stefan _
>From 277c710159849816bff4e4f5ccd1139348518620 Mon Sep 17 00:00:00 2001 From: sfan5 Date: Fri, 5 Jan 2018 14:19:25 +0100 Subject: [PATCH 5/6] dashdec: Avoid trying to read any segments beyond the last --- libavformat/dashdec.c | 8 +--- 1 file changed, 5 insertions(+), 3

[FFmpeg-devel] [PATCH 6/6] dashdec: Support SegmentTemplate inside Period

2018-01-07 Thread Stefan _
>From 89b42dc60156f1e030a30e13636651db41e9f34b Mon Sep 17 00:00:00 2001 From: sfan5 Date: Fri, 5 Jan 2018 15:32:23 +0100 Subject: [PATCH 6/6] dashdec: Support SegmentTemplate inside Period --- libavformat/dashdec.c | 27 +-- 1 file changed, 17

[FFmpeg-devel] [PATCH 1/6] dashdec: Expose bandwidth and representation ID as metadata

2018-01-07 Thread Stefan _
Hi, I did some work on the DASH demuxer. The primary goal was making it viable to play YouTube/Vimeo/... videos using the native demuxer, since mpv currently uses a workaround in form of Edit Decision Lists (EDL). Implemented features: * Exposing id / bitrate as stream metadata (similar to

[FFmpeg-devel] [PATCH 2/6] dashdec: Support for multiple video/audio streams

2018-01-07 Thread Stefan _
>From 857da994ba1f3466cd6a3b28d025a95301577ad2 Mon Sep 17 00:00:00 2001 From: sfan5 Date: Fri, 5 Jan 2018 00:03:06 +0100 Subject: [PATCH 2/6] dashdec: Support for multiple video/audio streams --- doc/demuxers.texi | 10 ++ libavformat/dashdec.c | 344

[FFmpeg-devel] [PATCH 3/6] dashdec: Search for segment timeline inside AdaptionSets too

2018-01-07 Thread Stefan _
>From c06d6cbcdc9092428d3764a969604d1f22725e56 Mon Sep 17 00:00:00 2001 From: sfan5 Date: Fri, 5 Jan 2018 00:19:53 +0100 Subject: [PATCH 3/6] dashdec: Search for segment timeline inside AdaptionSets too --- libavformat/dashdec.c | 40 +++- 1

Re: [FFmpeg-devel] [PATCH v3 2/2] libavcodec/hevcdec: implement skip_frame

2017-12-19 Thread Stefan _
On 07.12.2017 at 20:56 Stefan _ wrote: > On 07.12.2017 at 17:41 Michael Niedermayer wrote: >> The move and the functional change should be in seperate patches >> that keeps changes easy to read and understand >> >> [...] bump (Previous patch in same series was merged.

Re: [FFmpeg-devel] [PATCH] configure: remove libtls fallback check

2017-12-19 Thread Stefan _
On 19.12.2017 at 18:28 James Almer wrote: > On 12/19/2017 2:13 PM, Stefan _ wrote: >> Last patch had a minor issue, fixed version attached. > Applied. Thanks. > > The fallback check for that matter should be removed. It's pointless > since every supported version of the lib

Re: [FFmpeg-devel] [PATCHv2] configure: fix pkg-config check for libtls

2017-12-19 Thread Stefan _
Last patch had a minor issue, fixed version attached. From 6f6bcf77ce2aaf6a84858d34112f61128e779fda Mon Sep 17 00:00:00 2001 From: sfan5 Date: Tue, 19 Dec 2017 17:33:26 +0100 Subject: [PATCH] configure: fix pkg-config check for libtls This was not accounted for during merge and is

[FFmpeg-devel] [PATCH] configure: fix pkg-config check for libtls

2017-12-19 Thread Stefan _
Hi, Attached patch fixes an issue with the LibreSSL pkg-config check in configure. Between the creation of my libtls patchset and the date it was pushed, a refactor had changed all invocations of use_pkg_config to require_pkg_config, resulting in pkg-config never working. From

[FFmpeg-devel] [PATCH] configure: Fix case of static libmp3lame

2017-12-16 Thread Stefan Pöschel
Fixes #6918. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index d5bbb5b..20ca792 100755 --- a/configure +++ b/configure @@ -5853,7 +5853,7 @@ enabled libkvazaar&& require_pkg_config libkvazaar "kvazaar >= 0.8.1" kv enabled

Re: [FFmpeg-devel] [PATCH] libavformat: LibreSSL (libtls) support

2017-12-09 Thread Stefan _
n this would be a welcome fix and in my opinion > this is the best way to support LibreSSL going forward. If Stefan is happy > to continue to support this in the future if needed then im on-board with > merging this. Thanks for taking a look. Previously, I was under the impression that LibreSSL su

[FFmpeg-devel] [PATCH v3 2/2] libavcodec/hevcdec: implement skip_frame

2017-12-07 Thread Stefan _
On 07.12.2017 at 17:41 Michael Niedermayer wrote: > The move and the functional change should be in seperate patches > that keeps changes easy to read and understand > > [...] >From abeb2b106a64d96b216912d4272a734b123b62e2 Mon Sep 17 00:00:00 2001 From: sfan5 Date: Thu, 7 Dec

[FFmpeg-devel] [PATCH v3 1/2] libavcodec/hevc_filter: move AVDISCARD_NONREF switch-case into function

2017-12-07 Thread Stefan _
On 07.12.2017 at 17:41 Michael Niedermayer wrote: > The move and the functional change should be in seperate patches > that keeps changes easy to read and understand > > [...] >From b5d6c40c6516b90abeeffb7cf8ecd1ca1c3f7cb2 Mon Sep 17 00:00:00 2001 From: sfan5 Date: Thu, 7 Dec

Re: [FFmpeg-devel] [PATCH v2] libavcodec/hevc_filter: implement skip_frame

2017-12-06 Thread Stefan _
On 06.12.2017 at 18:32 Michael Niedermayer wrote: > This is duplicated, it should be moved into a seperate function > > [...] > Done. From 7a9651040c1c4815d82712cb98dbd7bcf8c085bb Mon Sep 17 00:00:00 2001 From: sfan5 Date: Tue, 5 Dec 2017 23:26:14 +0100 Subject: [PATCH]

[FFmpeg-devel] [PATCH] libavcodec/hevc_filter: implement skip_frame

2017-12-05 Thread Stefan _
On 04.12.2017 at 21:06 Carl Eugen Hoyos wrote: > 2017-12-01 0:22 GMT+01:00 Stefan _ <sf...@live.de>: > >> Attached patch adds full support for skip_loop_filter >> (all levels) to the hevc decoder. > Will you also work on -skip_frame for hevc? > > Carl Eugen I ga

[FFmpeg-devel] [PATCH] fate/hevc: add skip_loop_filter test

2017-12-05 Thread Stefan _
On 04.12.2017 at 18:49 Michael Niedermayer wrote: > will apply > > can you add a fate test for this ? > > thanks > > [...] Sure. I've picked a random sample where skip_loop_filter={nokey,all} had different output. Hope that's fine. From 2f4e5bf1e37d0fbfce0631ebb8e7011b5f8128f1 Mon Sep 17

Re: [FFmpeg-devel] [PATCH v2] libavcodec/hevc_filter: support for all skip_loop_filter levels

2017-12-03 Thread Stefan _
On 01.12.2017 at 17:45 Michael Niedermayer wrote: > AVDISCARD_NONREF is about frames which are not referenced by any other > doesnt ff_hevc_frame_nb_refs() produce the number of references curently > aka the other end of the reference arrows ? Yes, that sounds right. I've revised the patch to

[FFmpeg-devel] [PATCH] libavcodec/hevc_filter: support for all skip_loop_filter levels

2017-11-30 Thread Stefan _
Hi, Attached patch adds full support for skip_loop_filter (all levels) to the hevc decoder. I'm not too sure about the implementation of "nonref", since this email (http://ffmpeg.org/pipermail/ffmpeg-devel/2015-July/176116.html) mentions looking at nal_unit_type and temporal_id instead. Some

[FFmpeg-devel] [PATCH] libavformat: LibreSSL (libtls) support

2017-11-04 Thread Stefan _
Attached patch adds support for LibreSSL. Instead of trying to implement support into the existing tls_openssl.c using lots of #ifdefs (which was rejected previously(?)) this adds a new TLS backend making use of the new libtls library. Things to note: - Haven't looked at LibreSSL's license

Re: [FFmpeg-devel] [PATCH] configure/rtmpdh: Fix OpenSSL 1.1.0 support

2017-02-23 Thread Stefan _
Am 23.02.2017 um 02:07 schrieb Michael Niedermayer: > On Wed, Feb 22, 2017 at 11:15:45PM +0000, Stefan _ wrote: > >> From: sfan5 <sf...@live.de> > Is it intended that theres no full name in the git author field ? > >

Re: [FFmpeg-devel] [PATCH] configure/rtmpdh: Fix OpenSSL 1.1.0 support

2017-02-22 Thread Stefan _
The rtmpdh patch was kinda (completly) broken, fixed patch is attached. Concerning LibreSSL: They use OPENSSL_VERSION_NUMBER = 0x2050200fL which breaks the other openssl code in tls_openssl.c anyway, so LibreSSL support should probably be worried about in a different patch. From

[FFmpeg-devel] [PATCH] configure/rtmpdh: Fix OpenSSL 1.1.0 support

2017-02-22 Thread Stefan _
1) configure only looks for OPENSSL_init_ssl using pkg-config, this breaks in case pkg-config is not available (cross-compiling) 2) The rtmpdh code uses the DH struct from OpenSSL which was made private in the 1.1 series From 5cc140f85d8d7551ac0a2c51d5023bd8ba75ba4b Mon Sep 17 00:00:00 2001

[FFmpeg-devel] [PATCH] doc/muxers/mpegts: update doc after adding flag to embed an AC-3/E-AC-3 ES the DVB way

2015-12-25 Thread Stefan Pöschel
ag to embed an > AC-3/E-AC-3 ES the DVB way > > On Sun, Dec 13, 2015 at 11:54:32AM +0100, Stefan Pöschel wrote: > > So far an AC-3 elementary stream is refered to in the PMT according to > > System A (ATSC). An E-AC-3 ES in contrast is embedded the System B (DVB) > > way

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: add flag to embed an AC-3/E-AC-3 ES the DVB way

2015-12-23 Thread Stefan Pöschel
Just wanted to ask if there is any further information needed regarding my patch. > Gesendet: Sonntag, 13. Dezember 2015 um 11:54 Uhr > Von: "Stefan Pöschel" <basic.mas...@gmx.de> > An: ffmpeg-devel@ffmpeg.org > Betreff: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: ad

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: add flag to embed an AC-3/E-AC-3 ES the DVB way

2015-12-17 Thread Stefan Pöschel
*bump* Am 13.12.2015 um 15:23 schrieb Stefan Pöschel: > Am 13.12.2015 um 14:18 schrieb Moritz Barsnick: >> On Sun, Dec 13, 2015 at 11:54:32 +0100, Stefan Pöschel wrote: >>> So far an AC-3 elementary stream is refered to in the PMT according to >>> System A (ATSC

[FFmpeg-devel] [PATCH] avformat/mpegtsenc: add flag to embed an AC-3/E-AC-3 ES the DVB way

2015-12-13 Thread Stefan Pöschel
So far an AC-3 elementary stream is refered to in the PMT according to System A (ATSC). An E-AC-3 ES in contrast is embedded the System B (DVB) way. To fix this inconsistency, this commit changes the default E-AC-3 behaviour to use the ATSC way, too. Furthermore a new flag is added to optionally

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: add flag to embed an AC-3/E-AC-3 ES the DVB way

2015-12-13 Thread Stefan Pöschel
Am 13.12.2015 um 14:18 schrieb Moritz Barsnick: > On Sun, Dec 13, 2015 at 11:54:32 +0100, Stefan Pöschel wrote: >> So far an AC-3 elementary stream is refered to in the PMT according to >> System A (ATSC). An E-AC-3 ES in contrast is embedded the System B (DVB) way. > >

[FFmpeg-devel] mpegtsenc: signalling of AC-3 and E-AC-3 according to ATSC/DVB

2015-12-12 Thread Stefan Pöschel
since 2011). If there are no objections against my proposal, I would write and post a regarding patch. Thanks and regards, Stefan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel