rlm_pap code tidy

Alan DeKok aland at deployingradius.com
Sat Sep 29 06:56:29 CEST 2012


Matthew Newton wrote:
> 52b68318 -
>   moves header-like stuff into rlm_pap.h

  That's my only concern.  Moving C structs into header files is bad form.

> It now uses a function pointers to select the right auth function
> depending on the config attribute found. I was slightly worried
> that calling functions on this might slow things down a bit.
> However, running 1000 auths in a tight loop (non-threaded server,
> debug off) shows essentially no difference on my system - over
> three runs, old code avg 23.516s, new code avg 23.464s.

  Yeah.  It's better that way.  The old code was... interesting.

  Alan DeKok.


More information about the Freeradius-Devel mailing list