Re: [FFmpeg-devel] [PATCH 1/4] avformat/avio: Make avio_print_string_array() accept const pointers

[Andreas Rheinhardt <andreas.rheinhardt@outlook.com>]

Andreas Rheinhardt:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  doc/APIchanges        | 3 +++
>  libavformat/avio.h    | 2 +-
>  libavformat/aviobuf.c | 2 +-
>  3 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/doc/APIchanges b/doc/APIchanges
> index 10f6667e9e..055e6848a3 100644
> --- a/doc/APIchanges
> +++ b/doc/APIchanges
> @@ -2,6 +2,9 @@ The last version increases of all libraries were on 2023-02-09
>  
>  API changes, most recent first:
>  
> +2024-02-28 - xxxxxxxxxx - lavf 60.xx.100 - avio.h
> +  avio_print_string_array() now accepts an array of const pointers.
> +
>  2024-02-26 - xxxxxxxxxx - lavf 60.22.101 - avformat.h
>    AV_DISPOSITION_DEPENDENT may now also be used for video streams
>    intended to be merged with other video streams for presentation.
> diff --git a/libavformat/avio.h b/libavformat/avio.h
> index 887a397c37..43bce60ffd 100644
> --- a/libavformat/avio.h
> +++ b/libavformat/avio.h
> @@ -544,7 +544,7 @@ int avio_printf(AVIOContext *s, const char *fmt, ...) av_printf_format(2, 3);
>   * Usually you don't need to use this function directly but its macro wrapper,
>   * avio_print.
>   */
> -void avio_print_string_array(AVIOContext *s, const char *strings[]);
> +void avio_print_string_array(AVIOContext *s, const char * const strings[]);
>  
>  /**
>   * Write strings (const char *) to the context.
> diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c
> index 76780bc852..d80b8527bb 100644
> --- a/libavformat/aviobuf.c
> +++ b/libavformat/aviobuf.c
> @@ -1400,7 +1400,7 @@ int avio_printf(AVIOContext *s, const char *fmt, ...)
>      return ret;
>  }
>  
> -void avio_print_string_array(AVIOContext *s, const char *strings[])
> +void avio_print_string_array(AVIOContext *s, const char *const strings[])
>  {
>      for(; *strings; strings++)
>          avio_write(s, (const unsigned char *)*strings, strlen(*strings));

Will apply this patchset tomorrow unless there are objections.

- Andreas

_______________________________________________
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".