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] Fix PR c++/17609 on the 3.4 branch


On 20 Sep, Gabriel Dos Reis wrote:

> The testcase is OK for mainline, but I would rather not have the patch
> for 3.4.x for the moment.

Does that mean that you would rather postpone it to 3.4.6?

Just one point to consider:
The bogus error message is a really nasty one: It appears in places
that are not in the near neighborhood of the first (correct) error.
There can be lots of lines of code and lots of error messages in
between.

Trying to fix code that is not broken can frustrate users a lot.

> I'm a bit nervous about patches that touch the name lookup code.

I can understand that.

> I eventually convinced myself that the only way we could introduce a new
> variable is if the code already contained an error.

That is my understanding, too.

Regards,
Volker



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