[MPlayer-dev-eng] [PATCH] dvdnav part 2 - still frames

Diego Biurrun diego at biurrun.de
Mon Jul 30 14:43:23 CEST 2007


On Mon, Jul 30, 2007 at 02:40:49PM +0200, Ötvös Attila wrote:
> 
> 
> --- stream/stream_dvdnav.c	(revision 23928)
> +++ stream/stream_dvdnav.c	(working copy)
> @@ -163,10 +179,15 @@
>          return event;
>        case DVDNAV_STILL_FRAME: {
>          dvdnav_still_event_t *still_event = (dvdnav_still_event_t*)(buf);
> -        //if (priv->started) dvd_nav_still=1;
> -        //else
> -          dvdnav_still_skip(priv->dvdnav); // don't let dvdnav stall on this image
> -
> +        priv->still_length=still_event->length;
> +        if (priv->still_length!=255)                                // set still frame duration
> +            priv->duration = priv->still_length * 1000;
> +        else
> +            priv->duration = 0;
> +        if (priv->still_length==0x01) {                             // FIXME: TIME HACK
> +            pci_t* pnavpci = dvdnav_get_current_nav_pci( priv->dvdnav );
> +            priv->duration = dvdtimetomsec(&(pnavpci->pci_gi.e_eltm));

The rest of the file uses two spaces indentation, please respect that,
same below.

> --- mplayer.c	(revision 23928)
> +++ mplayer.c	(working copy)
> +    if(in_size<0) {                                     // is wait or still frame and no redecode last frame
> +        if (mpctx->last_buffer) {                       // free buffer

Nit: Please consistently place a space after if.

> --- command.c	(revision 23928)
> +++ command.c	(working copy)
> @@ -2760,9 +2760,12 @@
>  
>  		if (mp_dvdnav_handle_input
>  		    (mpctx->stream, cmd->args[0].v.i, &button)) {
> +// FIXME: if the mplayer returns from movie to still frame then the mplayer can't init the video codec
> +#if 0

s/the mplayer/MPlayer/g

Please break overly long lines.

Diego



More information about the MPlayer-dev-eng mailing list