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

Clément Bœsch ubitux at gmail.com
Sun Jan 16 13:14:41 CET 2011


On Thu, Jan 13, 2011 at 07:55:21AM +0100, Reimar Döffinger wrote:
> > [...]
> > -if(!mpctx->sh_video) goto main; // audio-only
> > -
> > -if(!reinit_video_chain()) {
> > -  if(!mpctx->sh_video){
> > -    if(!mpctx->sh_audio) goto goto_next_file;
> > -    goto main; // exit_player(MSGTR_Exit_error);
> > -  }
> > -}
> > -
> > -   if(vo_flags & 0x08 && vo_spudec)
> > -      spudec_set_hw_spu(vo_spudec,mpctx->video_out);
> > +  if (mpctx->sh_video)
> > +      reinit_video_chain();
> >  
> > +  if (mpctx->sh_video) {
> > +      if (vo_flags & 0x08 && vo_spudec)
> > +          spudec_set_hw_spu(vo_spudec, mpctx->video_out);
> >  #ifdef CONFIG_FREETYPE
> > -   force_load_font = 1;
> > +      force_load_font = 1;
> >  #endif
> > +  } else if (!mpctx->sh_audio)
> > +      goto goto_next_file;
> 
> While it's small enough to be readable/reviewable okish anyway,
> I'd prefer it if you applied reindenting the unchanged parts
> separately.
> Otherwise ok.

Applied without reindent, thank you.

-- 
Clément B.


More information about the MPlayer-dev-eng mailing list