Message ID | 20200525194302.24515-1-andreas.rheinhardt@gmail.com |
---|---|
State | Accepted |
Commit | c6e73ad2b783b81e043d1b04c69d8dbe93237068 |
Headers | show |
Series | [FFmpeg-devel,1/4] avformat/hlsplaylist: Add const where appropriate | expand |
Context | Check | Description |
---|---|---|
andriy/default | pending | |
andriy/make | success | Make finished |
andriy/make_fate | success | Make fate finished |
> 2020年5月26日 上午3:42,Andreas Rheinhardt <andreas.rheinhardt@gmail.com> 写道: > > Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> > --- > libavformat/hlsplaylist.c | 27 ++++++++++++++++----------- > libavformat/hlsplaylist.h | 23 +++++++++++++---------- > 2 files changed, 29 insertions(+), 21 deletions(-) > > diff --git a/libavformat/hlsplaylist.c b/libavformat/hlsplaylist.c > index 43f9d281ba..7a89846369 100644 > --- a/libavformat/hlsplaylist.c > +++ b/libavformat/hlsplaylist.c > @@ -35,8 +35,10 @@ void ff_hls_write_playlist_version(AVIOContext *out, int version) { > avio_printf(out, "#EXT-X-VERSION:%d\n", version); > } > > -void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, > - const char *filename, char *language, int name_id, int is_default) { > +void ff_hls_write_audio_rendition(AVIOContext *out, const char *agroup, > + const char *filename, const char *language, > + int name_id, int is_default) > +{ > if (!out || !agroup || !filename) > return; > > @@ -48,8 +50,10 @@ void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, > avio_printf(out, "URI=\"%s\"\n", filename); > } > > -void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, > - const char *filename, char *language, int name_id, int is_default) { > +void ff_hls_write_subtitle_rendition(AVIOContext *out, const char *sgroup, > + const char *filename, const char *language, > + int name_id, int is_default) > +{ > if (!out || !filename) > return; > > @@ -61,10 +65,11 @@ void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, > avio_printf(out, "URI=\"%s\"\n", filename); > } > > -void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, > - int bandwidth, const char *filename, char *agroup, > - char *codecs, char *ccgroup, char *sgroup) { > - > +void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, int bandwidth, > + const char *filename, const char *agroup, > + const char *codecs, const char *ccgroup, > + const char *sgroup) > +{ > if (!out || !filename) > return; > > @@ -112,7 +117,7 @@ void ff_hls_write_playlist_header(AVIOContext *out, int version, int allowcache, > } > } > > -void ff_hls_write_init_file(AVIOContext *out, char *filename, > +void ff_hls_write_init_file(AVIOContext *out, const char *filename, > int byterange_mode, int64_t size, int64_t pos) { > avio_printf(out, "#EXT-X-MAP:URI=\"%s\"", filename); > if (byterange_mode) { > @@ -125,8 +130,8 @@ int ff_hls_write_file_entry(AVIOContext *out, int insert_discont, > int byterange_mode, > double duration, int round_duration, > int64_t size, int64_t pos, //Used only if HLS_SINGLE_FILE flag is set > - char *baseurl, //Ignored if NULL > - char *filename, double *prog_date_time, > + const char *baseurl /* Ignored if NULL */, > + const char *filename, double *prog_date_time, > int64_t video_keyframe_size, int64_t video_keyframe_pos, int iframe_mode) { > if (!out || !filename) > return AVERROR(EINVAL); > diff --git a/libavformat/hlsplaylist.h b/libavformat/hlsplaylist.h > index a124bdcffb..4348a26c75 100644 > --- a/libavformat/hlsplaylist.h > +++ b/libavformat/hlsplaylist.h > @@ -37,24 +37,27 @@ typedef enum { > } PlaylistType; > > void ff_hls_write_playlist_version(AVIOContext *out, int version); > -void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, > - const char *filename, char *language, int name_id, int is_default); > -void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, > - const char *filename, char *language, int name_id, int is_default); > -void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, > - int bandwidth, const char *filename, char *agroup, > - char *codecs, char *ccgroup, char *sgroup); > +void ff_hls_write_audio_rendition(AVIOContext *out, const char *agroup, > + const char *filename, const char *language, > + int name_id, int is_default); > +void ff_hls_write_subtitle_rendition(AVIOContext *out, const char *sgroup, > + const char *filename, const char *language, > + int name_id, int is_default); > +void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, int bandwidth, > + const char *filename, const char *agroup, > + const char *codecs, const char *ccgroup, > + const char *sgroup); > void ff_hls_write_playlist_header(AVIOContext *out, int version, int allowcache, > int target_duration, int64_t sequence, > uint32_t playlist_type, int iframe_mode); > -void ff_hls_write_init_file(AVIOContext *out, char *filename, > +void ff_hls_write_init_file(AVIOContext *out, const char *filename, > int byterange_mode, int64_t size, int64_t pos); > int ff_hls_write_file_entry(AVIOContext *out, int insert_discont, > int byterange_mode, > double duration, int round_duration, > int64_t size, int64_t pos, //Used only if HLS_SINGLE_FILE flag is set > - char *baseurl, //Ignored if NULL > - char *filename, double *prog_date_time, > + const char *baseurl /* Ignored if NULL */, > + const char *filename, double *prog_date_time, > int64_t video_keyframe_size, int64_t video_keyframe_pos, int iframe_mode); > void ff_hls_write_end_list (AVIOContext *out); > > -- > 2.20.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". patchset lgtm Thanks Steven Liu
Steven Liu: > > >> 2020年5月26日 上午3:42,Andreas Rheinhardt <andreas.rheinhardt@gmail.com> 写道: >> >> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> >> --- >> libavformat/hlsplaylist.c | 27 ++++++++++++++++----------- >> libavformat/hlsplaylist.h | 23 +++++++++++++---------- >> 2 files changed, 29 insertions(+), 21 deletions(-) >> >> diff --git a/libavformat/hlsplaylist.c b/libavformat/hlsplaylist.c >> index 43f9d281ba..7a89846369 100644 >> --- a/libavformat/hlsplaylist.c >> +++ b/libavformat/hlsplaylist.c >> @@ -35,8 +35,10 @@ void ff_hls_write_playlist_version(AVIOContext *out, int version) { >> avio_printf(out, "#EXT-X-VERSION:%d\n", version); >> } >> >> -void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, >> - const char *filename, char *language, int name_id, int is_default) { >> +void ff_hls_write_audio_rendition(AVIOContext *out, const char *agroup, >> + const char *filename, const char *language, >> + int name_id, int is_default) >> +{ >> if (!out || !agroup || !filename) >> return; >> >> @@ -48,8 +50,10 @@ void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, >> avio_printf(out, "URI=\"%s\"\n", filename); >> } >> >> -void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, >> - const char *filename, char *language, int name_id, int is_default) { >> +void ff_hls_write_subtitle_rendition(AVIOContext *out, const char *sgroup, >> + const char *filename, const char *language, >> + int name_id, int is_default) >> +{ >> if (!out || !filename) >> return; >> >> @@ -61,10 +65,11 @@ void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, >> avio_printf(out, "URI=\"%s\"\n", filename); >> } >> >> -void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, >> - int bandwidth, const char *filename, char *agroup, >> - char *codecs, char *ccgroup, char *sgroup) { >> - >> +void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, int bandwidth, >> + const char *filename, const char *agroup, >> + const char *codecs, const char *ccgroup, >> + const char *sgroup) >> +{ >> if (!out || !filename) >> return; >> >> @@ -112,7 +117,7 @@ void ff_hls_write_playlist_header(AVIOContext *out, int version, int allowcache, >> } >> } >> >> -void ff_hls_write_init_file(AVIOContext *out, char *filename, >> +void ff_hls_write_init_file(AVIOContext *out, const char *filename, >> int byterange_mode, int64_t size, int64_t pos) { >> avio_printf(out, "#EXT-X-MAP:URI=\"%s\"", filename); >> if (byterange_mode) { >> @@ -125,8 +130,8 @@ int ff_hls_write_file_entry(AVIOContext *out, int insert_discont, >> int byterange_mode, >> double duration, int round_duration, >> int64_t size, int64_t pos, //Used only if HLS_SINGLE_FILE flag is set >> - char *baseurl, //Ignored if NULL >> - char *filename, double *prog_date_time, >> + const char *baseurl /* Ignored if NULL */, >> + const char *filename, double *prog_date_time, >> int64_t video_keyframe_size, int64_t video_keyframe_pos, int iframe_mode) { >> if (!out || !filename) >> return AVERROR(EINVAL); >> diff --git a/libavformat/hlsplaylist.h b/libavformat/hlsplaylist.h >> index a124bdcffb..4348a26c75 100644 >> --- a/libavformat/hlsplaylist.h >> +++ b/libavformat/hlsplaylist.h >> @@ -37,24 +37,27 @@ typedef enum { >> } PlaylistType; >> >> void ff_hls_write_playlist_version(AVIOContext *out, int version); >> -void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, >> - const char *filename, char *language, int name_id, int is_default); >> -void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, >> - const char *filename, char *language, int name_id, int is_default); >> -void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, >> - int bandwidth, const char *filename, char *agroup, >> - char *codecs, char *ccgroup, char *sgroup); >> +void ff_hls_write_audio_rendition(AVIOContext *out, const char *agroup, >> + const char *filename, const char *language, >> + int name_id, int is_default); >> +void ff_hls_write_subtitle_rendition(AVIOContext *out, const char *sgroup, >> + const char *filename, const char *language, >> + int name_id, int is_default); >> +void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, int bandwidth, >> + const char *filename, const char *agroup, >> + const char *codecs, const char *ccgroup, >> + const char *sgroup); >> void ff_hls_write_playlist_header(AVIOContext *out, int version, int allowcache, >> int target_duration, int64_t sequence, >> uint32_t playlist_type, int iframe_mode); >> -void ff_hls_write_init_file(AVIOContext *out, char *filename, >> +void ff_hls_write_init_file(AVIOContext *out, const char *filename, >> int byterange_mode, int64_t size, int64_t pos); >> int ff_hls_write_file_entry(AVIOContext *out, int insert_discont, >> int byterange_mode, >> double duration, int round_duration, >> int64_t size, int64_t pos, //Used only if HLS_SINGLE_FILE flag is set >> - char *baseurl, //Ignored if NULL >> - char *filename, double *prog_date_time, >> + const char *baseurl /* Ignored if NULL */, >> + const char *filename, double *prog_date_time, >> int64_t video_keyframe_size, int64_t video_keyframe_pos, int iframe_mode); >> void ff_hls_write_end_list (AVIOContext *out); >> >> -- >> 2.20.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". > > patchset lgtm > > Thanks > > Steven Liu Applied, thanks. - Andreas
diff --git a/libavformat/hlsplaylist.c b/libavformat/hlsplaylist.c index 43f9d281ba..7a89846369 100644 --- a/libavformat/hlsplaylist.c +++ b/libavformat/hlsplaylist.c @@ -35,8 +35,10 @@ void ff_hls_write_playlist_version(AVIOContext *out, int version) { avio_printf(out, "#EXT-X-VERSION:%d\n", version); } -void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, - const char *filename, char *language, int name_id, int is_default) { +void ff_hls_write_audio_rendition(AVIOContext *out, const char *agroup, + const char *filename, const char *language, + int name_id, int is_default) +{ if (!out || !agroup || !filename) return; @@ -48,8 +50,10 @@ void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, avio_printf(out, "URI=\"%s\"\n", filename); } -void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, - const char *filename, char *language, int name_id, int is_default) { +void ff_hls_write_subtitle_rendition(AVIOContext *out, const char *sgroup, + const char *filename, const char *language, + int name_id, int is_default) +{ if (!out || !filename) return; @@ -61,10 +65,11 @@ void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, avio_printf(out, "URI=\"%s\"\n", filename); } -void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, - int bandwidth, const char *filename, char *agroup, - char *codecs, char *ccgroup, char *sgroup) { - +void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, int bandwidth, + const char *filename, const char *agroup, + const char *codecs, const char *ccgroup, + const char *sgroup) +{ if (!out || !filename) return; @@ -112,7 +117,7 @@ void ff_hls_write_playlist_header(AVIOContext *out, int version, int allowcache, } } -void ff_hls_write_init_file(AVIOContext *out, char *filename, +void ff_hls_write_init_file(AVIOContext *out, const char *filename, int byterange_mode, int64_t size, int64_t pos) { avio_printf(out, "#EXT-X-MAP:URI=\"%s\"", filename); if (byterange_mode) { @@ -125,8 +130,8 @@ int ff_hls_write_file_entry(AVIOContext *out, int insert_discont, int byterange_mode, double duration, int round_duration, int64_t size, int64_t pos, //Used only if HLS_SINGLE_FILE flag is set - char *baseurl, //Ignored if NULL - char *filename, double *prog_date_time, + const char *baseurl /* Ignored if NULL */, + const char *filename, double *prog_date_time, int64_t video_keyframe_size, int64_t video_keyframe_pos, int iframe_mode) { if (!out || !filename) return AVERROR(EINVAL); diff --git a/libavformat/hlsplaylist.h b/libavformat/hlsplaylist.h index a124bdcffb..4348a26c75 100644 --- a/libavformat/hlsplaylist.h +++ b/libavformat/hlsplaylist.h @@ -37,24 +37,27 @@ typedef enum { } PlaylistType; void ff_hls_write_playlist_version(AVIOContext *out, int version); -void ff_hls_write_audio_rendition(AVIOContext *out, char *agroup, - const char *filename, char *language, int name_id, int is_default); -void ff_hls_write_subtitle_rendition(AVIOContext *out, char *sgroup, - const char *filename, char *language, int name_id, int is_default); -void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, - int bandwidth, const char *filename, char *agroup, - char *codecs, char *ccgroup, char *sgroup); +void ff_hls_write_audio_rendition(AVIOContext *out, const char *agroup, + const char *filename, const char *language, + int name_id, int is_default); +void ff_hls_write_subtitle_rendition(AVIOContext *out, const char *sgroup, + const char *filename, const char *language, + int name_id, int is_default); +void ff_hls_write_stream_info(AVStream *st, AVIOContext *out, int bandwidth, + const char *filename, const char *agroup, + const char *codecs, const char *ccgroup, + const char *sgroup); void ff_hls_write_playlist_header(AVIOContext *out, int version, int allowcache, int target_duration, int64_t sequence, uint32_t playlist_type, int iframe_mode); -void ff_hls_write_init_file(AVIOContext *out, char *filename, +void ff_hls_write_init_file(AVIOContext *out, const char *filename, int byterange_mode, int64_t size, int64_t pos); int ff_hls_write_file_entry(AVIOContext *out, int insert_discont, int byterange_mode, double duration, int round_duration, int64_t size, int64_t pos, //Used only if HLS_SINGLE_FILE flag is set - char *baseurl, //Ignored if NULL - char *filename, double *prog_date_time, + const char *baseurl /* Ignored if NULL */, + const char *filename, double *prog_date_time, int64_t video_keyframe_size, int64_t video_keyframe_pos, int iframe_mode); void ff_hls_write_end_list (AVIOContext *out);
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> --- libavformat/hlsplaylist.c | 27 ++++++++++++++++----------- libavformat/hlsplaylist.h | 23 +++++++++++++---------- 2 files changed, 29 insertions(+), 21 deletions(-)