[MPlayer-dev-eng] [PATCH] improvement of sami subtitles feature

[Clément Bœsch]

On Thu, Feb 14, 2013 at 03:19:02PM -0500, compn wrote:
> On Thu, 14 Feb 2013 19:38:53 +0100, Clément Bœsch wrote:
> >On Thu, Feb 14, 2013 at 04:18:16PM +0900, Jihyun Cho wrote:
> >[...]
> >> I separated and trimmed the patch.
> >> 'mplayer_sami_ass_converter.patch' depends on
> >> 'mplayer_sami_html_character_entities.patch', because it uses sami_entities
> >> table.
> >
> >AFAICT you are duplicating the SubRip-to-ASS code for the insane idea of
> 
> can the sami patch and subrip code be merged?
> 

Assuming you want the same features, they should, that's what my mail was
suggesting. But I was also actually questioning the whole point of this,
especially that first patch.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20130214/8a2ac5cf/attachment.asc>