[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