On 20-04 13:21, Juha Heinanen wrote:
Andrei Pelinescu-Onciul writes:
My vote would be to merge the names (it should be a minimal effort to copy & paste and then change the name in the module interface).
i didn't understand how you merge names. will the resulting module have two names for the same function?
A special note regarding maxfwd: there seem to be other differences. At least a quick look at the ser version showed that it supports dynamic config changes (in this case max_limit can be changed anytime at runtime, e.g.: sercmd cfg.set_int_now maxfwd.max_limit 5).
i didn't find that in module readme. the policy should be that whatever is not documented, does not exist.
For new code maybe, but I wouldn't be so picky about existing code. I do not want to throw away stuff that already exists just because it is not documented.
Jan.