[vlc-devel] [PATCH] Integrate the RTMPdump library to play rtmp and rtmpe streams.

[Jean-Baptiste Kempf]

On Sat, Jul 10, 2010 at 03:51:36PM +0300, Daniel Tisza wrote :
> Attached the new independent patch.


> +    unsigned int i;

> +    for ( i = 0; i < sizeof( rtmpdump_pars ) / sizeof( rtmpdump_param ); i++ )

You can use for ( unsigned int i = 0 ; ....


> +    p_sys->p_rtmp = RTMP_Alloc();
> +    if ( !p_sys->p_rtmp ) return VLC_EGENERIC;
VLC_ENOMEM maybe?

Any other opinions on this patch?

Best Regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/