Re: [patchv2] compile: Fix crash on cv-qualified self-reference

Pedro Alves
2015-07-01T16:16:50Z

On 07/01/2015 05:07 PM, Jan Kratochvil wrote:
> On Wed, 01 Jul 2015 17:35:10 +0200, Pedro Alves wrote:
>> On 07/01/2015 04:29 PM, Jan Kratochvil wrote:
>>> From 1 mandatory macro 3 mandatory macros and breaking all 3rd party patches.
>>> When GDB codebase finally gets compiled by clang++ I agree it will be an
>>> improvement.
>>
>> Seriously, that's a depressing and demotivating comment.
>> You want it done, but you also want it not done.
>
> Do you mean the first sentence or the second sentence?
>

The whole of it. If we had converted to pure C++ in one go, you'd have
had to revolve the same conflicts too. And the conflicts resulting from
the final conversion from TRY/CATCH to native try/catch will be super
trivial to resolve, as that won't move code around.

> I agree the first sentence is demotivating but I feel the same spending a lot
> of time rebasing patches again and again on changes that end up half-way or
> that are questionable whether they really cleaned up the code. Although
> specifically for this TRY_CATCH->TRY case I had fortunately only two such
> cases.

And still, you complain.

This is truly pointless. Sorry, I'm out of here.

Thanks,
Pedro Alves