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
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
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
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
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
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
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>:
>
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
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]
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
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
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
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.
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
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
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
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
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
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:
>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
>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
>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
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
>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
>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
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.
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
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
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
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
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
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
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
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]
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
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
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
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
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
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 ?
>
>
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
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
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
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
*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
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
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.
>
>
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
48 matches
Mail list logo