[vlc-devel] Re: [PATCH] Subtitle Support Again

Rémi Denis-Courmont rem at videolan.org
Wed Feb 7 22:30:51 CET 2007


Le mercredi 7 février 2007 23:22, Aaron Mady a écrit :
>             Took some further time looking at stripping out subtitle
> comments and so here is the new patch.  This version now also finds
> embedded formatting tags, but for the time being just prints them out
> as a message.  That should be useful for future work on formatting,
> however.

- do not use the heap when you can use the stack,
- follow VLC indenting conventions,
- avoid unnecessary empty line addition when submitting patches,
- provide a unified diff that applies at -p0 or -p1 to VLC trunk.


Rémi Denis-Courmont
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20070207/f1abcc33/attachment.sig>

More information about the vlc-devel mailing list