[vlc-devel] [PATCH 1/2] daala: add decoder support
Tristan Matthews
le.businessman at gmail.com
Fri Aug 22 09:36:40 CEST 2014
On Thu, Aug 21, 2014 at 4:42 AM, Julian Scheel <julian at jusst.de> wrote:
> Am 21.08.2014 um 09:14 schrieb Rafaël Carré:
>
>> On 08/21/14 08:40, Rafaël Carré wrote:
>>>
>>> On 08/21/14 08:38, Tristan Matthews wrote:
>>>>
>>>> ---
>>>> configure.ac | 5 +
>>>> include/vlc_fourcc.h | 1 +
>>>> modules/codec/Makefile.am | 7 +
>>>> modules/codec/daala.c | 506
>>>> ++++++++++++++++++++++++++++++++++++++++++++++
>>>> 4 files changed, 519 insertions(+)
>>>> create mode 100644 modules/codec/daala.c
>>>>
>>>> diff --git a/configure.ac b/configure.ac
>>>> index 5e3bb8c..26c0011 100644
>>>> --- a/configure.ac
>>>> +++ b/configure.ac
>>>> @@ -2782,6 +2782,11 @@ dnl
>>>> PKG_ENABLE_MODULES_VLC([THEORA], [], [ogg theoradec >= 1.0 theoraenc],
>>>> [experimental theora codec], [auto])
>>>>
>>>> dnl
>>>> +dnl Daala decoder plugin
>>>> +dnl
>>>> +PKG_ENABLE_MODULES_VLC([DAALA], [], [ogg daaladec], [experimental daala
>>>> codec], [auto])
>>>
>>>
>>> I don't know if it's a good idea to support a work in progress codec
>>
>>
>> Well daala people do not seem opposed to it, although maybe the plugin
>> should be default disabled, unless absence of current packaging in distros
>> is a high enough deterrent for people who want to use it actively.
>
>
> Are there any statements about API stability yet? Is it likely to break once
> in a week or is it mature enough to assume this code will compile most of
> the time without flaws?
The basic decoder API (which we're using) was inherited from Theora
and is not going to change that quickly (see
https://mf4.xiph.org/jenkins/job/daala-autotools/ws/doc/html/group__decfuncs.html)
so I think we should be ok.
>
>
>>> +
>>> +dnl
>>> dnl schroedinger decoder plugin (for dirac format video)
>>> dnl
>>> PKG_ENABLE_MODULES_VLC([SCHROEDINGER], [], [schroedinger-1.0 >=
>>> 1.0.10], [dirac decoder and encoder using schroedinger], [auto])
>>> diff --git a/include/vlc_fourcc.h b/include/vlc_fourcc.h
>>> index d00adf4..d8d8324 100644
>>> --- a/include/vlc_fourcc.h
>>> +++ b/include/vlc_fourcc.h
>>> @@ -50,6 +50,7 @@
>>> #define VLC_CODEC_WMVP VLC_FOURCC('W','M','V','P')
>>> #define VLC_CODEC_WMVP2 VLC_FOURCC('W','V','P','2')
>>> #define VLC_CODEC_VC1 VLC_FOURCC('V','C','-','1')
>>> +#define VLC_CODEC_DAALA VLC_FOURCC('d','a','a','l')
>>> #define VLC_CODEC_THEORA VLC_FOURCC('t','h','e','o')
>>> #define VLC_CODEC_TARKIN VLC_FOURCC('t','a','r','k')
>>> #define VLC_CODEC_DIRAC VLC_FOURCC('d','r','a','c')
>>> diff --git a/modules/codec/Makefile.am b/modules/codec/Makefile.am
>>> index 25968d7..c351929 100644
>>> --- a/modules/codec/Makefile.am
>>> +++ b/modules/codec/Makefile.am
>>> @@ -238,6 +238,13 @@ libtheora_plugin_la_LIBADD = $(THEORA_LIBS)
>>> EXTRA_LTLIBRARIES += libtheora_plugin.la
>>> codec_LTLIBRARIES += $(LTLIBtheora)
>>>
>>> +libdaala_plugin_la_SOURCES = codec/daala.c
>>> +libdaala_plugin_la_CFLAGS = $(AM_CFLAGS) $(DAALA_CFLAGS)
>>> +libdaala_plugin_la_LDFLAGS = $(AM_LDFLAGS) -rpath '$(codecdir)'
>>> +libdaala_plugin_la_LIBADD = $(DAALA_LIBS)
>>> +EXTRA_LTLIBRARIES += libdaala_plugin.la
>>> +codec_LTLIBRARIES += $(LTLIBdaala)
>>> +
>>> libtremor_plugin_la_SOURCES = codec/vorbis.c
>>> libtremor_plugin_la_CPPFLAGS = $(AM_CPPFLAGS) -DMODULE_NAME_IS_tremor
>>> libtremor_plugin_la_LDFLAGS = $(AM_LDFLAGS) -rpath '$(codecdir)'
>>> diff --git a/modules/codec/daala.c b/modules/codec/daala.c
>>> new file mode 100644
>>> index 0000000..6cbc3d8
>>> --- /dev/null
>>> +++ b/modules/codec/daala.c
>>> @@ -0,0 +1,506 @@
>>>
>>> +/*****************************************************************************
>>> + * daala.c: daala codec module making use of libdaala.
>>> +
>>> *****************************************************************************
>>> + * Copyright (C) 2014 VLC authors and VideoLAN
>>> + *
>>> + * Authors: Tristan Matthews <le.businessman at gmail.com>
>>> + * * Based on theora.c by: Gildas Bazin <gbazin at videolan.org>
>>> + *
>>> + * This program is free software; you can redistribute it and/or modify
>>> it
>>> + * under the terms of the GNU Lesser General Public License as published
>>> by
>>> + * the Free Software Foundation; either version 2.1 of the License, or
>>> + * (at your option) any later version.
>>> + *
>>> + * This program is distributed in the hope that it will be useful,
>>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>>> + * GNU Lesser General Public License for more details.
>>> + *
>>> + * You should have received a copy of the GNU Lesser General Public
>>> License
>>> + * along with this program; if not, write to the Free Software
>>> Foundation,
>>> + * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
>>> +
>>> *****************************************************************************/
>>> +
>>>
>>> +/*****************************************************************************
>>> + * Preamble
>>> +
>>> *****************************************************************************/
>>> +#ifdef HAVE_CONFIG_H
>>> +# include "config.h"
>>> +#endif
>>> +
>>> +#include <vlc_common.h>
>>> +#include <vlc_plugin.h>
>>> +#include <vlc_codec.h>
>>> +#include <vlc_input.h>
>>> +#include "../demux/xiph.h"
>>> +
>>> +#include <ogg/ogg.h>
>>
>>
>> Should it depend on ogg? Is it explicit in the .pc file?
>>
>>> +#include <daala/codec.h>
>>> +#include <daala/daaladec.h>
>>> +
>>> +#include <limits.h>
>>> +
>>>
>>> +/*****************************************************************************
>>> + * decoder_sys_t : daala decoder descriptor
>>> +
>>> *****************************************************************************/
>>> +struct decoder_sys_t
>>> +{
>>> + /* Module mode */
>>> + bool b_packetizer;
>>> +
>>> + /*
>>> + * Input properties
>>> + */
>>> + bool b_has_headers;
>>> +
>>> + /*
>>> + * Daala properties
>>> + */
>>> + daala_info di; /* daala bitstream settings */
>>> + daala_comment dc; /* daala comment information */
>>> + daala_dec_ctx *dcx; /* daala decoder context */
>>> +
>>> + /*
>>> + * Decoding properties
>>> + */
>>> + bool b_decoded_first_keyframe;
>>> +
>>> + /*
>>> + * Common properties
>>> + */
>>> + mtime_t i_pts;
>>> +};
>>> +
>>>
>>> +/*****************************************************************************
>>> + * Local prototypes
>>> +
>>> *****************************************************************************/
>>> +static int OpenDecoder ( vlc_object_t * );
>>> +static int OpenPacketizer( vlc_object_t * );
>>> +static void CloseDecoder ( vlc_object_t * );
>>> +
>>> +static void *DecodeBlock ( decoder_t *, block_t ** );
>>> +static int ProcessHeaders( decoder_t * );
>>> +static void *ProcessPacket ( decoder_t *, ogg_packet *, block_t ** );
>>> +
>>> +static picture_t *DecodePacket( decoder_t *, ogg_packet * );
>>> +
>>> +static void ParseDaalaComments( decoder_t * );
>>> +static void daala_CopyPicture( picture_t *, od_img * );
>>> +
>>> +vlc_module_begin ()
>>> + set_category( CAT_INPUT )
>>> + set_subcategory( SUBCAT_INPUT_VCODEC )
>>> + set_shortname( "Daala" )
>>> + set_description( N_("Daala video decoder") )
>>> + set_capability( "decoder", 100 )
>>> + set_callbacks( OpenDecoder, CloseDecoder )
>>> + add_shortcut( "daala" )
>>> +# define DEC_CFG_PREFIX "daala-"
>>
>>
>> unused
>>
>>> +
>>> + add_submodule ()
>>> + set_description( N_("Daala video packetizer") )
>>> + set_capability( "packetizer", 100 )
>>> + set_callbacks( OpenPacketizer, CloseDecoder )
>>> + add_shortcut( "daala" )
>>> +
>>> +vlc_module_end ()
>>> +
>>>
>>> +/*****************************************************************************
>>> + * OpenDecoder: probe the decoder and return score
>>> +
>>> *****************************************************************************/
>>> +static int OpenDecoder( vlc_object_t *p_this )
>>> +{
>>> + decoder_t *p_dec = (decoder_t*)p_this;
>>> + decoder_sys_t *p_sys;
>>> +
>>> + if( p_dec->fmt_in.i_codec != VLC_CODEC_DAALA )
>>> + {
>>> + return VLC_EGENERIC;
>>> + }
>>> +
>>> + /* Allocate the memory needed to store the decoder's structure */
>>> + if( ( p_dec->p_sys = p_sys = malloc(sizeof(*p_sys)) ) == NULL )
>>> + return VLC_ENOMEM;
>>
>>
>> I would split those 3 statements
>>
>>> + p_dec->p_sys->b_packetizer = false;
>>> + p_sys->b_has_headers = false;
>>> + p_sys->i_pts = VLC_TS_INVALID;
>>> + p_sys->b_decoded_first_keyframe = false;
>>> + p_sys->dcx = NULL;
>>> +
>>> + /* Set output properties */
>>> + p_dec->fmt_out.i_cat = VIDEO_ES;
>>> + p_dec->fmt_out.i_codec = VLC_CODEC_I420;
>>> +
>>> + /* Set callbacks */
>>> + p_dec->pf_decode_video = DecodeBlock;
>>> + p_dec->pf_packetize = DecodeBlock;
>>> +
>>> + /* Init supporting Daala structures needed in header parsing */
>>> + daala_comment_init( &p_sys->dc );
>>> + daala_info_init( &p_sys->di );
>>> +
>>> + return VLC_SUCCESS;
>>> +
>>> +}
>>> +
>>> +static int OpenPacketizer( vlc_object_t *p_this )
>>> +{
>>> + decoder_t *p_dec = (decoder_t*)p_this;
>>> +
>>> + int i_ret = OpenDecoder( p_this );
>>> +
>>> + if( i_ret == VLC_SUCCESS )
>>> + {
>>> + p_dec->p_sys->b_packetizer = true;
>>> + p_dec->fmt_out.i_codec = VLC_CODEC_DAALA;
>>> +
>>> + }
>>> +
>>> + return i_ret;
>>> +}
>>> +
>>>
>>> +/****************************************************************************
>>> + * DecodeBlock: the whole thing
>>> +
>>> ****************************************************************************
>>> + * This function must be fed with ogg packets.
>>> +
>>> ****************************************************************************/
>>> +static void *DecodeBlock( decoder_t *p_dec, block_t **pp_block )
>>> +{
>>> + decoder_sys_t *p_sys = p_dec->p_sys;
>>> + block_t *p_block;
>>> + ogg_packet oggpacket;
>>> +
>>> + if( !pp_block || !*pp_block ) return NULL;
>>> +
>>> + p_block = *pp_block;
>>> +
>>> + /* Block to Ogg packet */
>>> +
>>> + oggpacket.packet = p_block->p_buffer;
>>> + oggpacket.bytes = p_block->i_buffer;
>>> + oggpacket.granulepos = p_block->i_dts;
>>> + oggpacket.b_o_s = 0;
>>> + oggpacket.e_o_s = 0;
>>> + oggpacket.packetno = 0;
>>> +
>>> + /* Check for headers */
>>> + if( !p_sys->b_has_headers )
>>> + {
>>> + if( ProcessHeaders( p_dec ) )
>>> + {
>>> + block_Release( p_block );
>>> + return NULL;
>>> + }
>>> + p_sys->b_has_headers = true;
>>> + }
>>> +
>>> + return ProcessPacket( p_dec, &oggpacket, pp_block );
>>> +}
>>> +
>>>
>>> +/*****************************************************************************
>>> + * ProcessHeaders: process Daala headers.
>>> +
>>> *****************************************************************************/
>>> +static int ProcessHeaders( decoder_t *p_dec )
>>> +{
>>> + decoder_sys_t *p_sys = p_dec->p_sys;
>>> + ogg_packet oggpacket;
>>> + daala_setup_info *ds = NULL; /* daala setup information */
>>> +
>>> + unsigned pi_size[XIPH_MAX_HEADER_COUNT];
>>> + void *pp_data[XIPH_MAX_HEADER_COUNT];
>>> + unsigned i_count;
>>> + if( xiph_SplitHeaders( pi_size, pp_data, &i_count,
>>> + p_dec->fmt_in.i_extra, p_dec->fmt_in.p_extra)
>>> )
>>> + return VLC_EGENERIC;
>>> + if( i_count < 3 )
>>> + return VLC_EGENERIC;
>>> +
>>> + oggpacket.granulepos = -1;
>>> + oggpacket.e_o_s = 0;
>>> + oggpacket.packetno = 0;
>>> +
>>> + /* Take care of the initial info header */
>>> + oggpacket.b_o_s = 1; /* yes this actually is a b_o_s packet :) */
>>> + oggpacket.bytes = pi_size[0];
>>> + oggpacket.packet = pp_data[0];
>>> +
>>> + if( daala_decode_header_in( &p_sys->di, &p_sys->dc, &ds, &oggpacket
>>> ) < 0 )
>>> + {
>>> + msg_Err( p_dec, "this bitstream does not contain Daala video
>>> data" );
>>> + goto error;
>>> + }
>>> +
>>> + /* Set output properties */
>>> + if( !p_sys->b_packetizer )
>>> + {
>>> + if( p_sys->di.plane_info[0].xdec == 0 &&
>>> p_sys->di.plane_info[0].ydec == 0 &&
>>> + p_sys->di.plane_info[1].xdec == 1 &&
>>> p_sys->di.plane_info[1].ydec == 1 &&
>>> + p_sys->di.plane_info[2].xdec == 1 &&
>>> p_sys->di.plane_info[2].ydec == 1 )
>>> + {
>>> + p_dec->fmt_out.i_codec = VLC_CODEC_I420;
>>> + }
>>> + else if( p_sys->di.plane_info[0].xdec == 0 &&
>>> p_sys->di.plane_info[0].ydec == 0 &&
>>> + p_sys->di.plane_info[1].xdec == 0 &&
>>> p_sys->di.plane_info[1].ydec == 0 &&
>>> + p_sys->di.plane_info[2].xdec == 0 &&
>>> p_sys->di.plane_info[2].ydec == 0 )
>>> + {
>>> + p_dec->fmt_out.i_codec = VLC_CODEC_I444;
>>> + }
>>> + else
>>> + {
>>> + msg_Err( p_dec, "unknown chroma in daala sample" );
>>> + }
>>> + }
>>> +
>>> + p_dec->fmt_out.video.i_width = p_sys->di.pic_width;
>>> + p_dec->fmt_out.video.i_height = p_sys->di.pic_height;
>>> + if( p_sys->di.pic_width && p_sys->di.pic_height )
>>> + {
>>> + p_dec->fmt_out.video.i_visible_width = p_sys->di.pic_width;
>>> + p_dec->fmt_out.video.i_visible_height = p_sys->di.pic_height;
>>> + }
>>> +
>>> +
>>> + if( p_sys->di.pixel_aspect_denominator &&
>>> p_sys->di.pixel_aspect_numerator )
>>> + {
>>> + p_dec->fmt_out.video.i_sar_num =
>>> p_sys->di.pixel_aspect_numerator;
>>> + p_dec->fmt_out.video.i_sar_den =
>>> p_sys->di.pixel_aspect_denominator;
>>> + }
>>> + else
>>> + {
>>> + p_dec->fmt_out.video.i_sar_num = 1;
>>> + p_dec->fmt_out.video.i_sar_den = 1;
>>> + }
>>> +
>>> + if( p_sys->di.timebase_numerator > 0 &&
>>> p_sys->di.timebase_denominator > 0 )
>>> + {
>>> + p_dec->fmt_out.video.i_frame_rate =
>>> p_sys->di.timebase_numerator;
>>> + p_dec->fmt_out.video.i_frame_rate_base =
>>> p_sys->di.timebase_denominator;
>>> + }
>>> +
>>> + msg_Dbg( p_dec, "%dx%d %.02f fps video, frame content ",
>>> + p_sys->di.pic_width, p_sys->di.pic_height,
>>> +
>>> (double)p_sys->di.timebase_numerator/p_sys->di.timebase_denominator );
>>> +
>>> + /* The next packet in order is the comments header */
>>> + oggpacket.b_o_s = 0;
>>> + oggpacket.bytes = pi_size[1];
>>> + oggpacket.packet = pp_data[1];
>>> +
>>> + if( daala_decode_header_in( &p_sys->di, &p_sys->dc, &ds, &oggpacket
>>> ) < 0 )
>>> + {
>>> + msg_Err( p_dec, "Daala comment header is corrupted" );
>>> + goto error;
>>> + }
>>> +
>>> + ParseDaalaComments( p_dec );
>>> +
>>> + /* The next packet in order is the setup header
>>> + * We need to watch out that this packet is not missing as a
>>> + * missing or corrupted header is fatal. */
>>> + oggpacket.b_o_s = 0;
>>> + oggpacket.bytes = pi_size[2];
>>> + oggpacket.packet = pp_data[2];
>>> + if( daala_decode_header_in( &p_sys->di, &p_sys->dc, &ds, &oggpacket
>>> ) < 0 )
>>> + {
>>> + msg_Err( p_dec, "Daala setup header is corrupted" );
>>> + goto error;
>>> + }
>>> +
>>> + if( !p_sys->b_packetizer )
>>> + {
>>> + /* We have all the headers, initialize decoder */
>>> + if ( ( p_sys->dcx = daala_decode_alloc( &p_sys->di, ds ) ) ==
>>> NULL )
>>> + {
>>> +
>>> + msg_Err( p_dec, "Could not allocate Daala decoder" );
>>> + goto error;
>>> + }
>>> + }
>>> + else
>>> + {
>>> + p_dec->fmt_out.i_extra = p_dec->fmt_in.i_extra;
>>> + p_dec->fmt_out.p_extra = xrealloc( p_dec->fmt_out.p_extra,
>>> + p_dec->fmt_out.i_extra );
>>> + memcpy( p_dec->fmt_out.p_extra,
>>> + p_dec->fmt_in.p_extra, p_dec->fmt_out.i_extra );
>>> + }
>>> +
>>> +
>>> + /* Clean up the decoder setup info... we're done with it */
>>> + daala_setup_free( ds );
>>> +
>>> + return VLC_SUCCESS;
>>
>>
>> Remove those last 2 statements and use ret = VLC_SUCCESS ?
>>
>>> +
>>> +error:
>>> + /* Clean up the decoder setup info... we're done with it */
>>> + daala_setup_free( ds );
>>> + return VLC_EGENERIC;
>>> +}
>>> +
>>>
>>> +/*****************************************************************************
>>> + * ProcessPacket: processes a daala packet.
>>> +
>>> *****************************************************************************/
>>> +static void *ProcessPacket( decoder_t *p_dec, ogg_packet *p_oggpacket,
>>> + block_t **pp_block )
>>> +{
>>> + decoder_sys_t *p_sys = p_dec->p_sys;
>>> + block_t *p_block = *pp_block;
>>> + void *p_buf;
>>> +
>>> + if( (
>>> p_block->i_flags&(BLOCK_FLAG_DISCONTINUITY|BLOCK_FLAG_CORRUPTED) ) != 0 )
>>> + {
>>> + /* Don't send the the first packet after a discontinuity to
>>> + * daala_decode, otherwise we get purple/green display artifacts
>>> + * appearing in the video output */
>>> + block_Release(p_block);
>>> + return NULL;
>>> + }
>>> +
>>> + /* Date management */
>>> + if( p_block->i_pts > VLC_TS_INVALID && p_block->i_pts !=
>>> p_sys->i_pts )
>>> + {
>>> + p_sys->i_pts = p_block->i_pts;
>>> + }
>>> +
>>> + *pp_block = NULL; /* To avoid being fed the same packet again */
>>> +
>>> + if( p_sys->b_packetizer )
>>> + {
>>> + /* Date management */
>>> + p_block->i_dts = p_block->i_pts = p_sys->i_pts;
>>> +
>>> + p_block->i_length = p_sys->i_pts - p_block->i_pts;
>>> +
>>> + p_buf = p_block;
>>> + }
>>> + else
>>> + {
>>> + p_buf = DecodePacket( p_dec, p_oggpacket );
>>> + block_Release( p_block );
>>> + }
>>> +
>>> + /* Date management */
>>> + p_sys->i_pts += ( CLOCK_FREQ * p_sys->di.timebase_denominator /
>>> + p_sys->di.timebase_numerator ); /* 1 frame per
>>> packet */
>>> +
>>> + return p_buf;
>>> +}
>>> +
>>>
>>> +/*****************************************************************************
>>> + * DecodePacket: decodes a Daala packet.
>>> +
>>> *****************************************************************************/
>>> +static picture_t *DecodePacket( decoder_t *p_dec, ogg_packet
>>> *p_oggpacket )
>>> +{
>>> + decoder_sys_t *p_sys = p_dec->p_sys;
>>> + picture_t *p_pic;
>>> + od_img ycbcr;
>>> +
>>> + if (daala_decode_packet_in( p_sys->dcx, &ycbcr, p_oggpacket ) < 0)
>>> + return NULL; /* bad packet */
>>> +
>>> + /* Check for keyframe */
>>> + if( daala_packet_iskeyframe( p_oggpacket->packet, p_oggpacket->bytes
>>> ) )
>>> + p_sys->b_decoded_first_keyframe = true;
>>> +
>>> + /* If we haven't seen a single keyframe yet, don't let Daala decode
>>> + * anything, otherwise we'll get display artifacts. (This is
>>> impossible
>>> + * in the general case, but can happen if e.g. we play a network
>>> stream
>>> + * using a timed URL, such that the server doesn't start the video
>>> with a
>>> + * keyframe). */
>>> + if( !p_sys->b_decoded_first_keyframe )
>>> + return NULL; /* Wait until we've decoded the first keyframe */
>>
>>
>> Isn't it true for most video codecs anyway?
>
>
> Shouldn't this better be dealt with marking frames before first keyframe as
> BLOCK_FLAG_PREROLL?
> VLC core will take care of not displaying the frames then.
Will do.
>
>
>>> + /* Get a new picture */
>>> + p_pic = decoder_NewPicture( p_dec );
>>> + if( !p_pic ) return NULL;
>>> +
>>> + daala_CopyPicture( p_pic, &ycbcr );
>>> +
>>> + p_pic->date = p_sys->i_pts;
>>> +
>>> + return p_pic;
>>> +}
>>> +
>>>
>>> +/*****************************************************************************
>>> + * ParseDaalaComments:
>>> +
>>> *****************************************************************************/
>>> +static void ParseDaalaComments( decoder_t *p_dec )
>>> +{
>>> + char *psz_name, *psz_value, *psz_comment;
>>> + int i = 0;
>>> + /* Regarding the daala_comment structure: */
>>> +
>>> + /* The metadata is stored as a series of (tag, value) pairs, in
>>> + length-encoded string vectors. The first occurrence of the '='
>>> + character delimits the tag and value. A particular tag may
>>> + occur more than once, and order is significant. The character
>>> + set encoding for the strings is always UTF-8, but the tag names
>>> + are limited to ASCII, and treated as case-insensitive. See the
>>> + Daala specification, Section 6.3.3 for details. */
>>> +
>>> + /* In filling in this structure, daala_decode_header_in() will
>>> + null-terminate the user_comment strings for safety. However,
>>> + the bitstream format itself treats them as 8-bit clean vectors,
>>> + possibly containing null characters, and so the length array
>>> + should be treated as their authoritative length. */
>>> + while ( i < p_dec->p_sys->dc.comments )
>>
>>
>> Use for() ?
>>
>>> + {
>>> + int clen = p_dec->p_sys->dc.comment_lengths[i];
>>> + if ( clen <= 0 || clen >= INT_MAX ) { i++; continue; }
>>> + psz_comment = (char *)malloc( clen + 1 );
>>> + if( !psz_comment )
>>> + break;
>>> + memcpy( (void*)psz_comment,
>>> (void*)p_dec->p_sys->dc.user_comments[i], clen + 1 );
>>
>>
>> psz_comment[clen] = '\0';
>>
>>> + psz_name = psz_comment;
>>> + psz_value = strchr( psz_comment, '=' );
>>> + if( psz_value )
>>> + {
>>> + *psz_value = '\0';
>>> + psz_value++;
>>> +
>>> + if( !p_dec->p_description )
>>> + p_dec->p_description = vlc_meta_New();
>>> + /* TODO: Since psz_value can contain NULLs see if there is
>>> an
>>> + * instance where we need to preserve the full length of
>>> this string */
>>> + if( p_dec->p_description )
>>> + vlc_meta_AddExtra( p_dec->p_description, psz_name,
>>> psz_value );
>>> + }
>>> + free( psz_comment );
>>> + i++;
>>> + }
>>> +}
>>> +
>>>
>>> +/*****************************************************************************
>>> + * CloseDecoder: daala decoder destruction
>>> +
>>> *****************************************************************************/
>>> +static void CloseDecoder( vlc_object_t *p_this )
>>> +{
>>> + decoder_t *p_dec = (decoder_t *)p_this;
>>> + decoder_sys_t *p_sys = p_dec->p_sys;
>>> +
>>> + daala_info_clear(&p_sys->di);
>>> + daala_comment_clear(&p_sys->dc);
>>> + daala_decode_free(p_sys->dcx);
>>
>>
>>> + p_sys->dcx = NULL;
>>
>>
>> Not needed
>>
>>> + free( p_sys );
>>> +}
>>> +
>>>
>>> +/*****************************************************************************
>>> + * daala_CopyPicture: copy a picture from daala internal buffers to a
>>> + * picture_t structure.
>>> +
>>> *****************************************************************************/
>>> +static void daala_CopyPicture( picture_t *p_pic,
>>> + od_img *ycbcr )
>>> +{
>>> + const int i_planes = p_pic->i_planes < 3 ? p_pic->i_planes : 3;
>>> + for( int i_plane = 0; i_plane < i_planes; i_plane++ )
>>> + {
>>> + const int i_total_lines = __MIN(p_pic->p[i_plane].i_lines,
>>> + ycbcr->height >> ycbcr->planes[i_plane].ydec);
>>> + uint8_t *p_dst = p_pic->p[i_plane].p_pixels;
>>> + uint8_t *p_src = ycbcr->planes[i_plane].data;
>>> + const int i_dst_stride = p_pic->p[i_plane].i_pitch;
>>> + const int i_src_stride = ycbcr->planes[i_plane].ystride;
>>> + for( int i_line = 0; i_line < i_total_lines; i_line++ )
>>> + {
>>> + memcpy( p_dst, p_src, i_src_stride );
>>> + p_src += i_src_stride;
>>> + p_dst += i_dst_stride;
>>> + }
>>> + }
>>> +}
>>> +
>>>
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> https://mailman.videolan.org/listinfo/vlc-devel
>>
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
More information about the vlc-devel
mailing list