[vlc-devel] [PATCH] Correctly respond to a Connection Close message.

Bill C. Riemers briemers at redhat.com
Sun Jun 15 07:23:05 CEST 2008


I'm sorry, I am getting terse.

It is just that I started with what I thought was a request for a minor
bug fix.   I didn't like having an error message pop-up telling me all
my AVI's are corrupt when they aren't, and I like to be able to seek to
the previous place I was watching my video if I need to stop it for some
reason.

I started off with a simply patch of just a few lines.  Now after
responding to fix each objection I have done six revisions of that
patch, and I'm still not any closer to having a bug fix included in the
next release.

I can certainly understand if someone else has different ideas and how
to solve the problem, and would be quite happy to see ANY fix that works
go into the next release.   The only reason I wrote a patch to try to
fix the problem, is I saw several hundred open bugs in the bug tracking
system, and figured you guys needed more of a hand than just someone
reporting the problem.

Bill


Bill C. Riemers wrote:
> I give up.  It is virtually impossible to submit a patch for this project.
>
> Best of luck.
>
> Bill
>
>
> Rémi Denis-Courmont wrote:
>   
>> Le vendredi 13 juin 2008 21:49:18 Bill C. Riemers, vous avez écrit :
>>   
>>     
>>> OK.  This is patch three in the sequence.
>>>     
>>>       
>>   
>>     
>>> I've restored http CAN_FASTSEEK to false because often http is used over
>>> high latency networks.
>>>     
>>>       
>>   
>>     
>>> I've changed avi demux not to require CAN_FASTSEEK, because even over high
>>> latency networks, the http code has no problem avi's.
>>>     
>>>       
>> If AVI cannot be read at all without seeking, that is the proper solution. 
>> However, we have had this CAN_FASTSEEK check for almost five years, and many 
>> many releases. The AVI demux is supposed to work on non-seekable streams! I 
>> am not very comfortable downgrading from CAN_FASTSEEK to CAN_SEEK until we 
>> know why it broke (assuming it did break). Besides, it will only fix the 
>> problem for slow-seekable streams, and not for non-seekable ones.
>>
>> Please don't patch the AVI stuff, at least not in the same patchset...
>>
>>   
>>     
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>   




More information about the vlc-devel mailing list