It might add more complexity to try to work on the same variable, as I quickly
looked over the commit of the PR, one field is inside a cfg-param reload
structure, which is trickier to figure out if it was set because of a modparam
or because of a rpc command and then know which parameter name was used to set
it. I think it will bring more overhead in code maintenance than two variables
(or two fields in the cfg-param reload structure).
If two variables/fields is considered overhead, I would rather suggest to
remove the old second-based timeout completely and add the new one with a
different name, so on upgrading to the next release, the config has to be
updated as well.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3609#issuecomment-1773066161
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3609/c1773066...@github.com>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
