On 10/04/2017 10:44 AM, Jason Merrill wrote:
Hmm, I don't think we want to diagnose these; if a parameter-list
follows the parenthesized declarator, it isn't ambiguous.
True. I went with that approach
3 and 4 seem like false positives. The problematic cases are all ones
where the parenthes
On 10/04/2017 08:17 PM, Eric Gallager wrote:
On Tue, Oct 3, 2017 at 1:08 PM, Nathan Sidwell wrote:
[non-c++ people on CC, there's a reason ...]
This patch adds a new warning, concerning unnecessary parentheses on a
declaration. For instance:
prettyprinter (pp);
Could you check and see
On Tue, Oct 3, 2017 at 1:08 PM, Nathan Sidwell wrote:
> [non-c++ people on CC, there's a reason ...]
>
> This patch adds a new warning, concerning unnecessary parentheses on a
> declaration. For instance:
>prettyprinter (pp);
> That's a declaration of a pp variable of type prettyprinter -- no
On Wed, Oct 04, 2017 at 10:44:11AM -0400, Jason Merrill wrote:
> > 3) A couple of places do:
> >T (name // line break to avoid wrap
> > [LONGEXPR][OTHERLONGEXPR]);
> >
> > The parens aid the editor's formatter. The patch removes the parens, but I
> > can see there may be disagreement. I
On Tue, Oct 3, 2017 at 1:08 PM, Nathan Sidwell wrote:
> [non-c++ people on CC, there's a reason ...]
>
> This patch adds a new warning, concerning unnecessary parentheses on a
> declaration. For instance:
>prettyprinter (pp);
> That's a declaration of a pp variable of type prettyprinter -- no
