[MPlayer-dev-eng] [PATCH] ralf support with native demuxer

Alexander Strasser eclipse7 at gmx.net
Sat May 26 01:45:19 CEST 2012


Hi Roberto,

Roberto Togni wrote:
>  this adds support for ralf codec (ffmpeg decoder only, not the binary 
> dll) to our real demuxer.
> 
> Tested with the two ralf sample that I have.

  Patch looks good to me. 

  I noticed 2 minor points that should *not* be addressed in this patch:
  1) realloc is not used properly (failure case)
     not sure if it wouldn't even be better to solve it without the realloc
  2) the formatting of the (whole?) file is a bit weird
     - mix of tabs and spaces
     - probably more...
 
> I'll apply it in a few days if no comments (and if my svn account still
> works).

  If it doesn't work, please complain here. It will for sure be fixed :)
 
[...]

  Alexander


More information about the MPlayer-dev-eng mailing list