RFR: 7902652: Webrev: prints "awk: cmd. line:8: warning: regexp escape sequence `\@' is not a known regexp operator"

[Langer, Christoph]

I only made it to author there, so somebody else needs to sponsor it ��

/Christoph

> -----Original Message-----
> From: Stefan Karlsson <stefan.karlsson at oracle.com>
> Sent: Donnerstag, 23. April 2020 09:22
> To: Langer, Christoph <christoph.langer at sap.com>; code-tools-
> dev at openjdk.java.net
> Subject: Re: RFR: 7902652: Webrev: prints "awk: cmd. line:8: warning: regexp
> escape sequence `\@' is not a known regexp operator"
> 
> Thanks, Christoph.
> 
> I'm merely a jdk Reviewer and have no status in the code-tools project
> :) Could someone sponsor this change?
> 
> Thanks,
> StefanK
> 
> On 2020-04-23 07:47, Langer, Christoph wrote:
> > Hi Stefan,
> >
> > thanks for fixing this... it annoyed me in my Windows/Cygwin setup a lot
> ��
> >
> > Fix looks good and I can confirm that it works.
> >
> > Best regards
> > Christoph
> >
> >> -----Original Message-----
> >> From: code-tools-dev <code-tools-dev-bounces at openjdk.java.net> On
> >> Behalf Of Stefan Karlsson
> >> Sent: Montag, 20. April 2020 10:29
> >> To: code-tools-dev at openjdk.java.net
> >> Subject: RFR: 7902652: Webrev: prints "awk: cmd. line:8: warning: regexp
> >> escape sequence `\@' is not a known regexp operator"
> >>
> >> Hi all,
> >>
> >> Please review this patch to get rid of some warnings from webrev:
> >>
> >> https://cr.openjdk.java.net/~stefank/CODETOOLS-7902652/webrev.01/
> >> https://bugs.openjdk.java.net/browse/CODETOOLS-7902652
> >>
> >> After an update to my system I got awk 5.0.1, and now webrev complains
> >> about:
> >> "awk: cmd. line:8: warning: regexp escape sequence `\@' is not a known
> >> regexp operator"
> >>
> >> A couple of @'s seems to be incorrectly escaped. This patch replaces
> >> \@\@ with @@.
> >>
> >> I've generated webrevs with:
> >> - awk 4.2.1 without fix
> >> - awk 4.2.1 with fix
> >> - awk 5.0.1 without fix
> >> - awk 5.0.1 with fix
> >>
> >> and verified that they all generate the same output (except some
> >> timestamps).
> >>
> >> Thanks,
> >> StefanK