Re: [PATCH 1/7] Add MM Auth test; add auth_action_str() function

Hi Neels,

* Use struct value_string[] instead of switch{} for auth_action_str().

thanks, but... On Wed, Mar 30, 2016 at 12:45:36AM +0200, Neels Hofmeyr wrote:

+static inline const char *auth_action_str(enum auth_action a) +{ +#define AUTH_CASE(X) \ + case X: return #X

am I missing something? Cheers, Harald -- - Harald Welte <laforge@gnumonks.org> http://laforge.gnumonks.org/ ============================================================================ "Privacy in residential applications is a desirable marketing option." (ETSI EN 300 175-7 Ch. A6)