<br><font size=2 face="sans-serif">Hi Fenrir,</font>
<br>
<br><font size=2 face="sans-serif">The whitespace removal code does have
similar code in freetype.c. It has been submitted as part of a much larger
set of corrections that need to be made to the freetype module, and just
hasn't been included here.</font>
<br><font size=2 face="sans-serif">I'll work on changing the memcpy to
memmove and strlcpy instead of strncpy but won't have a chance till tomorrow.</font>
<br>
<br><font size=2 face="sans-serif">Bernie</font>
<br>
<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-bounces@videolan.org</font>
<p><font size=1 face="sans-serif">30/06/2007 09:49 AM</font>
<table border>
<tr valign=top>
<td bgcolor=white>
<div align=center><font size=1 face="sans-serif">Please respond to<br>
VLC media player developers <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">Re: [vlc-devel] [PATCH] Make
USF subtitles conform better to spec</font></table>
<br>
<table>
<tr valign=top>
<td>
<td></table>
<br></table>
<br>
<br>
<br><font size=2><tt>On Fri, Jun 22, 2007, Bernie Purcell wrote:<br>
> This patch is a tidied up version of code that was originally submitted<br>
> in a much larger patch. It is still a large patch, but now, at least,<br>
> most of the work centres solely on the subsdec.c module.<br>
> [....]<br>
 Is there any comment about this patch ?<br>
 If no I will commit it in a few days (3-4, with remarks below fixed).<br>
<br>
A few remarks:<br>
<br>
 - The code that removes multiple-spaces exists in quartztext and subsdec.<br>
  Is that wanted ? And if so why it is not also in freetype renderer
?<br>
<br>
 - memcpy( &ptr[x], &ptr[y], size ) is wrong.<br>
  You have to use memmove when source and destination overlaps. It
is used in<br>
  quartztext and subsdec when removing multiple-spaces<br>
<br>
 - You can use strlcpy(p,s,l) instead of strncpy( p, s, l); p[l] = 0;<br>
<br>
-- <br>
fenrir<br>
<br>
_______________________________________________<br>
vlc-devel mailing list<br>
vlc-devel@videolan.org<br>
http://mailman.videolan.org/listinfo/vlc-devel<br>
</tt></font>
<br>