This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[Diagnostic Patch] Clean-up diagnostic facilities in diagnostic.c

From: Paolo Carlini <paolo dot carlini at oracle dot com>
To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Cc: Jason Merrill <jason at redhat dot com>, David Malcolm <dmalcolm at redhat dot com>, Dodji Seketeli <dodji at redhat dot com>
Date: Mon, 6 Jun 2016 10:55:21 +0200
Subject: [Diagnostic Patch] Clean-up diagnostic facilities in diagnostic.c
Authentication-results: sourceware.org; auth=none

Hi,

yesterday I had the idea of this small clean-up: move the work done byemit_diagnostic to a new non-variadic diagnostic_impl and use it toimplement the former and all the various inform, warning, permerror, etc(lately we have the *_at_rich_loc variants too). Something similar canbe done for inform_n, warning_n and errror_n. There is the minorsubtlety of the declarations decorated with ATTRIBUTE_GCC_DIAG whenavailable to suppress the build-time warning: I think I did it the rightway, I took inspiration from the declarations of diagnostic_set_info,etc, in diagnostic.h.


Tested x86_64-linux.

Thanks,
Paolo.

////////////////////////////////

Attachment: CL_diagnostic
Description: Text document

Attachment: patch_diagnostic
Description: Text document

Follow-Ups:
- Re: [Diagnostic Patch] Clean-up diagnostic facilities in diagnostic.c
  - From: David Malcolm

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]