On Thu, Jun 16, 2022 at 03:28:58PM +0200, Theo Buehler wrote:
> On Thu, Jun 16, 2022 at 02:36:28PM +0200, Claudio Jeker wrote:
> > This diff kills external use of prefixlen2mask() and uses inet4applymask()
> > instead.  With this the IPv4 and IPv6 code is more similar.
> > Also I feel the code is a bit easier to read.
> >
> > Also kroute{,6}_match() is changed to take a struct bgpd_addr *.
> > This is another step towards removing lots of copy paste code.
> 
> I agree that it's easier to follow and cleaner after the diff.
> The tree you used to isn't quite up to date - there were some offsets.

That is quite possible.

> ok tb

Thanks

> [...]
> 
> > +const struct in_addr       inet4allone = { INADDR_BROADCAST };
> > +const struct in6_addr      inet6allone = {{{ 0xff, 0xff, 0xff, 0xff,
> > +     0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
> > +     0xff, 0xff, 0xff, 0xff }}};
> 
> Any reason not to make these static?

Oh, yeah I should. No need to export them. 

-- 
:wq Claudio

Reply via email to