[trivial PATCH 01/22] [libradius.h] eliminate build warnings

Alan DeKok aland at deployingradius.com
Tue Feb 2 00:09:54 CET 2010


Wang Tinggong wrote:
> hi, this is a series of trivial patch for eliminating build warnings.

 Here's my review:

Patches 01, (and related)
        19, 20
 - Don't put changes to a function prototype in a *separate* patch
   from changes to the source file.  This breaks the build.

Patches 11, 12, 13, 14, 15, 16
 - OTP code.  I'll let Frank review them.

Patches 05, 06, 07, 08, 10,
 - adding function prototypes to remove warnings is good.
   But they belong in a header file, not in a C file.

  Many of the rest are minor, and have no impact other than messages
when building.  i.e. these are low priority items.

  Alan DeKok.



More information about the Freeradius-Devel mailing list