[MPlayer-dev-eng] [PATCH] dvdnav part 3

[Ötvös Attila]

2006. december 9. 14.51 dátummal Nico Sabbi ezt írta:
> Ötvös Attila wrote:
> > +#if HAVE_COLOR_SPU
> > +if (dvdnav_color_spu_flg==-1) {
> > +    dvdnav_color_spu_flg=0;
> > +    if (video_out) {
> > +	if (!dvdnav_color_spu_flg &&
> > video_out->control(VOCTRL_YUVSPU_SUPPORT,NULL)==VO_TRUE) +	   
> > dvdnav_color_spu_flg=DVDNAV_SPU_YUV;
> > +	if (!dvdnav_color_spu_flg &&
> > video_out->control(VOCTRL_YUYSPU_SUPPORT,NULL)==VO_TRUE) +	   
> > dvdnav_color_spu_flg=DVDNAV_SPU_YUY;
> > +	if (!dvdnav_color_spu_flg &&
> > video_out->control(VOCTRL_RGBSPU_SUPPORT,NULL)==VO_TRUE) +	   
> > dvdnav_color_spu_flg=DVDNAV_SPU_RGB;
> > +	if (!dvdnav_color_spu_flg &&
> > video_out->control(VOCTRL_BGRSPU_SUPPORT,NULL)==VO_TRUE) +	   
> > dvdnav_color_spu_flg=DVDNAV_SPU_BGR;
> > +//	if (!dvdnav_color_spu_flg)
> > +//	    mp_msg(MSGT_CPLAYER,MSGL_INFO,MSGTR_MPDVDNAV_NoColorSpu);
> > +	}
> > +    }
>
> where are these VOCTRL_* defined and implemented? not in these 7
> patches, it seems. Maybe in a different patch that you forgot to send?
> If so, please send _only_ the implementation of these new voctrls in a
> separate patch

What do you think if the basic concept is right in: 
"[PATCH] Dvdnav part1 - color spu menu buttons"?
http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2006-June/043569.html

If required then I rewriteing the color spu patch, but I know what must i'm 
do...

Best regards!
Attila