[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