Hello,
load_tm() function uses a mixture of find_export and direct assignment to export the API for TM. I would like to update to the last one, since will cleanup the module exports structure and avoid casting problems when functions prototypes change.
Then I see that tm uses some defines for the names of the several functions exported to config, e.g., T_FORWARD_NONACK, ..., which will have no other usage, I would go for using directly the names in module exports - makes life easier when browsing the code and will become coherent.
Anyone against?
Thanks, Daniel