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

Clément Bœsch ubitux at gmail.com
Sat Jan 8 11:06:03 CET 2011


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.


More information about the MPlayer-dev-eng mailing list