[Ffmpeg-devel] Matroska Patch

Steve Lhomme steve.lhomme
Sat Mar 25 17:15:34 CET 2006


Diego Biurrun wrote:
> On Thu, Mar 23, 2006 at 10:55:48AM +0100, Diego Biurrun wrote:
>> On Thu, Mar 23, 2006 at 10:29:09AM +0100, Steve Lhomme wrote:
>>> Diego Biurrun wrote:
>>>> This case is different IMO.  The use of 'time' as variable name is
>>>> problematic.  You have to have a copy of the C standard lying around to
>>>> check which uses are allowed and which aren't to avoid shooting yourself
>>>> in the foot.  In this situation avoiding time as variable name outright
>>>> looks like the clean solution to me.
>>>>
>>>> @Steve: Variable renaming should go in a separate patch in any case.
>>> Here you go.
>> Michael, if you don't object, I'm going to apply this during the next
>> days.
> 
> Applied.


Thanks Diego.

So here is the more important patch that makes the block_time signed (as 
in the specs).

Steve
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: matroska-signed-block_time.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060325/eb64d737/attachment.asc>



More information about the ffmpeg-devel mailing list