[Ffmpeg-devel] Matroska Patch
Sat Mar 25 18:59:40 CET 2006
On Sat, Mar 25, 2006 at 05:15:34PM +0100, Steve Lhomme wrote:
> 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
> Thanks Diego.
> So here is the more important patch that makes the block_time signed (as
> in the specs).
rejected, this is wrong its int16 not 64 this change will not do anything
excpet maybe hiding a gcc warning, it will never become <0
More information about the ffmpeg-devel