[Ffmpeg-devel] [PATCH] WAVE and AIFF packets timestamps

Patrick Guimond patg
Tue Sep 5 21:08:12 CEST 2006


Michael Niedermayer wrote:
[...]
> this should be assert(pkt->pos >= st_start_pos) as the pakets before the
> start_pos really doesnt seem valid ...
> also id tend to put this in utils.c rather then raw.c and add a av_ instead
> of pcm_ prefix maybe call it av_cbr_set_ts()

fixed

[...]
> well, this definitly doesnt belong in this patch and for a seperate patch its
> probably too irrelevant ...

alright

[...]
>>  #ifdef CONFIG_MUXERS
>> -typedef struct {
>> -    offset_t form;
>> -    offset_t frames;
>> -    offset_t ssnd;
>> -} AIFFOutputContext;
>> -
> 
> this is mostly cosmetic and rejected, this can be done with far fewer
> changed lines
> 

Modified patch attached

Greetings,
Patrick Guimond

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-pcm_pts.patch
Type: text/x-patch
Size: 3961 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060905/00b397c0/attachment.bin>



More information about the ffmpeg-devel mailing list