<br><font size=2 face="sans-serif">I don't have a problem with how it is
done, but might need a bit more information from you on what it is you
are proposing, as I don't think I'm understanding it. One thing that would
need to not be compromised, though, is the ability to access the attachments
from other places also. The obvious example is the freetype, quartztext
and other text rendering modules who might want to load up ttf font attachments
and use those for the rendering of text. Would this change to implementation
prevent that?</font>
<br>
<br><font size=2 face="sans-serif">Bernie</font>
<br>
<br>
<br>
<table width=100%>
<tr valign=top>
<td width=40%><font size=1 face="sans-serif"><b>Laurent Aimar <fenrir@via.ecp.fr></b>
</font>
<br><font size=1 face="sans-serif">Sent by: vlc-devel-bounce@videolan.org</font>
<p><font size=1 face="sans-serif">31/05/2007 06:07 AM</font>
<table border>
<tr valign=top>
<td bgcolor=white>
<div align=center><font size=1 face="sans-serif">Please respond to<br>
vlc-devel@videolan.org</font></div></table>
<br>
<td width=59%>
<table width=100%>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">To</font></div>
<td valign=top><font size=1 face="sans-serif">vlc-devel@videolan.org</font>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">cc</font></div>
<td valign=top>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">Subject</font></div>
<td valign=top><font size=1 face="sans-serif">[vlc-devel] Re: [PATCH] Matroska
Attachments</font></table>
<br>
<table>
<tr valign=top>
<td>
<td></table>
<br></table>
<br>
<br>
<br><font size=2><tt>Hi,<br>
<br>
On Wed, May 30, 2007, Bernie Purcell wrote:<br>
> I don't know where else I could put it. It is a struct returned from
a<br>
> container demuxer (ie. other container formats besides MKV support<br>
> attachments) and it is a control message to the demuxer that returns
it.<br>
 I am no against adding such a functionnality but I do not like the way<br>
you retreive such attachment from subsdec.c.<br>
<br>
 It's a decoder and your are directly calling demux2_Control.<br>
Such thing is forbidden and may cause segfault. (demux2_Control can only<br>
be called by src/input/*.c, and decoder are in there own thread...)<br>
Beside you can have more than 1 demuxer per input.<br>
<br>
 I think the right way would be to directly send them to the subs decoder<br>
through es_format_t.<br>
 All we need would be to extend es_format_t.subs_format_t like this:<br>
                
 int i_attachment;<br>
                
 vlc_attachment_t **attachment;<br>
 And then mkv should just fill it on open.<br>
(Of course es_format_Init/Copy/Clean should handle this new fields)<br>
<br>
 I am not sure where vlc_attachment_t should be defined but vlc_es.h will<br>
be enougth to start with.<br>
<br>
 What do you think ?<br>
<br>
-- <br>
fenrir<br>
<br>
-- <br>
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/<br>
To unsubscribe, please read http://developers.videolan.org/lists.html<br>
</tt></font>
<br>