Hi,
On Mon, Dec 26, 2016 at 4:53 AM, Henrik Gramner wrote:
> On Mon, Dec 26, 2016 at 2:32 AM, Ronald S. Bultje
> wrote:
> > I know I'm terribly nitpicking here for the limited scope of the comment,
> > but this only matters for functions that have a return value. Do y
Hi,
On Sun, Dec 25, 2016 at 2:24 PM, Henrik Gramner wrote:
> When allocating stack space with an alignment requirement that is larger
> than the current stack alignment we need to store a copy of the original
> stack pointer in order to be able to restore it later.
>
> If we chose to use another
Hi,
On Wed, Nov 30, 2016 at 6:48 AM, Mark Thompson wrote:
> On 30/11/16 03:37, Ronald S. Bultje wrote:
> > Hi,
> >
> > On Mon, Nov 28, 2016 at 6:50 PM, Mark Thompson wrote:
> >
> >> ---
> >> Incomplete, but enough to work with the encoder in patch 3
Hi,
On Mon, Nov 28, 2016 at 6:50 PM, Mark Thompson wrote:
> ---
> Incomplete, but enough to work with the encoder in patch 3.
>
> Todo:
> * Superframe splitting.
>
>From what I understand, Anton ported the vp9 parser into a BSF, which
resolves this.
> * More options to control recombination.
Hi,
On Thu, Nov 17, 2016 at 7:37 AM, Martin Storsjö wrote:
> On Thu, 17 Nov 2016, Ronald S. Bultje wrote:
>
> Hi,
>>
>> On Mon, Nov 14, 2016 at 4:46 PM, Martin Storsjö wrote:
>>
>> The dc-only mode is already checked to work correctly above, but this
>
Hi,
On Mon, Nov 14, 2016 at 4:46 PM, Martin Storsjö wrote:
> The dc-only mode is already checked to work correctly above, but this
> allows benchmarking this mode for performance tuning, and allows making
> sure that it actually is correctly hooked up.
> ---
> tests/checkasm/vp9dsp.c | 6 ++
Hi,
On Fri, Jul 8, 2016 at 3:00 PM, Martin Storsjö wrote:
> On Fri, 8 Jul 2016, Ronald S. Bultje wrote:
>
> Hi,
>>
>> On Thu, Jul 7, 2016 at 7:08 PM, Janne Grunau
>> wrote:
>>
>> +#define SRC_BUF_STRIDE 32
>>>> +#define SRC_BUF_SIZE ((size
Hi,
On Thu, Jul 7, 2016 at 7:08 PM, Janne Grunau wrote:
> > +#define SRC_BUF_STRIDE 32
> > +#define SRC_BUF_SIZE ((size + 5) * SRC_BUF_STRIDE)
> > +// The 2 * stride + 2 offset is necessary to avoid reading out of
> bounds,
>
> that sounds a little misleading. just stating that the mc sub pixel
Hi,
On Thu, Jul 7, 2016 at 10:53 AM, Ronald S. Bultje
wrote:
> Hi,
>
> On Thu, Jul 7, 2016 at 9:52 AM, Alexandra Hájková <
> alexandra.khirn...@gmail.com> wrote:
>
>> On Thu, Jul 7, 2016 at 1:53 PM, Ronald S. Bultje
>> wrote:
>> > On Thu,
Hi,
On Thu, Jul 7, 2016 at 9:52 AM, Alexandra Hájková <
alexandra.khirn...@gmail.com> wrote:
> On Thu, Jul 7, 2016 at 1:53 PM, Ronald S. Bultje
> wrote:
> > On Thu, Jul 7, 2016 at 5:25 AM, Alexandra Hájková <
> > alexandra.khirn...@gmail.com> wrote:
>
Hi,
On Thu, Jul 7, 2016 at 5:25 AM, Alexandra Hájková <
alexandra.khirn...@gmail.com> wrote:
> else if (lc->cu.pred_mode == MODE_INTRA && c_idx == 0 &&
> log2_trafo_size == 2)
> -s->hevcdsp.transform_4x4_luma_add(dst, coeffs, stride);
> +s->hevcd
Hi,
one more thing...
On Tue, Jul 5, 2016 at 6:36 AM, Ronald S. Bultje wrote:
> On Tue, Jul 5, 2016 at 5:58 AM, Martin Storsjö wrote:
>
>> On Fri, 1 Jul 2016, Martin Storsjö wrote:
>>
>>> Index 0 is 16x16, 1 is 8x8, 2 is 4x4.
>>
>>
This isn't a
Hi,
On Tue, Jul 5, 2016 at 5:58 AM, Martin Storsjö wrote:
> On Fri, 1 Jul 2016, Martin Storsjö wrote:
>
> Index 0 is 16x16, 1 is 8x8, 2 is 4x4.
>> ---
>> libavcodec/vp8dsp.h | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavcodec/vp8dsp.h b/libavcodec/vp8dsp.h
>
Hi,
On Wed, May 25, 2016 at 7:38 PM, Kieran Kunhya wrote:
> On Wed, 25 May 2016 at 19:37 Vittorio Giovara
> wrote:
>
> > On Wed, May 25, 2016 at 10:49 AM, Ronald S. Bultje
> > wrote:
> > > Hi,
> > >
> > > On Wed, May 25, 2016 at 9:54 AM, Luca Ba
Hi,
On Wed, May 25, 2016 at 10:59 AM, Diego Biurrun wrote:
> On Wed, May 25, 2016 at 10:49:12AM -0400, Ronald S. Bultje wrote:
> > It's not like this is to be committed tomorrow, right? So take your time,
> > not urgent...
>
> Ummm, no; it's overdue already and wa
Hi,
On Wed, May 25, 2016 at 9:54 AM, Luca Barbato wrote:
> On 25/05/16 15:32, Ronald S. Bultje wrote:
> > I agree, ARM 32bit results would be very interesting.
>
> The odroid images I have are arm64 only (and I'm still figuring out how
> to set it up properly), if you h
Hi,
On Wed, May 25, 2016 at 8:20 AM, Luca Barbato wrote:
> On 25/05/16 12:31, Anton Khirnov wrote:
> > All the results quoted here seem to be from x86 and 32bit x86 is not
> > really all that relevant these days. Did anyone do any ARM tests?
>
> On power8 I'm seeing a 17% speedup for huffyuv, 10
Hi,
On Tue, May 24, 2016 at 3:47 PM, Luca Barbato wrote:
> On 23/05/16 17:01, Ronald S. Bultje wrote:
> > Howdy,
>
> Interesting. I spent a bit of time on it myself.
>
> I run some benchmark using a yuv422 file of the right size from the
> Tim's collection [directly
Howdy,
On Sun, May 22, 2016 at 5:27 AM, Alexandra Hájková <
alexandra.khirn...@gmail.com> wrote:
> > Do you have a tree for testing somewhere?
>
> Yes, there's github branch:
> https://github.com/sasshka/libav/tree/get_bits3.
>
Thanks!
> If I find decoders that are slower on 32bit after the pat
Hi,
On Sat, May 21, 2016 at 2:39 AM, Alexandra Hájková <
alexandra.khirn...@gmail.com> wrote:
> >
> > I noticed proresdec (for example) is not converted to the new bitstream
> > reader. Is there a reason for that?
>
> Not all the sets are sent yet, I'm sending them gradually to make the
> reviewi
Hi,
On Fri, May 20, 2016 at 4:11 PM, Alexandra Hájková <
alexandra.khirn...@gmail.com> wrote:
> This set is compilable together only.
I noticed proresdec (for example) is not converted to the new bitstream
reader. Is there a reason for that?
Also, since this patch basically converts the bitstr
Hi,
On Wed, Apr 27, 2016 at 7:37 AM, Alexandra Hájková <
alexandra.khirn...@gmail.com> wrote:
> ---
> libavcodec/bitstream.h | 475
> +
> 1 file changed, 475 insertions(+)
> create mode 100644 libavcodec/bitstream.h
>
> diff --git a/libavcodec/bit
Hi,
On Thu, Feb 25, 2016 at 2:40 PM, Anton Khirnov wrote:
> Quoting Ronald S. Bultje (2016-02-25 19:48:10)
> > Hi,
> >
> > given I'm writing a BSF right now...
> >
> > On Thu, Feb 25, 2016 at 10:05 AM, Anton Khirnov
> wrote:
> >
> >
Hi,
given I'm writing a BSF right now...
On Thu, Feb 25, 2016 at 10:05 AM, Anton Khirnov wrote:
> +int (*filter)(AVBSFContext *ctx, AVPacket *pkt);
>
How do we skip packets? Like, say we have a BSF that merges two packets
together into one, how does that work? Maybe it should return a pack
Hi,
On Sun, Jan 24, 2016 at 7:34 PM, Kieran Kunhya wrote:
> +static inline void filter(int16_t *output, ptrdiff_t out_stride, int16_t
> *low, ptrdiff_t low_stride,
> + int16_t *high, ptrdiff_t high_stride, int len,
> uint8_t clip)
Should this be a DSP function? (That i
Hi,
On Mon, Jan 18, 2016 at 9:37 AM, Henrik Gramner wrote:
> On Mon, Jan 18, 2016 at 2:35 PM, Ronald S. Bultje
> wrote:
> > On Sun, Jan 17, 2016 at 6:21 PM, Henrik Gramner
> wrote:
> >> @@ -386,8 +386,10 @@ DECLARE_REG_TMP_SIZE
> >> 0,1,2,3,4,5,6,7,8,9,10,11,1
Hi,
On Sun, Jan 17, 2016 at 6:21 PM, Henrik Gramner wrote:
> When allocating stack space with a larger alignment than the known stack
> alignment a temporary register is used for storing the stack pointer.
> Ensure that this isn't one of the registers used for passing arguments.
> ---
> libavut
Hi,
On Fri, Aug 21, 2015 at 10:02 AM, Anton Khirnov wrote:
> Quoting Ronald S. Bultje (2015-08-21 14:24:47)
> > Hi,
> >
> > On Wed, Aug 19, 2015 at 7:31 PM, James Almer wrote:
> >
> > > On 19/08/15 8:23 PM, Ronald S. Bultje wrote:
> > > > Hi,
Hi,
On Wed, Aug 19, 2015 at 7:31 PM, James Almer wrote:
> On 19/08/15 8:23 PM, Ronald S. Bultje wrote:
> > Hi,
> >
> > On Wed, Aug 19, 2015 at 6:34 PM, James Almer wrote:
> >
> >> On 19/08/15 4:43 PM, Anton Khirnov wrote:
> >>>
Hi,
On Wed, Aug 19, 2015 at 6:34 PM, James Almer wrote:
> On 19/08/15 4:43 PM, Anton Khirnov wrote:
> > ---
> > libavcodec/hevc.c | 6 +-
> > libavcodec/hevc.h | 2 +-
> > libavcodec/hevcdsp.c | 24 +-
> > libavcodec/hevcdsp.h | 5 +-
> > libavco
Hi,
On Sat, Aug 8, 2015 at 7:37 AM, Andreas Cadhalpun <
andreas.cadhal...@googlemail.com> wrote:
> Signed-off-by: Andreas Cadhalpun
> ---
> libavcodec/version.h | 54
> +--
> libavfilter/version.h | 10 +-
> libavformat/version.h | 6 +++
Hi,
On Sun, Aug 9, 2015 at 11:54 AM, Andreas Cadhalpun <
andreas.cadhal...@googlemail.com> wrote:
> Hi,
>
> On 09.08.2015 12:57, Ronald S. Bultje wrote:
> > Yeah, I'm with this. Andreas, the correct fix is to update applications,
> > even if that means vendor-s
Hi,
On Tue, Jul 21, 2015 at 12:58 PM, Kostya Shishkov wrote:
> On Tue, Jul 21, 2015 at 11:52:55AM -0400, Dave Rice wrote:
> > Hi all,
> [...]
> > The FFV1 specification work may also be reviewed at github [5] with
> recent rendering in HTML [6] and PDF [7] available. To participate in the
> curr
Hi,
On Sun, Jun 28, 2015 at 5:28 AM, Andreas Cadhalpun <
andreas.cadhal...@googlemail.com> wrote:
> On 27.06.2015 23:01, Michael Niedermayer wrote:
> > On Sat, Jun 27, 2015 at 08:36:15PM +0200, Andreas Cadhalpun wrote:
> >> Claiming to have decoded more bytes than the packet size is wrong.
> >>
>
Hi,
On Sun, Jun 7, 2015 at 10:05 AM, Andreas Cadhalpun <
andreas.cadhal...@googlemail.com> wrote:
> +#define MARGIN (16 << 2)
> +#define MAX_MB_SIZE (((INT16_MAX - MARGIN) >> 6) + 1)
>
So this is roughly 9 bits.
> +if (s->avctx->coded_width > MAX_MB_SIZE * 16 ||
> +s->avctx->coded
Hi,
On Tue, Jan 13, 2015 at 8:43 AM, Luca Barbato wrote:
>
> On 13/01/15 13:43, Martin Storsjö wrote:
> > From: Michael Niedermayer
> >
> > This fixes out of array reads and/or infinite loops.
> >
> > CC: libav-sta...@libav.org
> > Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> > ---
>
Hi,
On Mon, Sep 29, 2014 at 7:07 AM, Luca Barbato wrote:
> On 29/09/14 11:35, Hendrik Leppkes wrote:
>
>> On Mon, Sep 29, 2014 at 9:33 AM, Nicolas Bertrand <
>> nicoinatte...@gmail.com>
>> wrote:
>>
>> Finally a 1st optimization patch for jpeg2000!
>>> To start: an easy one.
>>>
>>> The main he
Hi,
On Mon, Sep 22, 2014 at 12:35 PM, Georg Stein
wrote:
> when i call
>
> int result = avformat_open_input(&this->avFormatContext, fileNameChar, fmt,
> NULL);
>
> the result is 0 but the AVFormatContext does not has any codec/stream in
> the command line tool from ffmepg can download the stream
Hi,
On Fri, Aug 29, 2014 at 3:51 PM, Diego Biurrun wrote:
>
> Am 29.08.2014 20:03 schrieb "Ronald S. Bultje" :
> > On Fri, Aug 29, 2014 at 1:30 PM, Kieran Kunhya wrote:
> > > "You're throwing a blob at me without individual author's
> > &g
Hi,
On Fri, Aug 29, 2014 at 1:30 PM, Kieran Kunhya wrote:
> "You're throwing a blob at me without individual author's
> contribution, I don't think that's
> appropriate."
>
> You've decided to selectively ignore the second part of the message.
> As I read that it says he is not happy with those
Hi,
On Mon, Apr 14, 2014 at 1:31 PM, Martin Storsjö wrote:
> On Mon, 14 Apr 2014, Ronald S. Bultje wrote:
>
>> On Mon, Apr 14, 2014 at 4:49 AM, Martin Storsjö wrote:
>>
>>> On Mon, 14 Apr 2014, Kostya Shishkov wrote:
>>> On Mon, Apr 14, 2014 at 11
Hi,
On Mon, Apr 14, 2014 at 4:49 AM, Martin Storsjö wrote:
> On Mon, 14 Apr 2014, Kostya Shishkov wrote:
>
> On Mon, Apr 14, 2014 at 11:41:40AM +0300, Martin Storsjö wrote:
>>
>>> This hopefully fixes the vp7 fate test on sparc.
>>> ---
>>> None of the current vp8 fate tests trigger these codep
Hi,
On Thu, Nov 14, 2013 at 7:58 PM, Luca Barbato wrote:
> From: "Ronald S. Bultje"
>
> Originally written by Ronald S. Bultje and
> Clément Bœsch
>
> Further contributions by:
Please define the "further contributions". I am the original author,
togeth
Hi,
On Mon, Nov 4, 2013 at 11:27 AM, Luca Barbato wrote:
> From: "Ronald S. Bultje"
>
> Originally written by Ronald S. Bultje with the
> help of Clément Bœsch .
>
No. The decoder was written by A and B, not A with the help of B.
Further contributions by:
> A
Hi,
On Fri, Oct 25, 2013 at 4:35 AM, Peter B. wrote:
> On 10/24/2013 11:52 PM, Ronald S. Bultje wrote:
> > Hi,
> >
> > On Thu, Oct 24, 2013 at 5:29 PM, Peter B. wrote:
> >
> >> On 10/21/2013 12:43 AM, Luca Barbato wrote:
> >>> Basically you
Hi,
On Thu, Oct 24, 2013 at 5:29 PM, Peter B. wrote:
> On 10/21/2013 12:43 AM, Luca Barbato wrote:
> > Basically you should make sample files (so 1 frame or 3 frame sample)
> > that sports specific corner cases or common cases.
> >
> > e.g. all the kind of capture you want to make using it.
>
>
Hi,
On Tue, Oct 22, 2013 at 10:17 AM, Derek Buitenhuis <
derek.buitenh...@gmail.com> wrote:
> On 10/22/2013 3:08 PM, Ronald S. Bultje wrote:
> > And that supposedly-magic doesn't concern you?
>
> Definitely does.
>
> Guess the consensus is to fail immediately?
Hi,
On Tue, Oct 22, 2013 at 10:06 AM, Derek Buitenhuis <
derek.buitenh...@gmail.com> wrote:
> On 10/22/2013 2:47 PM, Ronald S. Bultje wrote:
> > Right. So the thinking might be, should we perhaps take action based on
> > this hypothetical error-like value of ret, suc
Hi,
On Tue, Oct 22, 2013 at 9:29 AM, Derek Buitenhuis <
derek.buitenh...@gmail.com> wrote:
> On 10/22/2013 2:15 PM, Ronald S. Bultje wrote:
> > if ((ret = init_get_bits8(...)) >= 0)
> > ret = ff_mpeg4_decode_...(..);
>
> Currently this accomplishes nothing, si
Hi,
On Tue, Oct 22, 2013 at 7:19 AM, Derek Buitenhuis <
derek.buitenh...@gmail.com> wrote:
> On 10/22/2013 12:09 PM, Luca Barbato wrote:
> >> -init_get_bits(&gb, s->avctx->extradata,
> s->avctx->extradata_size*8);
> >> -ret = ff_mpeg4_decode_picture_header(s, &gb);
> >> +
Hi,
On Mon, Oct 14, 2013 at 2:48 PM, Luca Barbato wrote:
> On 14/10/13 15:54, Ronald S. Bultje wrote:
> > Hi,
> >
> > On Sun, Oct 13, 2013 at 11:55 PM, Luca Barbato
> wrote:
> >
> >> -static av_noinline void emulated_edge_mc_sse(
Hi,
On Sun, Oct 13, 2013 at 11:55 PM, Luca Barbato wrote:
> -static av_noinline void emulated_edge_mc_sse(uint8_t *buf, const uint8_t
> *src,
> - ptrdiff_t linesize,
> +static av_noinline void emulated_edge_mc_sse(uint8_t * buf,const uint8_t
> *src,
>
Hi Loren,
On Sat, Jun 15, 2013 at 5:53 PM, Loren Merritt wrote:
> 1.5x-1.8x faster on sandybridge
> ---
> libavutil/lls.c | 3 +++
> libavutil/lls.h | 1 +
> libavutil/x86/lls.asm| 31 +++
> libavutil/x86/lls_init.c | 6 +-
> 4 files chang
Hi,
On Fri, Feb 22, 2013 at 11:25 AM, Ronald S. Bultje wrote:
> On Thu, Feb 21, 2013 at 1:11 AM, Martin Storsjö wrote:
>> +%if WIN64
>> +cglobal deblock_%1_luma_intra_8, 4,6,16,0x10
>> +%else
>> cglobal deblock_%1_luma_intra_8, 4,6,16,ARCH_X86_64*0x50-0x50
>> +
Hi,
On Thu, Feb 21, 2013 at 1:11 AM, Martin Storsjö wrote:
> +%if WIN64
> +cglobal deblock_%1_luma_intra_8, 4,6,16,0x10
> +%else
> cglobal deblock_%1_luma_intra_8, 4,6,16,ARCH_X86_64*0x50-0x50
> +%endif
I believe this doesn't need a stack pointer, so you can use -0x10.
Ronald
_
Hi,
On Feb 14, 2013 4:59 AM, "Diego Biurrun" wrote:
> On Wed, Feb 13, 2013 at 05:53:36PM -0500, Daniel Kang wrote:
> > @@ -56,6 +107,44 @@ PUT_PIXELS8_X2
> >
> > +%macro PUT_PIXELS8_X2_MMX 0-1
> > +%if %0 == 1
> > +cglobal put%1_pixels8_x2, 4,4
> > +%else
> > +cglobal put_pixels8_x2, 4,4
> > +%en
From: "Ronald S. Bultje"
In case of no-transform, integrate it with put_pixels4/8(). Intra PCM
is changed to not use h->mb anymore (saves a memcpy). The one during
update_thread_context() init is removed by removing the memcpy() that
clobbered it in the first place. Together, this m
Hi,
On Jan 28, 2013 11:44 PM, "Luca Barbato" wrote:
>
> On 28/01/13 20:32, Ronald S. Bultje wrote:
> > From: "Ronald S. Bultje"
> >
> > This allows objects to use just halfpel MC without depending on all
> > of dsputil. E.g. indeo3, interpla
From: "Ronald S. Bultje"
---
libavcodec/dsputil_template.c | 48 ---
1 file changed, 48 deletions(-)
diff --git a/libavcodec/dsputil_template.c b/libavcodec/dsputil_template.c
index f5811c1..a8e4e0b 100644
--- a/libavcodec/dsputil_templa
Hi,
On Mon, Jan 28, 2013 at 10:53 AM, Diego Biurrun wrote:
> ---
> libavcodec/x86/h264_qpel.c |2 +-
> libavcodec/x86/idct_sse2_xvid.c |2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
OK.
Ronald
___
libav-devel mailing list
liba
Hi,
On Mon, Jan 28, 2013 at 11:01 AM, Luca Barbato wrote:
> On 28/01/13 20:00, Diego Biurrun wrote:
>> On Thu, Jan 24, 2013 at 11:44:00AM +0100, Diego Biurrun wrote:
>>> On Wed, Jan 23, 2013 at 04:45:22PM +0100, Luca Barbato wrote:
From: Diego Biurrun
Signed-off-by: Luca Barbato
Hi,
On Sun, Jan 27, 2013 at 2:03 PM, Martin Storsjö wrote:
> From: Michael Niedermayer
>
> This makes the code actually work.
>
> ---
> Allegedly.
> ---
> libavcodec/x86/hpeldsp.asm |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/x86/hpeldsp.asm b/libavcod
Hi,
On Sun, Jan 27, 2013 at 1:25 PM, Derek Buitenhuis
wrote:
> From: Michael Niedermayer
>
> We have to make some symetric changes elsewhere as this increases
> the precission with which samples are stored.
>
> Signed-off-by: Michael Niedermayer
> Signed-off-by: Derek Buitenhuis
> ---
> libsw
Hi,
On Sun, Jan 27, 2013 at 1:25 PM, Derek Buitenhuis
wrote:
> From: Michael Niedermayer
>
> planar rgb formats do not use the table
>
> Signed-off-by: Michael Niedermayer
> Signed-off-by: Derek Buitenhuis
> ---
> libswscale/yuv2rgb.c |3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(
Hi,
On Sun, Jan 27, 2013 at 1:25 PM, Derek Buitenhuis
wrote:
> From: Michael Niedermayer
>
> This code path is not implemented and makes not much sense to implement
> either.
>
> Signed-off-by: Michael Niedermayer
> Signed-off-by: Derek Buitenhuis
> ---
> libswscale/utils.c |3 +++
> 1 fi
Hi,
On Sun, Jan 27, 2013 at 1:25 PM, Derek Buitenhuis
wrote:
> From: Michael Niedermayer
>
> The function doesnt support >8bit currently
>
> Signed-off-by: Michael Niedermayer
> Signed-off-by: Derek Buitenhuis
> ---
> libswscale/swscale_unscaled.c | 10 +-
> 1 file changed, 9 insert
Hi,
On Sun, Jan 27, 2013 at 1:25 PM, Derek Buitenhuis
wrote:
> From: Michael Niedermayer
>
> Signed-off-by: Michael Niedermayer
> Signed-off-by: Derek Buitenhuis
> ---
> libswscale/output.c | 28 +---
> libswscale/utils.c| 16
Hi,
On Sun, Jan 27, 2013 at 1:28 PM, Derek Buitenhuis
wrote:
> From: Daniel Kang
>
> Signed-off-by: Derek Buitenhuis
> ---
> libavcodec/x86/hpeldsp.asm |6 +++---
> libavcodec/x86/mpeg4qpel.asm |6 +++---
> 2 files changed, 6 insertions(+), 6 deletions(-)
OK.
Ronald
___
Hi,
On Sat, Jan 26, 2013 at 4:58 PM, Daniel Kang wrote:
> ---
> Make hpeldsp.asm
> ---
> libavcodec/x86/Makefile |2 +
> libavcodec/x86/dsputil.asm| 142 ++
> libavcodec/x86/dsputil_avg_template.c | 789 ++---
> libavcodec/x86/dsputil_m
Hi,
On Sat, Jan 26, 2013 at 10:01 AM, Daniel Kang wrote:
> On Sat, Jan 26, 2013 at 3:23 AM, Diego Biurrun wrote:
>> On Sat, Jan 26, 2013 at 12:32:16AM -0500, Daniel Kang wrote:
>>> --- a/libavcodec/x86/dsputil.asm
>>> +++ b/libavcodec/x86/dsputil.asm
>>> @@ -879,3 +884,984 @@ cglobal avg_pixels1
Hi,
On Sat, Jan 26, 2013 at 10:59 AM, Derek Buitenhuis
wrote:
> On 2013-01-26 1:57 PM, Diego Biurrun wrote:
>> Indentation is off, see the other function declarations in that file
>> for how it should look like.
>
> Right. Fixed locally.
>
> Will re-send after more review.
Functionally lgtm.
Ro
From: "Ronald S. Bultje"
---
libavcodec/x86/dsputil.asm| 188 --
libavcodec/x86/h264_qpel_8bit.asm | 169 ++
2 files changed, 169 insertions(+), 188 deletions(-)
diff --git a/libavcodec/x86/dsputil.asm b/liba
Hi,
On Thu, Jan 24, 2013 at 9:41 AM, Janne Grunau wrote:
> The code was copied from per cpu extension init function so the checks
> for supported extensions was overlooked.
> ---
> libavcodec/x86/h264_qpel.c | 139
> -
> 1 file changed, 74 insertions(
Hi,
On Thu, Jan 24, 2013 at 7:56 AM, Janne Grunau wrote:
> ---
> libavcodec/h264.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/libavcodec/h264.c b/libavcodec/h264.c
> index 9e9384b..38a6f5e 100644
> --- a/libavcodec/h264.c
> +++ b/libavcodec/h264.c
> @@ -2556,6 +2556,7 @@ static int
Hi,
On Wed, Jan 23, 2013 at 1:16 PM, Daniel Kang wrote:
> On Wed, Jan 23, 2013 at 4:14 PM, Daniel Kang wrote:
>> On Wed, Jan 23, 2013 at 12:36 PM, Ronald S. Bultje
>> wrote:
>>> Hi Daniel,
>>>
>>> On Tue, Jan 22, 2013 at 11:19 PM, Daniel Kang
>
Hi Daniel,
On Tue, Jan 22, 2013 at 11:19 PM, Daniel Kang wrote:
> @@ -1330,10 +1087,12 @@ static void OPNAME ## qpel8_mc12_ ## MMX(uint8_t
> *dst, uint8_t *src,\
> { \
> uint64_t half[8 + 9];
Hi,
On Wed, Jan 23, 2013 at 7:45 AM, Luca Barbato wrote:
> From: Diego Biurrun
>
> Signed-off-by: Luca Barbato
> ---
>
> Rebased after the move to int16_t.
>
> En passant, adding the missing memsets as pointed by Ronald.
>
> libavcodec/bfin/Makefile | 4 ++--
> libavcodec/bfin/dsputil_b
Hi,
On Tue, Jan 22, 2013 at 10:26 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Tue, Jan 22, 2013 at 8:49 PM, Ronald S. Bultje wrote:
>> Hi,
>>
>> On Fri, Jan 18, 2013 at 2:37 PM, Diego Biurrun wrote:
>> [..]
>>
>> This patch doesn't convert sh4
Hi,
On Tue, Jan 22, 2013 at 8:49 PM, Ronald S. Bultje wrote:
> Hi,
>
> On Fri, Jan 18, 2013 at 2:37 PM, Diego Biurrun wrote:
> [..]
>
> This patch doesn't convert sh4 and ppc. I can do ppc, I don't have
> access to a sh4 cross-compilation environment.
And arm
Hi,
On Fri, Jan 18, 2013 at 2:37 PM, Diego Biurrun wrote:
[..]
This patch doesn't convert sh4 and ppc. I can do ppc, I don't have
access to a sh4 cross-compilation environment.
Ronald
___
libav-devel mailing list
libav-devel@libav.org
https://lists.li
From: "Ronald S. Bultje"
The input is not guarenteed to be aligned.
---
libavcodec/vp3dsp.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/libavcodec/vp3dsp.c b/libavcodec/vp3dsp.c
index 1883099..0ce6b81 100644
--- a/libavcodec/vp3dsp.c
+++ b/libavcode
From: "Ronald S. Bultje"
---
libavcodec/alpha/dsputil_alpha.c | 8
libavcodec/arm/dsputil_init_neon.c | 8
libavcodec/dsputil.c | 2 +-
libavcodec/dsputil.h | 2 +-
libavcodec/sh4/dsputil_align.c | 8
libavc
Hi,
On Tue, Jan 22, 2013 at 7:11 AM, Ronald S. Bultje wrote:
> Hi,
>
> On Tue, Jan 22, 2013 at 4:00 AM, Diego Biurrun wrote:
>> On Mon, Jan 21, 2013 at 06:02:38PM -0800, Ronald S. Bultje wrote:
>>>
>>> --- a/libavcodec/dsputil.h
>>> +++ b/libavcodec/d
From: "Ronald S. Bultje"
---
libavcodec/dsputil.c | 2 +-
libavcodec/dsputil.h | 2 +-
libavcodec/vc1dec.c | 2 +-
libavcodec/x86/dsputil_mmx.c | 22 +++---
4 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/libavcodec/d
From: "Ronald S. Bultje"
These are never used.
---
libavcodec/dsputil.c | 31 -
libavcodec/dsputil_template.c | 64 ---
2 files changed, 54 insertions(+), 41 deletions(-)
diff --git a/libavcodec/dsputil.c b/
Hi,
On Tue, Jan 22, 2013 at 7:49 AM, Diego Biurrun wrote:
> On Tue, Jan 22, 2013 at 07:12:21AM -0800, Ronald S. Bultje wrote:
>> On Tue, Jan 22, 2013 at 1:53 AM, Diego Biurrun wrote:
>> > On Mon, Jan 21, 2013 at 06:18:22PM -0800, Ronald S. Bultje wrote:
>> >> O
Hi,
On Tue, Jan 22, 2013 at 8:59 AM, Diego Elio Pettenò
wrote:
> On 22/01/2013 03:02, Ronald S. Bultje wrote:
>>
>> This is never used.
>
> This has a strange effect on the other avg_pixels8_* functions, me and
> Luca have been looking into it today — it's not bad, b
Hi,
On Tue, Jan 22, 2013 at 1:53 AM, Diego Biurrun wrote:
> On Mon, Jan 21, 2013 at 06:18:22PM -0800, Ronald S. Bultje wrote:
>> On Mon, Jan 21, 2013 at 4:04 PM, Diego Biurrun wrote:
>> > It does not help as an abstraction and adds dsputil dependencies.
>>
>> I lik
Hi,
On Tue, Jan 22, 2013 at 4:00 AM, Diego Biurrun wrote:
> On Mon, Jan 21, 2013 at 06:02:38PM -0800, Ronald S. Bultje wrote:
>>
>> --- a/libavcodec/dsputil.h
>> +++ b/libavcodec/dsputil.h
>> @@ -281,15 +281,15 @@ typedef struct DSPContext {
>>
>> /**
Hi,
On Fri, Jan 18, 2013 at 4:24 PM, Loren Merritt wrote:
> On Fri, 18 Jan 2013, Vitor Sessak wrote:
>> On Wed, Jan 16, 2013 at 1:58 AM, Ronald S. Bultje wrote:
>>
>>> +INIT_XMM sse
>>> +cglobal vorbis_inverse_coupling, 3, 3, 6, mag, ang, block_size
>
From: "Ronald S. Bultje"
---
libavcodec/x86/Makefile | 1 +
libavcodec/x86/dsputil_mmx.c| 3 --
libavcodec/x86/dsputil_mmx.h| 2 -
libavcodec/x86/vorbisdsp.asm| 83 +
libavcodec/x86/vorbisdsp_i
From: "Ronald S. Bultje"
---
libavcodec/dsputil.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h
index b01c912..e4bbd46 100644
--- a/libavcodec/dsputil.h
+++ b/libavcodec/dsputil.h
@@ -277,7 +277,7 @@ type
Hi,
On Mon, Jan 21, 2013 at 4:04 PM, Diego Biurrun wrote:
> It does not help as an abstraction and adds dsputil dependencies.
I like the commit. I do want to add, though, that you're not actually
practically removing the dsputil dependency from a lot of files (at
build time), even though the dep
Hi,
On Mon, Jan 21, 2013 at 2:31 PM, Luca Barbato wrote:
> On 21/01/13 22:43, Luca Barbato wrote:
>> Actually the idct_dc_add not sure if we can safely mix and match them,
>> probably we can.
>
> I'd push tomorrow the updated patch, idct_dc_add is compatible from what
> I can see.
Please make su
Hi,
On Mon, Jan 21, 2013 at 1:13 PM, Luca Barbato wrote:
> On 21/01/13 20:38, Luca Barbato wrote:
>> +void ff_vp3dsp_init_bfin(VP3DSPContext *c, int flags)
>> +{
>> +c->idct_add = ff_bfin_vp3_idct_add;
>> +c->idct_put = ff_bfin_vp3_idct_put;
>
> c->idct missing.
vp3dsp has no idct.
Ro
From: "Ronald S. Bultje"
This is never used.
---
libavcodec/alpha/dsputil_alpha.c | 5 -
libavcodec/arm/dsputil_init_neon.c| 7 -
libavcodec/arm/dsputil_neon.S | 6 +-
libavcodec/dsputil.c | 1 -
libavcodec/dsputil.h
Hi,
On Jan 21, 2013 11:59 AM, "Diego Biurrun" wrote:
>
> On Mon, Jan 21, 2013 at 11:04:25AM -0800, Ronald S. Bultje wrote:
> > On Mon, Jan 21, 2013 at 10:58 AM, Diego Biurrun
wrote:
> > > On Mon, Jan 21, 2013 at 10:45:44AM -0800, Ronald S. Bultje wrote:
Hi,
On Mon, Jan 21, 2013 at 10:58 AM, Diego Biurrun wrote:
> On Mon, Jan 21, 2013 at 10:45:44AM -0800, Ronald S. Bultje wrote:
>> From: "Ronald S. Bultje"
>>
>> ---
>> configure | 7 ---
>> libavcodec/Makefile |
From: "Ronald S. Bultje"
The function is only used in VP3 and VP5, so no need to have it in
DSPContext.
---
libavcodec/dsputil.c | 1 -
libavcodec/dsputil.h | 2 --
libavcodec/dsputil_template.c | 4
libavcodec/vp3.c | 2 +-
libavcode
From: "Ronald S. Bultje"
The function is only used in VP3 and VP5, so no need to have it in
DSPContext.
---
libavcodec/dsputil.c | 1 -
libavcodec/dsputil.h | 2 --
libavcodec/dsputil_template.c | 4
libavcodec/vp3.c | 2 +-
libavcode
1 - 100 of 4240 matches
Mail list logo