[FFmpeg-devel] [PATCH] rtsp.c: EOS support

[Luca Barbato]

Ronald S. Bultje wrote:
> Hi,
> 
> attached is something that adds sort-of functional EOS support to the
> RTSP demuxer. It works, but I don't think it's entirely ready, so see
> it partially as a RFC. For example, I'm wondering if people agree
> setting RTSPState::state is a good way to maintain EOS state on
> subsequent read_packet() calls, and I'm interested in other approaches
> that do the same thing.

I'd add rtcp BYE before implementing that

The patch looks fine but why removing the comment?

-                /* XXX: parse message */
+                if (rt->state != RTSP_STATE_PLAYING)
+                    return 0;

lu

-- 

Luca Barbato
Gentoo Council Member
Gentoo/linux Gentoo/PPC
http://dev.gentoo.org/~lu_zero