On 5/6/2019 9:19 PM, Andreas Rheinhardt wrote:
> Andreas Rheinhardt:
>> At this point, ts already includes the ts_offset so that the relative
>> time written with the cluster is already given by ts - mkv->cluster_pts.
>> It is this number that needs to fit into an int16_t.
>>
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com>
>> ---
>> The only difference between this version and the earlier version is the
>> authorship information. My earlier emails were munged and in order to
>> make the committer's life easier, I'll resend the whole patchset.
>> I would appreciate reviews and comments.
>> libavformat/matroskaenc.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c
>> index 1c98c0dceb..c006cbf35c 100644
>> --- a/libavformat/matroskaenc.c
>> +++ b/libavformat/matroskaenc.c
>> @@ -2404,7 +2404,7 @@ static int mkv_write_packet_internal(AVFormatContext
>> *s, AVPacket *pkt, int add_
>> ts += mkv->tracks[pkt->stream_index].ts_offset;
>>
>> if (mkv->cluster_pos != -1) {
>> - int64_t cluster_time = ts - mkv->cluster_pts +
>> mkv->tracks[pkt->stream_index].ts_offset;
>> + int64_t cluster_time = ts - mkv->cluster_pts;
>> if ((int16_t)cluster_time != cluster_time) {
>> av_log(s, AV_LOG_WARNING, "Starting new cluster due to
>> timestamp\n");
>> mkv_start_new_cluster(s, pkt);
>>
> Ping for the whole patchset.
>
> - Andreas
Set pushed. Thanks!
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
