[FFmpeg-devel] [PATCH] avformat/rtsp: Put strings instead of pointers to strings into array

[Ross Nicholson]

> On 20 Apr 2020, at 02:42, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
> 
> In this example, the difference in length between the shortest and
> longest string is three, so that not using pointers to strings saves
> space even on 32bit systems.
> 
> Moreover, there is no need to use a sentinel here; it can be replaced
> with FF_ARRAY_ELEMS.

Thanks, again.

> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
> I have to admit that this is untested.
> 
> libavformat/rtsp.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
> index 0a6462000d..b2b3f32011 100644
> --- a/libavformat/rtsp.c
> +++ b/libavformat/rtsp.c
> @@ -2526,10 +2526,11 @@ static int rtp_read_header(AVFormatContext *s)
> 
>     p = strchr(s->url, '?');
>     if (p) {
> -        static const char *filters[][2] = {{"sources", "incl"}, {"block", "excl"}, {NULL, NULL}};
> +        static const char filters[][2][8] = { { "sources", "incl" },
> +                                              { "block",   "excl" } };
>         int i;
>         char *q;
> -        for (i = 0; filters[i][0]; i++) {
> +        for (i = 0; i < FF_ARRAY_ELEMS(filters); i++) {
>             if (av_find_info_tag(filters_buf, sizeof(filters_buf), filters[i][0], p)) {
>                 q = filters_buf;
>                 while ((q = strchr(q, ',')) != NULL)
> -- 
> 2.20.1
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".