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

On Sat, Dec 25, 2010 at 05:29:32PM +0100, Clément Bœsch wrote:

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…

Ping. -- Clément B.