[FFmpeg-devel] [PATCH 1/2] refactored srtdec

[Clément Bœsch]

On Fri, Apr 10, 2015 at 04:14:59AM -0300, James Almer wrote:
[...]
> IMO all this belongs in ass.c/h instead of two new files.

Not really, it's about parsing "html"-like markup. The fact that it's
generating ASS is accessory. It could be something else (if the internal
representation of decoded text sub wasn't ASS typically, which could
happen in the future). The complex logic really is about parsing that
markup, I believe it belongs in this separate file.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150421/456c0a43/attachment.asc>