[FFmpeg-devel] [PATCH] ffserver rtsp bug fixes
Wed May 19 14:53:25 CEST 2010
On Wed, May 19, 2010 at 12:10:33AM -0700, Howard Chu wrote:
> Luca Abeni wrote:
>> On 05/19/2010 08:44 AM, Howard Chu wrote:
>>> Luca Abeni wrote:
>>>> On 05/19/2010 08:24 AM, Howard Chu wrote:
>>>>>> Note that I still haven't gotten ffserver to successfully stream
>>>>>> anything from
>>>>>> a file, there are plenty more problems left to chase down.
>>>>> It turns out that streaming just audio from FLV files was working fine.
>>>>> But the H264 video was encoded as the AVC1 subtype, and none of the sdp
>>>>> or rtp code handled this. This patch gets RTP working for me from an
>>>>> (AVC1) file source.
>>>> I think this is the wrong solution. The (IMHO) correct one would be to
>>>> a bitstream filter to convert the NAL syntax.
>>> Why is that a better solution?
>> Because in this way we can separate different functionalities (converting
>> the bitstream syntax - in the bitstream filter - and packetising the
>> bitstream - in the rtp mixer) in different parts of the code. And we can
>> avoid code duplication (and we can reduce the complexity of the rtp muxer
>> AFAIR, the needed bitstream filter is already there, it just needs to be
>> used (I used this solution in one of my programs, some time ago... If I
>> remember well, I just needed to fix some minor issue. But then it worked).
> I don't think the muxer's complexity has been harmed by these patches. On
> the other hand, I've just taken a look at the h264_mp4toannexb filter
> source code and it is quite dense. Also, anything that does
> "alloc_and_copy" is a bad idea from a memory and CPU resource perspective.
> If bitstream filters were capable of operating in-place on the data, I
> might agree with you. But to me it just looks like a useless pig.
if you want bsfs to do something they cant currently, i suggest you fix bsfs
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel