[MPlayer-dev-eng] [PATCH] [TEST FUNC] Multi-channel reorder function

[Rich Felker]

On Thu, Nov 29, 2007 at 10:57:34AM +0100, Reimar Döffinger wrote:
> Hello,
> On Thu, Nov 29, 2007 at 10:12:10AM +0800, Ulion wrote:
> > 2007/11/29, Corey Hickey <bugfood-ml at fatooh.org>:
> [...]
> > > Anyway, with all that done, I'm through with nitpicking. I think you
> > > should threaten to apply my re-indented version of your patch. Give an
> > > ultimatum of a few days, and make sure Reimar has a chance to respond if
> > > he wants, since he had an opinion on channel reordering long ago. If
> > > everybody ignores you, apply.
> > 
> > I'd like to, but, before and after I post this patch, I ever discussed
> > with Reimar and other developers with it on irc, all current patches
> > for channel order fix were considered as hacks, for hacks, they does
> > not care performance so much. I don't known whether the opinion
> > changed now, so I have to ask Reimar about this, which one of our
> > patches could be acceptable to commit or after some additional
> > adoptions, Reimar?
> 
> I do not care much what will be applied, though I would prefer it if
> most of the new code is confined to the problematic decoders.

Agree strongly. Nasty code should always be confined to the nasty
special cases that necessitate it.

Rich