[MPlayer-dev-eng] [PATCH][TRIVIAL] Few cosmetics to x11_common

Reimar Döffinger Reimar.Doeffinger at gmx.de
Tue Nov 3 09:19:53 CET 2009


On Tue, Nov 03, 2009 at 01:04:45AM -0300, Reynaldo H. Verdejo Pinochet wrote:
> @@ -1123,9 +1121,9 @@
>      XMapWindow(mDisplay, vo_window);
>      XClearWindow(mDisplay, vo_window);
>      // wait for map
> -    do {
> +    do
>        XNextEvent(mDisplay, &xev);
> -    } while (xev.type != MapNotify || xev.xmap.event != vo_window);
> +    while (xev.type != MapNotify || xev.xmap.event != vo_window);

It definitely should be tested.
Some of the linebreak changes (particularly the strargcmp stuff) is a 
bit questionable, you could say it decreases readability, same for
the {} for nested loops.



More information about the MPlayer-dev-eng mailing list