[MPlayer-cygwin] reworked vo_winvidix.c
Sascha Sommer
saschasommer at freenet.de
Mon Jun 27 07:00:16 CEST 2005
On Wednesday 22 June 2005 20:47, Erik Lunchpail wrote:
> > Could you seperate the noncosmetic changes?
> > The cosmetic changes are unacceptable in their
> > current form.
>
> Sure, here's 2. First allows runtime resizing while
> keeping the same aspect. The second one isn't all that
> important, and shouldn't affect the commandline
> version at all, but helps me out with a gui as it
> prevents sending a quit message to it after every
> file.
>
Thanks. Both patches applied.
Though one thing to remember for other patches is that you have to put the
variable declaration to the beginning of a block. Otherwise gcc 2.95 won't be
able to compile it.
> And i'll try to avoid cosmetic changes in the future.
> I'm a graphic designer, it's hard for me when things
> don't line up, and whitespace is generally a very bad
> thing in my field.
>
Nevermind. It is only that these cosmetic changes make
it hard to track the real changes.
Sascha
More information about the MPlayer-cygwin
mailing list