[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