On Tue, Jan 28, 2020 at 09:30:17AM -0500, David Malcolm wrote:
>       * diagnostic-core.h (warning_at): Rename overload to...
>       (warning_with_metadata_at): ...this.
I think this is just too long, especially for a function used at least
in the analyzer pretty often, leading to lots of ugly formatting.
Can't you use warning_meta or something similar instead?

>       (emit_diagnostic_valist): Delete decl of overload taking
>       diagnostic_metadata.
>       * diagnostic.c (emit_diagnostic_valist): Likewise for defn.
>       (warning_at): Rename overload taking diagnostic_metadata to...
>       (warning_with_metadata_at): ...this.

        Jakub

Reply via email to