On Thu, Nov 30, 2023 at 12:39:22PM -0500, Marek Polacek wrote:
> On Thu, Nov 30, 2023 at 06:37:21PM +0100, Florian Weimer wrote:
> > * Marek Polacek:
> > 
> > >> +void
> > >> +implicit_function_declaration (void)
> > >> +{
> > >> +  f1 (); /* { dg-warning "'f1' \\\[-Wimplicit-function-declaration\\\]" 
> > >> } */
> > >> +}
> > >> +
> > >> +extern implicit_int_1; /* { dg-warning "'implicit_int_1' 
> > >> \\\[-Wimplicit-int\\\]" } */
> > >
> > > Oy, these \ tend to get unwieldy.  You could probably just say
> > > { dg-warning {-Wimplicit-int} }
> > 
> > I wanted to have some more context for the other files that might get
> > the explict line number wrong.
> > 
> > >> +    return incompatible_pointer_types; /* { dg-error "returning 'int 
> > >> \\\* \\\(\\\*\\\)\\\(int\\\)' from a function with incompatible return 
> > >> type 'int \\\*' \\\[-Wincompatible-pointer-types\\\]" } */
> > >
> > > And here maybe
> > > { dg-error {returning 'int \* \(\*\)\(int\)' from a function with 
> > > incompatible return type 'int \*' \[-Wincompatible-pointer-types\]} }
> > > could work the same.  But you don't have to go and change it; I don't
> > > want to make more work for you.
> > 
> > Oh, I'm not too familiar with Tcl.  I haven't seen single quotes being
> > used elsewhere.  I prefer not to change it unless there is another
> > reason to rework all this.  So … let's wait?
> 
> Sure.

Note, I think \\\s are used in thousands of dg-error/dg-warning etc.
in other tests already, I think it isn't a big deal.
find testsuite -type f | xargs grep 'dg-\(warning\|error\|message\).*".*\\\\\\' 
| wc -l
4782
Sure, with using {} instead of "" one can get rid of some of the \s.

        Jakub

Reply via email to