[MPlayer-dev-eng] [PATCH] fix dvdnav multi-cell menus

Dominik 'Rathann' Mierzejewski dominik at greysector.net
Tue Aug 16 20:35:46 CEST 2011


On Tuesday, 16 August 2011 at 08:18, Steaphan Greene wrote:
[...]
> The attached patch fixes both of these issues, and I have tested it
> extensively (always with my MPEG2 sync patch, of course) with both
> normal, and interactive DVDs, and including several examples of actual
> still frames.  It seems to solve the problem and cause no other new ones
> that I can find.  Note, it is really only a 2-line change (the rest are
> all just white-space changes to correct the indenting).

We prefer dvdnav-discuss mailing list for patches to dvdnav code:
https://lists.mplayerhq.hu/mailman/listinfo/dvdnav-discuss

Also, like for MPlayer, please split whitespace changes into a separate
patch. Note that you can use "final" identation in the lines that
you change in the functional part of the patch and just bring the rest
in line with a second, whitespace-only patch.

Thanks for your contributions.

Regards,
Dominik

-- 
MPlayer http://mplayerhq.hu | RPMFusion http://rpmfusion.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