[MPlayer-dev-eng] [PATCH] Get rid of the "main" label

Clément Bœsch ubitux at gmail.com
Sat Dec 25 17:29:32 CET 2010


Hi,

I'd like a feedback on this patch: there is still a "main" label in
mplayer.c. This is patch to get rid of it.

The code first check if there is a video context (is it the correct name
for sh_video?). If there is, the video chain is reset. After that, if
neither sh_video nor sh_audio are set, it goes to the next file.

Then, two things are done, but only if there was a video context at first,
so I just moved this in the "main" code where sh_video is checked. It
seems no previous code depends on the spu state and the font load flag.

Btw, mplayer.c really needs at least a reindent; the current state is
quiet a mess…

-- 
Clément B.
Not sent from a jesusPhone.


More information about the MPlayer-dev-eng mailing list