On Sun, 10 Jun 2012 18:16:48 +0200, David Girault <[email protected]> wrote: > Le dimanche 10 juin 2012 à 16:24 +0200, Anton Khirnov a écrit : > > On Sun, 10 Jun 2012 15:08:35 +0200, David Girault <[email protected]> > > wrote: > > > > > > Yes, it was very similar (igsmenudec.c was started as a fork of > > > pgssubdec.c), and thus my first implementation use the subtitle API. But > > > I switch to another media type for two main reason: > > > - I don't want to change subtitle API, > > > > That's not a very convincing reason without more information =p > > Why not? > > First, subtitle API may be actually used by a lot of players, so I'm > pretty cautious before touching an API. >
Well you're not breaking anything. From what i see you're only adding two fields. That is backward compatible and doesn't break API or ABI. Which reminds me -- what are those fields even for? They don't seem to be used in your patchset. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
