Re: [PATCH] diagnostics: Add %{...%} pretty-format support for URLs and use it in -Wpsabi diagnostcs

[Richard Sandiford]

Jakub Jelinek via Gcc-patches <gcc-patches@gcc.gnu.org> writes:
> @@ -5747,16 +5748,18 @@ rs6000_discover_homogeneous_aggregate (m
>                 unsigned uid = TYPE_UID (TYPE_MAIN_VARIANT (type));
>                 if (uid != last_reported_type_uid)
>                   {
> +                   const char *url
> +                     = get_changes_url ("gcc-10/changes.html#empty_base");
>                     if (empty_base_seen & 1)
>                       inform (input_location,
>                               "parameter passing for argument of type %qT "
>                               "when C++17 is enabled changed to match C++14 "
> -                             "in GCC 10.1", type);
> +                             "in %{GCC 10.1%}", type, url);
>                     else
>                       inform (input_location,
>                               "parameter passing for argument of type %qT "
>                               "with %<[[no_unique_address]]%> members "
> -                             "changed in GCC 10.1", type);
> +                             "changed in %{GCC 10.1%}", type, url);
>                     last_reported_type_uid = uid;
>                   }
>               }

Looks like there's a missing free() for this one.

Thanks,
Richard