cvs commit: src/sbin/route route.c

Joerg Sonnenberger joerg at britannica.bec.de
Tue Mar 15 21:37:26 PST 2005


On Tue, Mar 15, 2005 at 09:21:36PM -0800, Chris Pressey wrote:
> > I think that should be
> > 
> >   splim = (u_short *)sa->sa_data +
> >       (sa->sa_len - offsetof(sa->sa_data)) / sizeof(u_short);
> > 
> > but that'd be the same as the thing I committed, just longer.
> 
> (With the exception of the "+ 1", which AFAICT is there so that the
> division by sizeof(u_short) (or ">> 1" in the old version) doesn't cut
> off the last short if sa_len is odd...)

You are right with the versions being functionally identical.
The compiler should create the same code, the difference is
which the limit is based on.

What I meant about the u_short usage is, that the hex printing should be
done byte wise, not word wise.

Joerg





More information about the Commits mailing list