[MPlayer-dev-eng] New patches coming

Benjamin Zores ben at geexbox.org
Sat May 15 11:33:57 CEST 2010


On Fri, May 14, 2010 at 3:48 PM, Uoti Urpala <uoti.urpala at pp1.inet.fi> wrote:
> On Fri, 2010-05-14 at 10:30 +0200, Dan Oscarsson wrote:
>> I plan do do a little cleaning and removing unneeded debug code before
>> publishing. I have split my enhancements into six small independent
>> patches and seven bigger patches that must be done in order. Hopefully
>> that is more acceptable for you? I have also added a special option to
>
> Of course it's hard to comment without seeing the patches, but at least
> your previous versions had problems unrelated to to any superficial
> issues like splitting. So changing just that is unlikely to make them
> significantly more acceptable.
>
> [...]

Dan,

Despite of Uoti's comments and based on your expected announced
ChangeLog, I'd still advise you to post these patches. They might
benefit to other people.

I have no idea how good/bad they could be but that's really not the
problem actually.
If you really intend to have them merged one day, I'd suggest you post
them to this list, one mail per patch, for them to be reviewed.

Please use the following for each mail:

[PATCH #x1/yz - Description]

And describe its content.
That's the only chance you got.

Also, be patient, because except from Reimar or maybe Uoti, there's
nearly no more reviewer (at least with this domain of expertise) in
MPlayer these days.

Ben



More information about the MPlayer-dev-eng mailing list