[FFmpeg-devel] [PATCH] Bluray Subtitle Support, v3

stev391 at exemail.com.au stev391
Sat Jul 25 02:13:13 CEST 2009


On Fri, 2009-07-24 at 15:23 +0200, Diego Biurrun wrote:
> On Fri, Jul 24, 2009 at 05:57:24PM +1000, stev391 at exemail.com.au wrote:
> > 
> > Updated patch including Diego's, Aurelien's & Baptiste's comments.
> > Thanks for the excellent and quick feedback.
> 
Thanks again for the comments, I'm glad somebody is interested in my
patch.

Updated patch (from v2) to include Diego's & Aurelien's comments.

Summary of changes (in addition to comments):
* Removed code parsing the window segment as no new information is
provided.
    - Removed associated context variables.
    - Added comment in pgssub_decode() detailing structure of window
      segment for future reference if required.
* Removed some temporary variables from code that were not used to
validate the data before storing in the subtitle context.

Regards,
Stephen

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bluray_subtitles_ffmpeg_19507_v3.diff
Type: text/x-patch
Size: 17181 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090725/443f0104/attachment.bin>



More information about the ffmpeg-devel mailing list