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: [try2] PR28943 Unusable error message when using a conditional-expression with multiple type arguments


"Manuel López-Ibáñez" <lopezibanez@gmail.com> writes:

| On 31 Jan 2007 04:15:12 -0600, Gabriel Dos Reis <gdr@cs.tamu.edu> wrote:
| > "Manuel López-Ibáñez" <lopezibanez@gmail.com> writes:
| >
| > I would prefer the "improved" diagnostic to say something like
| >
| >    third operand to the conditional operator is of type 'void', but
| >    the second operand is neither a throw-expression nor of type 'void'
| >
| > and swap second/third the other around to cover the symmetric case.
| >
| 
| Here is the improved patch. Bootstrapped and regression tested. I hope
| this is what you wanted.

Please split the long lines into something much shorter, taking
advantage of automatic string concatenaion -- see the coding standard.

The testcase is more verbose than it needs to be.

OK with those changes.

-- Gaby


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