Re: [PATCH] libcpp, v2: Add -Wtrailing-whitespace= warning

2024-10-11 Thread Joseph Myers
On Thu, 19 Sep 2024, Jakub Jelinek wrote: > Here is a patch which currently allows blank (' ' '\t') and space (' ' '\t' > '\f' '\v'), cntrl not yet added, not anything non-ASCII, but in theory could > be added later (though, non-ASCII would be just for inside of comments, > say non-breaking space

Re: [PATCH] libcpp, v2: Add -Wtrailing-whitespace= warning

2024-09-19 Thread Eric Gallager
On Thu, Sep 19, 2024 at 4:35 PM Jakub Jelinek wrote: > > On Thu, Sep 19, 2024 at 08:17:24AM +0200, Richard Biener wrote: > > On Wed, Sep 18, 2024 at 7:33 PM Jakub Jelinek wrote: > > > > > > On Wed, Sep 18, 2024 at 06:17:58PM +0100, Richard Sandiford wrote: > > > > +1 I'd much rather learn about

[PATCH] libcpp, v2: Add -Wtrailing-whitespace= warning

2024-09-19 Thread Jakub Jelinek
On Thu, Sep 19, 2024 at 08:17:24AM +0200, Richard Biener wrote: > On Wed, Sep 18, 2024 at 7:33 PM Jakub Jelinek wrote: > > > > On Wed, Sep 18, 2024 at 06:17:58PM +0100, Richard Sandiford wrote: > > > +1 I'd much rather learn about this kind of error before the code reaches > > > a review tool :)