[libav-devel] [PATCH] lavc: add MMAL hardware decoder wrapper

2015-03-24 Thread wm4
Based on a patch by Rodger Combs. --- Changed the way the configure test works. Now it will first run a test without adding the insane RPI driver include and library paths (which are usually missing from the compiler's standard paths). Added a call to bcm_host_init() on init. Fixed some minor

Re: [libav-devel] [PATCH 1/3] log: Introduce a more verbose debug level

2015-03-24 Thread Vittorio Giovara
On Mon, Mar 16, 2015 at 8:57 AM, Vittorio Giovara vittorio.giov...@gmail.com wrote: And deprecate av_dlog macro. --- This seemed to be the general consensus on IRC. Up for discussion. Vittorio cmdutils.c | 1 + doc/APIchanges | 3 +++

Re: [libav-devel] [PATCH] Canopus HQ/HQA decoder

2015-03-24 Thread Derek Buitenhuis
On 3/24/2015 6:10 PM, Tristan Matthews wrote: Pointers first, int after please. Can you explain this in more details please? I think he means for packing, see: http://www.catb.org/esr/structure-packing/ It's actually for the far sillier reason: useless cosmetics. - Derek

Re: [libav-devel] [PATCH] Canopus HQ/HQA decoder

2015-03-24 Thread Derek Buitenhuis
On 3/24/2015 6:53 PM, Luca Barbato wrote: No, it is exactly to keep the structure packed a little as I explained to Vittorio privately. So, micro-optimizations / ricing? Diego (Flameeyes) explained in a number of blogpost why it is useful, sadly pahole is not available for mac. I can't much

Re: [libav-devel] [PATCH 2/2] avcodec: add libdcadec decoder

2015-03-24 Thread Anton Khirnov
Quoting Hendrik Leppkes (2015-03-23 12:45:19) --- configure | 4 + libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 1 + libavcodec/libdcadec.c | 197 + 4 files changed, 203 insertions(+) create mode 100644

Re: [libav-devel] [PATCH] Canopus HQ/HQA decoder

2015-03-24 Thread Martin Storsjö
On Tue, 24 Mar 2015, Derek Buitenhuis wrote: On 3/24/2015 6:53 PM, Luca Barbato wrote: No, it is exactly to keep the structure packed a little as I explained to Vittorio privately. So, micro-optimizations / ricing? For some cases, it does matter. I don't agree it matters in any

Re: [libav-devel] [PATCH 2/2] avcodec: add libdcadec decoder

2015-03-24 Thread Anton Khirnov
Quoting Hendrik Leppkes (2015-03-24 09:14:20) On Tue, Mar 24, 2015 at 7:35 AM, Anton Khirnov an...@khirnov.net wrote: Quoting Hendrik Leppkes (2015-03-23 12:45:19) diff --git a/libavcodec/libdcadec.c b/libavcodec/libdcadec.c new file mode 100644 index 000..7ca8942 --- /dev/null

Re: [libav-devel] [PATCH] Canopus HQ/HQA decoder

2015-03-24 Thread Tristan Matthews
On Tue, Mar 24, 2015 at 3:03 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 3/24/2015 6:53 PM, Luca Barbato wrote: No, it is exactly to keep the structure packed a little as I explained to Vittorio privately. So, micro-optimizations / ricing? Diego (Flameeyes) explained in a

Re: [libav-devel] [PATCH] Canopus HQ/HQA decoder

2015-03-24 Thread Tristan Matthews
On Tue, Mar 24, 2015 at 4:09 PM, Tristan Matthews le.business...@gmail.com wrote: On Tue, Mar 24, 2015 at 3:03 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 3/24/2015 6:53 PM, Luca Barbato wrote: No, it is exactly to keep the structure packed a little as I explained to Vittorio

Re: [libav-devel] [PATCH] Canopus HQ/HQA decoder

2015-03-24 Thread James Almer
On 22/03/15 12:49 PM, Vittorio Giovara wrote: +// AAN IDCT If this isn't already in the tree somewhere and it's generic enough that it can be reused, then it should be shared like faanidct and added to idctdsp. And if it's HQ/HQA specific, it still could be split into a new hqdsp context for

Re: [libav-devel] [PATCH] Canopus HQ/HQA decoder

2015-03-24 Thread Tristan Matthews
On Mon, Mar 23, 2015 at 8:40 AM, Vittorio Giovara vittorio.giov...@gmail.com wrote: On Sun, Mar 22, 2015 at 7:36 PM, Luca Barbato lu_z...@gentoo.org wrote: +typedef struct HQContext { +AVCodecContext *avctx; + +VLC hq_ac_vlc; +VLC hqa_cbp_vlc; +DECLARE_ALIGNED(16, int16_t,

Re: [libav-devel] [PATCH] lavc: add Intel libmfx-based H.264 encoder

2015-03-24 Thread Luca Barbato
On 23/03/15 15:32, Anton Khirnov wrote: --- Changelog | 1 + configure | 4 + libavcodec/Makefile | 2 + libavcodec/allcodecs.c| 1 + libavcodec/qsv_internal.h | 13 ++ libavcodec/qsvdec.h

Re: [libav-devel] [PATCH 2/2] avcodec: add libdcadec decoder

2015-03-24 Thread Luca Barbato
On 23/03/15 19:46, Nicolas George wrote: I will trim the discussion somewhat. Please do not trim the important part: On 23/03/15 19:31, Luca Barbato wrote: On 23/03/15 18:38, Nicolas George wrote: There is something you still do not seem to realize: if your stack is corrupted, abort() is just

Re: [libav-devel] [PATCH 2/2] avcodec: add libdcadec decoder

2015-03-24 Thread Hendrik Leppkes
On Tue, Mar 24, 2015 at 7:35 AM, Anton Khirnov an...@khirnov.net wrote: Quoting Hendrik Leppkes (2015-03-23 12:45:19) diff --git a/libavcodec/libdcadec.c b/libavcodec/libdcadec.c new file mode 100644 index 000..7ca8942 --- /dev/null +++ b/libavcodec/libdcadec.c @@ -0,0 +1,197 @@ +/* +