<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2016-05-24 23:51 GMT+02:00 Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On 24 May, Emeric Grange wrote :<br>
<span class="">> +static void LoadChapterGoPro( demux_t *p_demux, MP4_Box_t *p_hmmt )<br>
> +{<br>
> +    demux_sys_t *p_sys = p_demux->p_sys;<br>
> +<br>
> +    p_sys->p_title = vlc_input_title_New();<br>
> +    for( unsigned i = 0; i < BOXDATA(p_hmmt)->i_chapter_count; i++ )<br>
> +    {<br>
> +        seekpoint_t *s = vlc_seekpoint_New();<br>
> +        if (s)<br>
> +        {<br>
> +            asprintf( &s->psz_name, "HiLight tag #%u", i+1 );<br>
<br>
</span>Shouldn't you check asprintf return before calling EnsureUTF8?<br></blockquote><div><br></div><div>No pb I will. By the way I was wondering do I really need to call EnsureUTF8() if I know the text is ascii compliant? (short enough, no special chars, etc)</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Rest is good :)<br>
<br>
With my kindest regards,<br>
<span class=""><font color="#888888"><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" rel="noreferrer" target="_blank">http://www.jbkempf.com/</a> - <a href="tel:%2B33%20672%20704%20734" value="+33672704734">+33 672 704 734</a><br>
Sent from my Electronic Device<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</font></span></blockquote></div><br></div></div>