Date: Tue, 14 Mar 2023 22:09:29 -0600 From: Warner Losh <imp@bsdimp.com> To: Jessica Clarke <jrtc27@freebsd.org>, Elyes Haouas <ehaouas@noos.fr> Cc: Warner Losh <imp@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@freebsd.org>, "dev-commits-src-main@freebsd.org" <dev-commits-src-main@freebsd.org> Subject: Re: git: 44357c18ccb9 - main - ppp: Remove trailing semicolon Message-ID: <CANCZdfoHsZrPJP-NmgPw1rtBc4M1FwPiBKhrLC-xiEW87=M39Q@mail.gmail.com> In-Reply-To: <27233978-E3A2-46BD-A2A9-BA1854833129@freebsd.org> References: <202303150239.32F2dj2H060188@gitrepo.freebsd.org> <27233978-E3A2-46BD-A2A9-BA1854833129@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000004c630c05f6e8833b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, Mar 14, 2023 at 9:45=E2=80=AFPM Jessica Clarke <jrtc27@freebsd.org>= wrote: > On 15 Mar 2023, at 02:39, Warner Losh <imp@FreeBSD.org> wrote: > > > > The branch main has been updated by imp: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=3D44357c18ccb9df24a925d43ccc467f6= 5b9ed6bc6 > > > > commit 44357c18ccb9df24a925d43ccc467f65b9ed6bc6 > > Author: Elyes Haouas <ehaouas@noos.fr> > > AuthorDate: 2023-03-15 02:09:16 +0000 > > Commit: Warner Losh <imp@FreeBSD.org> > > CommitDate: 2023-03-15 02:39:34 +0000 > > > > ppp: Remove trailing semicolon > > > > Signed-off-by: Elyes Haouas <ehaouas@noos.fr> > > Reviewed by: imp > > Pull Request: https://github.com/freebsd/freebsd-src/pull/654 > > --- > > usr.sbin/ppp/ncp.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/usr.sbin/ppp/ncp.h b/usr.sbin/ppp/ncp.h > > index 70b00a422310..806c943a891e 100644 > > --- a/usr.sbin/ppp/ncp.h > > +++ b/usr.sbin/ppp/ncp.h > > @@ -96,8 +96,8 @@ extern void ncp2initial(struct ncp *); > > ncp_ClearUrgentPorts(&(ncp)->cfg.urgent.tcp) > > #define ncp_ClearUrgentUdpPorts(ncp) \ > > ncp_ClearUrgentPorts(&(ncp)->cfg.urgent.udp) > > -#define ncp_ClearUrgentTOS(ncp) (ncp)->cfg.urgent.tos =3D 0; > > -#define ncp_SetUrgentTOS(ncp) (ncp)->cfg.urgent.tos =3D 1; > > +#define ncp_ClearUrgentTOS(ncp) (ncp)->cfg.urgent.tos =3D 0 > > +#define ncp_SetUrgentTOS(ncp) (ncp)->cfg.urgent.tos =3D 1 > > do { ... } while (0)? > That would likely be better... I'll leave that to the original submitter (cc'd) to do as a refinement and submit a followup... I'm not sure how to code automatic detection of this into my script that checks for style(9) compliance and I should have tweaked to this when I Was reviewing it... Thanks Warner > Jess > > > > > #ifndef NOINET6 > > #define isncp(proto) ((proto) =3D=3D PROTO_IPCP || (proto) =3D=3D PROTO= _IPV6CP) > > --0000000000004c630c05f6e8833b Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote">= <div dir=3D"ltr" class=3D"gmail_attr">On Tue, Mar 14, 2023 at 9:45=E2=80=AF= PM Jessica Clarke <<a href=3D"mailto:jrtc27@freebsd.org">jrtc27@freebsd.= org</a>> wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"marg= in:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1e= x">On 15 Mar 2023, at 02:39, Warner Losh <imp@FreeBSD.org> wrote:<br> > <br> > The branch main has been updated by imp:<br> > <br> > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D44357c18ccb9= df24a925d43ccc467f65b9ed6bc6" rel=3D"noreferrer" target=3D"_blank">https://= cgit.FreeBSD.org/src/commit/?id=3D44357c18ccb9df24a925d43ccc467f65b9ed6bc6<= /a><br> > <br> > commit 44357c18ccb9df24a925d43ccc467f65b9ed6bc6<br> > Author:=C2=A0 =C2=A0 =C2=A0Elyes Haouas <<a href=3D"mailto:ehaouas@= noos.fr" target=3D"_blank">ehaouas@noos.fr</a>><br> > AuthorDate: 2023-03-15 02:09:16 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Warner Losh <imp@FreeBSD.org><br> > CommitDate: 2023-03-15 02:39:34 +0000<br> > <br> >=C2=A0 =C2=A0 ppp: Remove trailing semicolon<br> > <br> >=C2=A0 =C2=A0 Signed-off-by: Elyes Haouas <<a href=3D"mailto:ehaouas= @noos.fr" target=3D"_blank">ehaouas@noos.fr</a>><br> >=C2=A0 =C2=A0 Reviewed by: imp<br> >=C2=A0 =C2=A0 Pull Request: <a href=3D"https://github.com/freebsd/freeb= sd-src/pull/654" rel=3D"noreferrer" target=3D"_blank">https://github.com/fr= eebsd/freebsd-src/pull/654</a><br> > ---<br> > usr.sbin/ppp/ncp.h | 4 ++--<br> > 1 file changed, 2 insertions(+), 2 deletions(-)<br> > <br> > diff --git a/usr.sbin/ppp/ncp.h b/usr.sbin/ppp/ncp.h<br> > index 70b00a422310..806c943a891e 100644<br> > --- a/usr.sbin/ppp/ncp.h<br> > +++ b/usr.sbin/ppp/ncp.h<br> > @@ -96,8 +96,8 @@ extern void ncp2initial(struct ncp *);<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0ncp_ClearUrgentPorts(&(ncp= )->cfg.urgent.tcp)<br> > #define ncp_ClearUrgentUdpPorts(ncp) \<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0ncp_ClearUrgentPorts(&(ncp= )->cfg.urgent.udp)<br> > -#define ncp_ClearUrgentTOS(ncp) (ncp)->cfg.urgent.tos =3D 0;<br> > -#define ncp_SetUrgentTOS(ncp) (ncp)->cfg.urgent.tos =3D 1;<br> > +#define ncp_ClearUrgentTOS(ncp) (ncp)->cfg.urgent.tos =3D 0<br> > +#define ncp_SetUrgentTOS(ncp) (ncp)->cfg.urgent.tos =3D 1<br> <br> do { ... } while (0)?<br></blockquote><div><br></div><div>That would likely= be better...=C2=A0 I'll leave that to the original submitter (cc'd= ) to do as a refinement and submit a followup...</div><div>I'm not sure= how to code automatic detection of this into my script that checks for sty= le(9) compliance</div><div>and I should have tweaked to this when I Was rev= iewing it...</div><div><br></div><div>Thanks</div><div><br></div><div>Warne= r<br></div><div>=C2=A0</div><blockquote class=3D"gmail_quote" style=3D"marg= in:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1e= x"> Jess<br> <br> > <br> > #ifndef NOINET6<br> > #define isncp(proto) ((proto) =3D=3D PROTO_IPCP || (proto) =3D=3D PROT= O_IPV6CP)<br> <br> </blockquote></div></div> --0000000000004c630c05f6e8833b--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfoHsZrPJP-NmgPw1rtBc4M1FwPiBKhrLC-xiEW87=M39Q>