Re: [PATCH 00/37] removal of some c++ keywords

From: Johannes Sixt <j6t@kdbg.org>
Date: Jan 30, 2018, 20:54 UTC
Message-ID: <cd708bfc-6f92-18d8-9121-ed413748f14a@kdbg.org>
In-Reply-To: <20180129223728.30569-1-bmwill@google.com>

Am 29.01.2018 um 23:36 schrieb Brandon Williams:

Show 8 quoted lines

> A while back there was some discussion of getting our codebase into a state
> where we could use a c++ compiler if we wanted to (for various reason like
> leveraging c++ only analysis tools, etc.).  Johannes Sixt had a very large
> patch that achieved this but it wasn't in a state where it could be upstreamed.
> I took that idea and did some removals of c++ keywords (new, template, try,
> this, etc) but broke it up into several (well maybe more than several) patches.
> I don't believe I've captured all of them in this series but this is at least
> moving a step closer to being able to compile using a c++ compiler.

Cool! The patches all look reasonable. Some keywords remain: 'delete', 'private', 'thread_local', 'not', 'xor', 'explicit', 'export'.

> I don't know if this is something the community still wants to move towards,
> but if this is something people are still interested in, and this series is
> wanted, then we can keep doing these sort of conversions in chunks slowly.

I've rebased my patches on top of this series. They are available from

   https://github.com/j6t/git.git c-plus-plus

-- Hannes