eap_tnc.c source not stricly C
I know good style says newbies should lurk before posting, but anyway: Is freeradius supposed to be C89? src/modules/rlm_eap/types/rlm_eap_tnc/eap_tnc.c Is full of C++ comments and C99isms. -- REALITY.SYS not found: Universe halted.
Andrew Hood wrote:
I know good style says newbies should lurk before posting, but anyway:
Is freeradius supposed to be C89?
It's supposed to be as portable as possible.
src/modules/rlm_eap/types/rlm_eap_tnc/eap_tnc.c
Is full of C++ comments and C99isms.
Yes. Most of those should be fixed. As always, patches are welcome. Alan DeKok.
Alan DeKok wrote:
Andrew Hood wrote:
I know good style says newbies should lurk before posting, but anyway:
Is freeradius supposed to be C89?
It's supposed to be as portable as possible.
src/modules/rlm_eap/types/rlm_eap_tnc/eap_tnc.c
Is full of C++ comments and C99isms.
Yes. Most of those should be fixed.
As always, patches are welcome.
OK -- REALITY.SYS not found: Universe halted.
Alan DeKok wrote:
Andrew Hood wrote:
I know good style says newbies should lurk before posting, but anyway:
Is freeradius supposed to be C89?
It's supposed to be as portable as possible.
src/modules/rlm_eap/types/rlm_eap_tnc/eap_tnc.c
Is full of C++ comments and C99isms.
Yes. Most of those should be fixed.
As always, patches are welcome.
I'm not going to subscribe to developers just to send one patch, especially with all the warnings for non-core developers. This removes those C99-isms and C++ comments which gcc+linux finds. -- REALITY.SYS not found: Universe halted.
Andrew Hood wrote:
Alan DeKok wrote:
Andrew Hood wrote:
I know good style says newbies should lurk before posting, but anyway:
Is freeradius supposed to be C89?
It's supposed to be as portable as possible.
src/modules/rlm_eap/types/rlm_eap_tnc/eap_tnc.c
Is full of C++ comments and C99isms.
Yes. Most of those should be fixed.
As always, patches are welcome.
I'm not going to subscribe to developers just to send one patch, especially with all the warnings for non-core developers.
For future info, there's always the bug tracker: http://bugs.freeradius.org I've got patches in via there.
participants (3)
-
Alan DeKok -
Andrew Hood -
Phil Mayers