Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
  • * Re: [FFmpeg-devel] [PATCH 1/5] avformat/movenc: remove unused argument from get_sample_flags()
       [not found] <tencent_E47115C78906AEAD65415302C85B53E07305@qq.com>
       [not found] ` <tencent_C5562B9C08E0A05AE23499813C34033EC509@qq.com>
@ 2022-03-29  5:02 ` "zhilizhao(赵志立)"
  1 sibling, 0 replies; 2+ messages in thread
From: "zhilizhao(赵志立)" @ 2022-03-29  5:02 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

Ping for patch 1-3.

> On Dec 3, 2021, at 1:06 PM, Zhao Zhili <quinkblack@foxmail.com> wrote:
> 
> ---
> libavformat/movenc.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/libavformat/movenc.c b/libavformat/movenc.c
> index 38ff90833a..634a829f28 100644
> --- a/libavformat/movenc.c
> +++ b/libavformat/movenc.c
> @@ -4427,7 +4427,7 @@ static int mov_write_mfhd_tag(AVIOContext *pb, MOVMuxContext *mov)
>     return 0;
> }
> 
> -static uint32_t get_sample_flags(MOVTrack *track, MOVIentry *entry)
> +static uint32_t get_sample_flags(MOVIentry *entry)
> {
>     return entry->flags & MOV_SYNC_SAMPLE ? MOV_FRAG_SAMPLE_FLAG_DEPENDS_NO :
>            (MOV_FRAG_SAMPLE_FLAG_DEPENDS_YES | MOV_FRAG_SAMPLE_FLAG_IS_NON_SYNC);
> @@ -4487,7 +4487,7 @@ static int mov_write_tfhd_tag(AVIOContext *pb, MOVMuxContext *mov,
>         /* Set the default flags based on the second sample, if available.
>          * If the first sample is different, that can be signaled via a separate field. */
>         if (track->entry > 1)
> -            track->default_sample_flags = get_sample_flags(track, &track->cluster[1]);
> +            track->default_sample_flags = get_sample_flags(&track->cluster[1]);
>         else
>             track->default_sample_flags =
>                 track->par->codec_type == AVMEDIA_TYPE_VIDEO ?
> @@ -4512,11 +4512,11 @@ static int mov_write_trun_tag(AVIOContext *pb, MOVMuxContext *mov,
>             flags |= MOV_TRUN_SAMPLE_DURATION;
>         if (track->cluster[i].size != track->default_size)
>             flags |= MOV_TRUN_SAMPLE_SIZE;
> -        if (i > first && get_sample_flags(track, &track->cluster[i]) != track->default_sample_flags)
> +        if (i > first && get_sample_flags(&track->cluster[i]) != track->default_sample_flags)
>             flags |= MOV_TRUN_SAMPLE_FLAGS;
>     }
>     if (!(flags & MOV_TRUN_SAMPLE_FLAGS) && track->entry > 0 &&
> -         get_sample_flags(track, &track->cluster[0]) != track->default_sample_flags)
> +         get_sample_flags(&track->cluster[0]) != track->default_sample_flags)
>         flags |= MOV_TRUN_FIRST_SAMPLE_FLAGS;
>     if (track->flags & MOV_TRACK_CTTS)
>         flags |= MOV_TRUN_SAMPLE_CTS;
> @@ -4538,7 +4538,7 @@ static int mov_write_trun_tag(AVIOContext *pb, MOVMuxContext *mov,
>         avio_wb32(pb, moof_size + 8 + track->data_offset +
>                       track->cluster[first].pos); /* data offset */
>     if (flags & MOV_TRUN_FIRST_SAMPLE_FLAGS)
> -        avio_wb32(pb, get_sample_flags(track, &track->cluster[first]));
> +        avio_wb32(pb, get_sample_flags(&track->cluster[first]));
> 
>     for (i = first; i < end; i++) {
>         if (flags & MOV_TRUN_SAMPLE_DURATION)
> @@ -4546,7 +4546,7 @@ static int mov_write_trun_tag(AVIOContext *pb, MOVMuxContext *mov,
>         if (flags & MOV_TRUN_SAMPLE_SIZE)
>             avio_wb32(pb, track->cluster[i].size);
>         if (flags & MOV_TRUN_SAMPLE_FLAGS)
> -            avio_wb32(pb, get_sample_flags(track, &track->cluster[i]));
> +            avio_wb32(pb, get_sample_flags(&track->cluster[i]));
>         if (flags & MOV_TRUN_SAMPLE_CTS)
>             avio_wb32(pb, track->cluster[i].cts);
>     }
> -- 
> 2.31.1
> 

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 2+ messages in thread
    • end of thread, other threads:[~2022-03-29  5:02 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <tencent_E47115C78906AEAD65415302C85B53E07305@qq.com>
     [not found] ` <tencent_C5562B9C08E0A05AE23499813C34033EC509@qq.com>
2021-12-28 12:43   ` [FFmpeg-devel] [PATCH 1/5] avformat/movenc: remove unused argument from get_sample_flags() "zhilizhao(赵志立)"
2022-03-29  5:02 ` "zhilizhao(赵志立)"

    Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git