git: effc8e57fbf0 - main - libalias: Style cleanup

Lutz Donnerhacke lutz at donnerhacke.de
Sat May 15 19:58:30 UTC 2021


On Sat, May 15, 2021 at 10:41:53PM +0300, Konstantin Belousov wrote:
> On Sat, May 15, 2021 at 07:01:07AM +0000, Lutz Donnerhacke wrote:
> > @@ -285,21 +283,20 @@ static int	UdpAliasOut(struct libalias *, struct ip *, int, int create);
> >  static int	TcpAliasIn(struct libalias *, struct ip *);
> >  static int	TcpAliasOut(struct libalias *, struct ip *, int, int create);
> >  
> > -static int
> > -IcmpAliasIn1(struct libalias *la, struct ip *pip)
> > -{
> > -
> > -	LIBALIAS_LOCK_ASSERT(la);
> >  /*
> >      De-alias incoming echo and timestamp replies.
> >      Alias incoming echo and timestamp requests.
> >  */
> > +static int
> > +IcmpAliasIn1(struct libalias *la, struct ip *pip)
> > +{
> > +	LIBALIAS_LOCK_ASSERT(la);
> >  	struct alias_link *lnk;
> >  	struct icmp *ic;
> And this is not in style.  Declarations must precede executable statements,
> even asserts.

Of course it was a style commit.

+ /*
+      De-alias incoming echo and timestamp replies.
+      Alias incoming echo and timestamp requests.
+  */
  static int
  IcmpAliasIn1(struct libalias *la, struct ip *pip)
  {
-
	LIBALIAS_LOCK_ASSERT(la);
- /*
-      De-alias incoming echo and timestamp replies.
-      Alias incoming echo and timestamp requests.
-  */
 	struct alias_link *lnk;


More information about the dev-commits-src-all mailing list