[MPlayer-dev-eng] [patch] slow menus
Dominik 'Rathann' Mierzejewski
dominik at rangers.eu.org
Sun Jan 7 18:53:00 CET 2007
On Sunday, 07 January 2007 at 13:26, Adam Sulmicki wrote:
[...]
> can folks review it for correctness?
>
> I would be curious how it works for other people who use -dr and menus.
>
> This is the patch, but without any identation
>
> http://www.missl.cs.umd.edu/~adam/mplayer.slow-menu.basic.diff
>
> Same as above but with indentation
>
> http://www.missl.cs.umd.edu/~adam/mplayer.slow-menu.diff
http://www.mplayerhq.hu/DOCS/tech/patches.txt
[...]
5. Read your patch. We'll *refuse* it if it changes indentation of the
code or if it does tab/space conversion or other cosmetic changes!
Your patch still contains a cosmetic change, i.e. this:
- } else {
+ } else { // show=0
+
[...]
9. Send your patch to the mplayer-dev-eng mailing list as a base64-encoded
attachment with the subject line:
'[PATCH] very short description of the patch'.
In the mail, describe in a few sentences what you change and why.
You haven't attached it to your message. I know all these rules may seem
unimportant and trivial to you, but please bear with us.
Regards,
R.
--
MPlayer developer and RPMs maintainer: http://mplayerhq.hu http://rpm.livna.org
There should be a science of discontent. People need hard times and
oppression to develop psychic muscles.
-- from "Collected Sayings of Muad'Dib" by the Princess Irulan
More information about the MPlayer-dev-eng
mailing list