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]

Re: [PATCH 0/4] (v2) Generating patches from fix-it hints


On 08/24/2016 07:13 PM, David Malcolm wrote:
Here's a much less ambitious version of the patch kit, which
eliminates any attempt to write to the user's source
code (getting rid of edit_context::apply_changes and
 -fdiagnostics-apply-fixits).

It implements -fdiagnostics-generate-patch.  In so doing, it
tightens up the exact semantics of fix-its; see [1] for an
example of where that's useful.

I need review of at least patches 1 and 2 (which are unchanged
from v1 of the kit).  I believe I can self-approve patches 3
and 4 as part of my "diagnostics maintainer" role; are they
acceptable to those who objected to the earlier kit? (now
that there's no attempt to write to source files).

Successfully bootstrapped&regrtested the combination of the patches
on x86_64-pc-linux-gnu.

OK for trunk? (assuming individual bootstraps&regrtesting)

[1] https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01751.html
My understanding was #1 was approved by Bernd. I think the update to #1 which removes the unnecessary explicit namespaces is fine.

Have you addressed the question/concern for #2?

https://gcc.gnu.org/ml/gcc-patches/2016-08/msg02125.html

Jeff


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