[Bug 269] Many compiler warnings with gcc 4.0
aland at ox.org
Thu Sep 1 19:08:33 CEST 2005
Paul TBBle Hampson <Paul.Hampson at Pobox.com> wrote:
> Can we throw a couple of #defines in there too?
> #define vp_strval data.strvalue
> #define vp_octets data.octets
> That way vp->vp_strval is a direct substitute for vp->strvalue. It carries a
> risk of namespace pollution though.
Try using "st_mtime" in your code. The headers for the stat()
routine do the same thing.
> (I'm not too fussed, I'm as happy as not with unions, but I know
> some people dislike the extra component.)
> Otherwise, an anonymous union might do it, if I recall correctly?
I'm not sure how portable that is.
In any case vp->vp_strvalue looks fine to me. It also lets us do
vp->vp_ipv6addr and other things, which would be very good.
More information about the Freeradius-Devel