[MPlayer-cvslog] r29456 - in trunk: etc/codecs.conf libmpdemux/mp_taglists.c

[Diego Biurrun]

On Thu, Jul 30, 2009 at 11:12:04AM +0200, Nico Sabbi wrote:
> On Thursday 30 July 2009 11:03:34 Diego Biurrun wrote:
> > On Thu, Jul 30, 2009 at 10:36:17AM +0200, nicodvb wrote:
> > > Log:
> > > added inexistant TRHD fourcc to handle TRUEHD streams in
> > > forthcoming patch
> >
> > This is the wrong way around.  Now we have a revision that claims
> > to support a feature but doesn't.
> 
> but the other way around would have used an inexistant fourcc

That's why it should have been one commit, not two.  If they depend on
each other, they should not be separated.

Diego