LGTM
-Original Message-
From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of
kaustubh.ra...@imgtec.com
Sent: Friday, October 27, 2017 6:59 PM
To: ffmpeg-devel@ffmpeg.org
Cc: Kaustubh Raste
Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc uni vt and hv mc msa
LGTM
-Original Message-
From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of
kaustubh.ra...@imgtec.com
Sent: Friday, October 27, 2017 6:35 PM
To: ffmpeg-devel@ffmpeg.org
Cc: Kaustubh Raste
Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Improve hevc bi hz and hv mc msa
when use frame_pts option, the output image name can be set with PTS
of current frame.
Signed-off-by: Steven Liu
---
doc/muxers.texi | 9 +
libavformat/img2enc.c | 8
2 files changed, 17 insertions(+)
diff --git a/doc/muxers.texi b/doc/muxers.texi
index 91bbe673c5..af5349
From 3ee6e7f01aa2c709afa7ad23f39af919c64f85d5 Mon Sep 17 00:00:00 2001
From: Jun Zhao
Date: Tue, 31 Oct 2017 16:09:45 +0800
Subject: [PATCH 1/3] examples/transcoding: suppress build warning.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
suppress the "w
From e192b42ab0fdc4b8faad50ff7bb01c6028945f19 Mon Sep 17 00:00:00 2001
From: Jun Zhao
Date: Tue, 31 Oct 2017 16:11:06 +0800
Subject: [PATCH 2/3] examples/filtering_video: suppress the build warning.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
suppres
From 6e8dbeb70f03312deee47a3cf0b5b47a7241ffb5 Mon Sep 17 00:00:00 2001
From: Jun Zhao
Date: Tue, 31 Oct 2017 16:12:36 +0800
Subject: [PATCH 3/3] examples/filtering_audio: suppress the build warning.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
suppres
2017-10-31 16:22 GMT+08:00 Jun Zhao :
>
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
From 3ee6e7f01aa2c709afa7ad23f39af919c64f85d5 Mon Sep 17 00:00:00 2001
From: Jun Zhao
Date: Tue, 31
2017-10-31 16:22 GMT+08:00 Jun Zhao :
>
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
patchset LGTM
___
ffmpeg-devel mailing list
ffmpeg-devel
Le decadi 10 brumaire, an CCXXVI, Martin Vignali a écrit :
> lgtm
Thanks, pushed.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmp
On Tue, 31 Oct 2017 at 02:26 Michael Niedermayer
wrote:
> On Thu, Oct 19, 2017 at 11:46:47AM -0700, Nikolas Bowe wrote:
> > Found via fuzzing.
> > /tmp/poc is a 1 MB mpegts file generated via fuzzing, where 1 packet has
> many NALUs
> > Before this change:
> > $ /usr/bin/time -f "\t%M Max Resid
On 31/10/17 02:37, Jun Zhao wrote:
> From d1e105057e93e7c2788d6d684292db9008fbf3ac Mon Sep 17 00:00:00 2001
> From: Jun Zhao
> Date: Tue, 31 Oct 2017 10:19:08 +0800
> Subject: [PATCH 2/2] lavc/vaapi_encode_h264: correct bit_rate_scale setting.
>
> As H264 Spec 2012 E.2.2, bit_rate_scale means the
On 31/10/17 02:37, Jun Zhao wrote:
> From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001
> From: Jun Zhao
> Date: Tue, 31 Oct 2017 10:13:42 +0800
> Subject: [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size calculate.
>
> when rc_buffer_size didn't setting, always use
Variables used in inline assembly need to be marked with attribute((used)).
Static constants already were, via the define of DECLARE_ASM_CONST.
But DECLARE_ALIGNED does not add this attribute, and some of the variables
defined with it are const only used in inline assembly, and therefore
appeared d
From: Kaustubh Raste
Use immediate unsigned saturation for clip to max saving one vector register.
Signed-off-by: Kaustubh Raste
---
libavcodec/mips/hevc_macros_msa.h |9 +
libavcodec/mips/hevc_mc_uniw_msa.c | 1598 +---
2 files changed, 965 insertions(+),
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
> Of Marton Balint
> Sent: October 30, 2017 9:26 PM
> To: FFmpeg development discussions and patches de...@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD
> GPUs
> >>> +AMF_ASSIGN_PROPERTY_BOOL(res, ctx->encoder,
> >> AMF_VIDEO_ENCODER_HEVC_DE_BLOCKING_FILTER_DISABLE,
> >> deblocking_filter);
> >>
> >> What about SAO?
> >
> > SAO ???
>
> You're looking at AV_CODEC_FLAG_LOOP_FILTER to disable this, so you
> might want to consider both loop filters in H
On 2017/10/31 18:19, Mark Thompson wrote:
> On 31/10/17 02:37, Jun Zhao wrote:
>> From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001
>> From: Jun Zhao
>> Date: Tue, 31 Oct 2017 10:13:42 +0800
>> Subject: [PATCH 1/2] lavc/vaapi_encode: correct the HRD buffer size
>> calculate
On 2017/10/31 18:26, Mark Thompson wrote:
> On 31/10/17 02:37, Jun Zhao wrote:
>> From d1e105057e93e7c2788d6d684292db9008fbf3ac Mon Sep 17 00:00:00 2001
>> From: Jun Zhao
>> Date: Tue, 31 Oct 2017 10:19:08 +0800
>> Subject: [PATCH 2/2] lavc/vaapi_encode_h264: correct bit_rate_scale setting.
>>
>
On Tue, Oct 31, 2017 at 12:16:18PM +, Thomas Köppe wrote:
> Variables used in inline assembly need to be marked with attribute((used)).
This should not be the case.
Variables referenced through in/out operands should not need that.
Only ones accessed directly bypassing operands should need thi
On 31/10/17 15:18, Jun Zhao wrote:
> On 2017/10/31 18:26, Mark Thompson wrote:
>> On 31/10/17 02:37, Jun Zhao wrote:
>>> From d1e105057e93e7c2788d6d684292db9008fbf3ac Mon Sep 17 00:00:00 2001
>>> From: Jun Zhao
>>> Date: Tue, 31 Oct 2017 10:19:08 +0800
>>> Subject: [PATCH 2/2] lavc/vaapi_encode_h2
On 31/10/17 15:11, Jun Zhao wrote:> On 2017/10/31 18:19, Mark Thompson wrote:
>> On 31/10/17 02:37, Jun Zhao wrote:
>>> From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001
>>> From: Jun Zhao
>>> Date: Tue, 31 Oct 2017 10:13:42 +0800
>>> Subject: [PATCH 1/2] lavc/vaapi_encode: co
when use frame_pts option, the output image name can be set with PTS
of current frame.
Signed-off-by: Steven Liu
---
doc/muxers.texi | 9 +
libavformat/img2enc.c | 7 +++
2 files changed, 16 insertions(+)
diff --git a/doc/muxers.texi b/doc/muxers.texi
index 91bbe673c5..af5349e
+Teresa, who drafted the patch initially.
On 31 October 2017 at 15:38, Michael Niedermayer
wrote:
> On Tue, Oct 31, 2017 at 12:16:18PM +, Thomas Köppe wrote:
> > Variables used in inline assembly need to be marked with
> attribute((used)).
>
> This should not be the case.
> Variables referen
On 2017/10/31 23:45, Mark Thompson wrote:
> On 31/10/17 15:11, Jun Zhao wrote:> On 2017/10/31 18:19, Mark Thompson wrote:
>>> On 31/10/17 02:37, Jun Zhao wrote:
From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001
From: Jun Zhao
Date: Tue, 31 Oct 2017 10:13:42 +0
Nack.
Le septidi 7 brumaire, an CCXXVI, Paul B Mahol a écrit :
> Signed-off-by: Paul B Mahol
> ---
> libavfilter/vf_tile.c | 12 +++-
> 1 file changed, 3 insertions(+), 9 deletions(-)
>
> diff --git a/libavfilter/vf_tile.c b/libavfilter/vf_tile.c
> index 87e0b940cf..5752ca080e 100644
>
On 31/10/17 16:00, Jun Zhao wrote:
> On 2017/10/31 23:45, Mark Thompson wrote:
>> On 31/10/17 15:11, Jun Zhao wrote:> On 2017/10/31 18:19, Mark Thompson wrote:
On 31/10/17 02:37, Jun Zhao wrote:
> From 7eef9be1c8a92bf625d62a0f97f762f1342c6d78 Mon Sep 17 00:00:00 2001
> From: Jun Zhao
On Tue, Oct 31, 2017 at 03:52:14PM +, Thomas Köppe wrote:
> +Teresa, who drafted the patch initially.
>
> On 31 October 2017 at 15:38, Michael Niedermayer
> wrote:
>
> > On Tue, Oct 31, 2017 at 12:16:18PM +, Thomas Köppe wrote:
> > > Variables used in inline assembly need to be marked wi
On Tue, Oct 31, 2017 at 12:40:24AM +0100, Carl Eugen Hoyos wrote:
> 2017-10-30 9:47 GMT+01:00 Muhammad Faiz :
> > On Sun, Oct 29, 2017 at 3:55 AM, Carl Eugen Hoyos
> > wrote:
> >> Hi!
> >>
> >> Attached patch fixes a random testcase for ticket #6785 here but I
> >> don't know if this is the corre
Removing the nvenc/cuvid headers from the tree would imply the following
procedure for anyone wanting to build ffmpeg with nvenc/cuvid:
Register with nvidia to get access to their Video SDK Downloads.
Extract the headers from their massive SDK.
Patch them so that ffmpeg can make use of them. The
On Mon, Oct 30, 2017 at 11:51:30PM +0100, Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch makes sure the loas muxer does not try to write
> anything but aac and latm.
>
> Please comment, Carl Eugen
> latmenc.c |4
> 1 file changed, 4 insertions(+)
> 2b64f3d5ecb189e77b85dbab7a6cbfe9657
On 31/10/17 16:34, Timo Rothenpieler wrote:
> Removing the nvenc/cuvid headers from the tree would imply the following
> procedure for anyone wanting to build ffmpeg with nvenc/cuvid:
>
> Register with nvidia to get access to their Video SDK Downloads.
> Extract the headers from their massive SDK
On Sun, Oct 29, 2017 at 11:11:43PM +0100, pkv.stream wrote:
> minor change, adding a test for number of channels for
> mapping_family 1 when channel_config_code is > 0x82
> Passes fate + patcheck.
> Regards
> pkv
> mpegts.c | 84
> ---
On Mon, Oct 30, 2017 at 8:07 PM, Michael Niedermayer
wrote:
> On Mon, Oct 30, 2017 at 02:14:35PM +0700, Muhammad Faiz wrote:
>> On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz wrote:
>> > Fix tsan warnings.
>> >
>> > Signed-off-by: Muhammad Faiz
>> > ---
>> > libavutil/crc.c | 49 +++
On Mon, Oct 30, 2017 at 11:36:34AM +, Manojkumar Bhosale wrote:
> LGTM
applied
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever,
On Mon, Oct 30, 2017 at 11:36:41AM +, Manojkumar Bhosale wrote:
> LGTM
applied
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is
On Tue, Oct 31, 2017 at 06:59:57AM +, Manojkumar Bhosale wrote:
> LGTM
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Avoid a single point of failure, be that a person or equipment.
signature.asc
Description: Digital signature
On Tue, Oct 31, 2017 at 07:00:03AM +, Manojkumar Bhosale wrote:
> LGTM
will apply
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the h
On Tue, Oct 31, 2017 at 07:00:08AM +, Manojkumar Bhosale wrote:
> LGTM
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
Signed-off-by: Aaron Levinson
---
configure | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure b/configure
index ea22d692b9..572299c9d3 100755
--- a/configure
+++ b/configure
@@ -3327,7 +3327,7 @@ avformat_deps="avcodec avutil"
avformat_suggest="libm network zlib"
Signed-off-by: Aaron Levinson
---
configure | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure b/configure
index 572299c9d3..2ac6fed98a 100755
--- a/configure
+++ b/configure
@@ -5994,7 +5994,7 @@ enabled libkvazaar&& require_pkg_config
libkvazaar "kvazaar >= 0
This makes the autobsf feature behave the same as the manual
bitstream filtering in ffmpeg.c
Fixes ticket #6794
Signed-off-by: James Almer
---
libavformat/mux.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/libavformat/mux.c b/libavformat/mux.c
index 77e03ee5ba..1445e7d
On Tue, 31 Oct 2017, Mironov, Mikhail wrote:
[...]
I see some confusion. The user can call send_frame/receive_packet in any
order, and you can implement send_frame and receive_packet any way you
want, the only thing you have to guarantee is that you cannot return EAGAIN
for both send_frame and
When ffmpeg is built using MSVC, symbols are stored separately from
executables and libraries in .pdb files. However, unlike with
gcc-based builds, when make install is invoked, symbols, in the form
of .pdb files, which are important for debugging, are not copied to
bindir. This change corrects t
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
> Of Marton Balint
> Sent: October 31, 2017 2:06 PM
> To: FFmpeg development discussions and patches de...@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD
> GPUs
On Tue, Oct 31, 2017 at 6:50 PM, Aaron Levinson
wrote:
> Signed-off-by: Aaron Levinson
> ---
> configure | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index ea22d692b9..572299c9d3 100755
> --- a/configure
> +++ b/configure
> @@ -3327,7 +3327
Hello,
First patch submission... hopefully I do all of this correctly.?
Added basic support for H264 Annex G (Scalable Video Coding) to RTP module.
Implement splitting of PACSI NAL units (type 30).
Used existing H264 SPS decoder to parse Subset SPS NAL units (type 15) in h264
extradata.
As a
On Tue, Oct 31, 2017 at 12:47:06AM -0300, James Almer wrote:
> This prevents making the DCAParseError enum part of the ABI.
>
> Signed-off-by: James Almer
> ---
> libavcodec/dca.c | 11 ---
> libavcodec/dca.h | 12 ++--
> 2 files changed, 18 insertions(+), 5 deletions(-)
this ch
It's needed for the same reason the used attribute was already added to the
"static const" variables. For those, when building with just -O2, they
could be removed by optimization since they had local (file) scope, and we
couldn't see the uses in the inline assembly (without the used attribute).
Wi
Signed-off-by: Paul B Mahol
---
libavfilter/vf_tile.c | 17 +
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/libavfilter/vf_tile.c b/libavfilter/vf_tile.c
index 87e0b940cf..8eb0dc2097 100644
--- a/libavfilter/vf_tile.c
+++ b/libavfilter/vf_tile.c
@@ -23,6 +23,7 @@
On 10/31/2017 11:44 AM, Hendrik Leppkes wrote:
On Tue, Oct 31, 2017 at 6:50 PM, Aaron Levinson
wrote:
Signed-off-by: Aaron Levinson
---
configure | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure b/configure
index ea22d692b9..572299c9d3 100755
--- a/configure
Fixes: Ticket6487
Signed-off-by: Michael Niedermayer
---
libavformat/avformat.h| 1 +
libavformat/utils.c | 16
tests/ref/fate/mov-zombie | 4 ++--
3 files changed, 15 insertions(+), 6 deletions(-)
diff --git a/libavformat/avformat.h b/libavformat/avformat.h
index b
Dear List,
I am compiling a stripped-down, static versions of the libraries, i.e. :
libavcodec.a libavformat.a libavutil.a libswscale.a libavfilter.a
libswresample.a
and I'm "baking them in" into a custom shared-library.
All external libraries are disabled, and configure script tells me that
>
> This reduces memory consumption to linear from qudratic but i think
> it still can be made to allocate very large amounts of memory.
> That is with many small NAL units MAX_MBPAIR_SIZE would be allocated
> for each.in worst case.
> So this does fix the qudratic issue but not the OOM issue.
> Us
On 10/31/2017 2:25 AM, Michael Niedermayer wrote:
> (though as said, this fix is not ideal or complete, I would very much
> prefer if this would be fixed by using a single buffer or any other
> solution that avoids the speedloss.)
Using a single buffer would be marginally faster, but it does not
This prevents making the DCAParseError enum part of the ABI.
Signed-off-by: James Almer
---
Now actually paying attention to -Wempty-body warnings.
libavcodec/dca.c | 11 ---
libavcodec/dca.h | 12 ++--
2 files changed, 18 insertions(+), 5 deletions(-)
diff --git a/libavcodec/d
>
> Am I missing something here?
>
> P.S. I see Kieran mailed the same thing as I wrote this.
>
Further to Derek's excellent explanation, I think the best way is to go
back to the old "in-place" NAL parsing code for H264.
I will prepare a patch to do this.
Kieran
_
Thank you Kieran, Michael, Derek, and Carl.
I ran some benchmarks and the patch I provided really does have a speed
impact so I was considering alternative ways to achieve the same result.
After a little thought last week I came to the same conclusion as Michael
that a single rbsp_buffer per packe
On Tue, 31 Oct 2017 at 22:36 Niki Bowe
wrote:
> I was going to prepare a patch, but it sounds like Kieran is going to.
> Thank you Kieran. Let me know if there's anything you want me to do, or if
> you would prefer me to do it for the experience.
>
> As for the NAL memory pool, I think I see what
Le 31/10/2017 à 6:28 PM, Michael Niedermayer a écrit :
mpegts.c | 84
---
1 file changed, 70 insertions(+), 14 deletions(-)
6648bab53735f1efaf0d6cd698dbc2398b483c3f
0002-libavf-mpegts-opus-demuxing-for-mapping-family-255.patch
From
Le 29/10/2017 à 4:44 PM, Michael Niedermayer a écrit :
On Sat, Oct 28, 2017 at 03:49:13AM +0200, pkv.stream wrote:
mpegtsenc.c | 67
+---
1 file changed, 60 insertions(+), 7 deletions(-)
1423a193788547952e3c4cdcb424b4724b0f1157
000
2017-10-31 20:38 GMT+01:00 Sampsa Riikonen :
> I am compiling
Please post on the user mailing list:
http://ffmpeg.org/contact.html#MailingLists
Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-d
Partially fixes t/6699.
---
libavformat/mov.c | 125 +++--
tests/fate/mov.mak | 8
2 files changed, 90 insertions(+), 43 deletions(-)
diff --git a/libavformat/mov.c b/libavformat/mov.c
index 60f0228e2d..a295445651 100644
--- a/libavformat/mo
Attaching the fate sample.
On Tue, Oct 31, 2017 at 4:42 PM, Sasi Inguva wrote:
> Partially fixes t/6699.
> ---
> libavformat/mov.c | 125 ++
> +--
> tests/fate/mov.mak | 8
> 2 files changed, 90 insertions(+), 43 deletions(-)
>
> diff --gi
2017-10-31 17:52 GMT+01:00 Mark Thompson :
> On 31/10/17 16:34, Timo Rothenpieler wrote:
>> We're not bundling entire 3rd party libs in-tree here, just
>> API headers for system/driver APIs.
>
> What exactly would the policy be, then?
>
> "External headers for a system/driver API may be included i
On Tue, Oct 31, 2017 at 01:24:58AM -0300, James Almer wrote:
> This removes sizeof(AC3HeaderInfo) from the ABI.
>
> Signed-off-by: James Almer
> ---
> libavformat/movenc.c | 72
> +++-
> 1 file changed, 43 insertions(+), 29 deletions(-)
LGTM
thx
On 31/10/17 23:52, Carl Eugen Hoyos wrote:
> 2017-10-31 17:52 GMT+01:00 Mark Thompson :
>> On 31/10/17 16:34, Timo Rothenpieler wrote:
>
>>> We're not bundling entire 3rd party libs in-tree here, just
>>> API headers for system/driver APIs.
>>
>> What exactly would the policy be, then?
>>
>> "Exte
2017-11-01 1:04 GMT+01:00 Mark Thompson :
> This is why I'm concerned that we are facilitating anti-open
> behaviour from Nvidia by making it easier to use the
> closed software than more open alternatives.
But you do agree that there is nothing "open" about the AMD
driver in question?
Carl Eugen
2017-10-31 17:28 GMT+01:00 Michael Niedermayer :
>> rematrix.c | 21 +++--
>> 1 file changed, 15 insertions(+), 6 deletions(-)
>> 9f50e61370cc72de000bb1174cbd8666388fdd9a 0001-lswr-Support-s32p.patch
>> From a93b9309d74f5eadece371ee1e682d266af6cd83 Mon Sep 17 00:00:00 2001
>> F
Hi Sampsa
On Tue, Oct 31, 2017 at 21:38:42 +0200, Sampsa Riikonen wrote:
> Dear List,
> I am compiling a stripped-down, static versions of the libraries, i.e. :
>
> libavcodec.a libavformat.a libavutil.a libswscale.a libavfilter.a
> libswresample.a
>
> and I'm "baking them in" into a custom sha
On 01/11/17 00:07, Carl Eugen Hoyos wrote:
> 2017-11-01 1:04 GMT+01:00 Mark Thompson :
>> This is why I'm concerned that we are facilitating anti-open
>> behaviour from Nvidia by making it easier to use the
>> closed software than more open alternatives.
>
> But you do agree that there is nothing
On 01/11/17 00:17, Mark Thompson wrote:
> On 01/11/17 00:07, Carl Eugen Hoyos wrote:
>> 2017-11-01 1:04 GMT+01:00 Mark Thompson :
>>> This is why I'm concerned that we are facilitating anti-open
>>> behaviour from Nvidia by making it easier to use the
>>> closed software than more open alternatives
2017-11-01 1:23 GMT+01:00 Mark Thompson :
> Or libx264, which also lacks headers inside ffmpeg. (We
> could dynamically load libx264 using reverse-enginereed
> non-GPL headers and then be able to enable it everywhere!
But only with --enable-gpl, making the stunt less useful.
Anyway, this wasn't
Signed-off-by: Moritz Barsnick
---
The only remaining unchecked one I could find in lavf. Fixes a warning,
obviously.
libavfilter/vf_ocr.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/libavfilter/vf_ocr.c b/libavfilter/vf_ocr.c
index e003982f05..abfff49438 100644
--- a
On 01/11/17 00:29, Carl Eugen Hoyos wrote:
> 2017-11-01 1:23 GMT+01:00 Mark Thompson :
>
>> Or libx264, which also lacks headers inside ffmpeg. (We
>> could dynamically load libx264 using reverse-enginereed
>> non-GPL headers and then be able to enable it everywhere!
>
> But only with --enable-g
Hi
On Tue, Oct 31, 2017 at 04:29:18PM +, Teresa Johnson wrote:
> It's needed for the same reason the used attribute was already added to the
> "static const" variables. For those, when building with just -O2, they
> could be removed by optimization since they had local (file) scope, and we
> c
On Wed, Nov 01, 2017 at 12:34:45AM +, Mark Thompson wrote:
> On 01/11/17 00:29, Carl Eugen Hoyos wrote:
> > 2017-11-01 1:23 GMT+01:00 Mark Thompson :
> >
> >> Or libx264, which also lacks headers inside ffmpeg. (We
> >> could dynamically load libx264 using reverse-enginereed
> >> non-GPL head
2017-11-01 1:34 GMT+01:00 Mark Thompson :
> On 01/11/17 00:29, Carl Eugen Hoyos wrote:
>> 2017-11-01 1:23 GMT+01:00 Mark Thompson :
>>
>>> Or libx264, which also lacks headers inside ffmpeg. (We
>>> could dynamically load libx264 using reverse-enginereed
>>> non-GPL headers and then be able to ena
2017-10-31 19:00 GMT+01:00 James Almer :
> This makes the autobsf feature behave the same as the manual
> bitstream filtering in ffmpeg.c
>
> Fixes ticket #6794
>
> Signed-off-by: James Almer
> ---
> libavformat/mux.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/lib
2017-10-31 19:53 GMT+01:00 Breeden, Joshua :
> Hello,
>
> First patch submission... hopefully I do all of this correctly.?
Your patches contain a lot of trailing whitespace (afaict...)
that cannot be committed to our repository, please remove it.
To find it you can use the script "tools/patcheck"
On Tue, Oct 31, 2017 at 04:29:02PM +0800, Steven Liu wrote:
> 2017-10-31 16:22 GMT+08:00 Jun Zhao :
> >
> >
> > ___
> > ffmpeg-devel mailing list
> > ffmpeg-devel@ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
>
> patchset LGTM
will
2017-10-31 17:38 GMT+01:00 Michael Niedermayer :
> On Mon, Oct 30, 2017 at 11:51:30PM +0100, Carl Eugen Hoyos wrote:
>> Hi!
>>
>> Attached patch makes sure the loas muxer does not try to write
>> anything but aac and latm.
>>
>> Please comment, Carl Eugen
>
>> latmenc.c |4
>> 1 file chan
On Tue, Oct 31, 2017 at 08:24:37PM +, Derek Buitenhuis wrote:
> On 10/31/2017 2:25 AM, Michael Niedermayer wrote:
> > (though as said, this fix is not ideal or complete, I would very much
> > prefer if this would be fixed by using a single buffer or any other
> > solution that avoids the spee
On Tue, Oct 31, 2017 at 04:42:56PM -0700, Sasi Inguva wrote:
> Partially fixes t/6699.
> ---
> libavformat/mov.c | 125
> +++--
> tests/fate/mov.mak | 8
> 2 files changed, 90 insertions(+), 43 deletions(-)
Missing: ./tests/ref/fate/mov-ibi
On Wed, Nov 01, 2017 at 02:17:06AM +0100, Carl Eugen Hoyos wrote:
> 2017-10-31 17:38 GMT+01:00 Michael Niedermayer :
> > On Mon, Oct 30, 2017 at 11:51:30PM +0100, Carl Eugen Hoyos wrote:
> >> Hi!
> >>
> >> Attached patch makes sure the loas muxer does not try to write
> >> anything but aac and latm
On Tue, Oct 31, 2017 at 05:27:35PM -0300, James Almer wrote:
> This prevents making the DCAParseError enum part of the ABI.
>
> Signed-off-by: James Almer
> ---
> Now actually paying attention to -Wempty-body warnings.
>
> libavcodec/dca.c | 11 ---
> libavcodec/dca.h | 12 ++--
2017-11-01 8:30 GMT+08:00 Moritz Barsnick :
> Signed-off-by: Moritz Barsnick
> ---
>
> The only remaining unchecked one I could find in lavf. Fixes a warning,
> obviously.
>
> libavfilter/vf_ocr.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/libavfilter/vf_ocr.c b/l
Added the fate ref file to the commit. Thx.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Partially fixes t/6699.
---
libavformat/mov.c| 125 +++
tests/fate/mov.mak | 8 +++
tests/ref/fate/mov-ibi-elst-starts-b | 33 +
3 files changed, 123 insertions(+), 43 deletions(-)
create mode 100644 tests/ref/fate/m
Thanks. Sent the corrected patch.
On Tue, Oct 31, 2017 at 7:14 PM, Michael Niedermayer wrote:
> On Tue, Oct 31, 2017 at 04:42:56PM -0700, Sasi Inguva wrote:
> > Partially fixes t/6699.
> > ---
> > libavformat/mov.c | 125 ++
> +--
> > tests/fate/
From: Kaustubh Raste
Remove loops and unroll as block sizes are known.
Load the specific destination bytes instead of MSA load and pack.
Remove unused macro and functions.
Signed-off-by: Kaustubh Raste
---
libavcodec/mips/h264qpel_msa.c | 1002 +++-
1 file c
90 matches
Mail list logo
