Thread (13 messages) flat view 13 messages, 7 authors, 2017-07-12

Re: [PATCH] strbuf: use designated initializers in STRBUF_INIT

From: brian m. carlson <hidden>
Date: 2017-07-12 01:26:44

On Tue, Jul 11, 2017 at 07:24:07AM +0200, Johannes Sixt wrote:
Am 11.07.2017 um 02:05 schrieb brian m. carlson:
quoted
I have tried compiling Git with a C++ compiler, so that I could test
whether that was a viable alternative for MSVC in case its C++ mode
supported features its C mode did not.  Let's just say that the
compilation aborted very quickly and I gave up after a few minutes.
It's 3 cleanup patches and one hacky patch with this size:

 80 files changed, 899 insertions(+), 807 deletions(-)

to compile with C++. It passed the test suite last time I tried. Getting rid
of the remaining 1000+ -fpermissive warnings is a different matter, though.
Yeah, that's the size I was thinking.  My goal was "easily achievable in
half an hour," which it didn't seem like at the time.
I can revive the patches if there is interest.
I'd be interested in at least a pointer to them if you have one.  I
think it might allow us to take advantage of desirable features that are
in the intersection of C99 and C++.
-- 
brian m. carlson / brian with sandals: Houston, Texas, US
https://www.crustytoothpaste.net/~bmc | My opinion only
OpenPGP: https://keybase.io/bk2204

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help