[MPlayer-dev-eng] [PATCH] -vobsub support for mencoder

[Michael Niedermayer]

On Wed, Feb 10, 2010 at 02:57:43PM -0500, compn wrote:
> On Wed, 10 Feb 2010 17:37:08 +0100, Gianluigi Tiesi wrote:
> >On Wed, Feb 10, 2010 at 06:43:13AM +0200, Uoti Urpala wrote:
> >> On Wed, 2010-02-10 at 04:56 +0100, Gianluigi Tiesi wrote:
> >> > I'm working on moving some of mplayer subtitle code in mpcommon
> >> > and make mencoder use it.
> >> > 
> >> > The attached patch should not be considered as final
> >> > but I would to post to have some feedbacks / corrections
> >> > 
> >> > Yes a better way would be vf_vobsub.c, it looks not trivial
> >> > to reuse the current vobsub code (I'm not so expert in frame processing tough)
> >> 
> >> This is a bad idea. You shouldn't tie more code to the mess in MEncoder.
> >> 
> >> If you really need more functionality then a more sound approach is to
> >> add the encoding functionality you need to MPlayer (start from MPlayer
> >> with vo yuv4mpeg and add the encoding functionality you need, instead of
> >> starting from MEncoder and trying to fix it).
> >> 
> >
> >you cannot encode faster than realtime this way, or I'm wrong?
> 
> you are wrong, it should work about the same way as -ao pcm -benchmark
> 
> but are we sure that starting over and adding a bunch of code to mplayer

> is the right way to do it ? if we are to drop mencoder, why not just
> focus on getting binary codec support into ffmpeg?

iam in favor of seeing binary codec support added to ffmpeg

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100213/164f5af4/attachment.pgp>