[MPlayer-dev-eng]

Andreas Hess jaska at gmx.net
Sat Feb 8 00:50:16 CET 2003


On Sat, Feb 08, 2003 at 12:46:02AM +0100, Arpi wrote:
> >  
> >      /* Crack or decrypt CSS title key for current VTS */
> > -    if(i_ret<0) i_ret = _dvdcss_titlekey( dvdcss, i_block, p_title_key );
> > +    if( i_ret < 0 )
> > +    {
> > +        i_ret = _dvdcss_titlekey( dvdcss, i_block, p_title_key );
> >  
> >      if( i_ret < 0 )
> >      {
> >          _dvdcss_error( dvdcss, "fatal error in vts css key" );
> >          return i_ret;
> >      }
> > -    else if( i_ret == 0 )
> > +
> > +        if( i_ret == 0 )
> >      {
> >          _dvdcss_debug( dvdcss, "unencrypted title" );
> > -        /* Still store this in the cache, so we don't need to check again. */
> > +            /* We cache this anyway, so we don't need to check again. */
> > +        }
> >      }
> 
> is this correct? i think that } is misplaced and should be ~ 15 lines above.
> or the indentation of the code sucks so badly?

Yes, this is correct, this is just the product of -bB, take the previous
patch with the date "Sat, 08 Feb 2003 00:11:06 +0100", this is the same
without the bogus #ifdefs and #includes and without -bB and therefore
correct identation.


Andreas


More information about the MPlayer-dev-eng mailing list