On 17/09/14 13:01, Olle E. Johansson wrote:

On 17 Sep 2014, at 12:54, Daniel-Constantin Mierla <miconda@gmail.com> wrote:

Hello,

historically 'tm' stays for 'transaction management' -- the the x for the other module was for eXtensions.

The new module doesn't really do any management of transaction, but store structures that include the id of transactions with custom info (at this moment an AoR). It has a completely different hash table in memory (so nothing new in tm structure). Internally it binds to tm to get some callbacks executed, but also to registrar module for getting access to new branches.

Being in relation only with Transaction, storing some extra info for it, not sure the 'm' makes sense to add.

The whole idea seems to be to manipulate a transaction - add a branch to an existing transaction, so the "m" certainly makes sense to me... ;-)

Well, somehow around that but not exactly :-) -- because the function doing that is in tm, added by commit: 468a2c645f5fcb66376b81b06e69eb8f5377bb02

The tsilo module just calls it internally via tm api. The module is managing an internal hash table, acting on some callbacks from tm and calling functions from registrar and tm. There are many other modules calling functions from tm (e.g., dialog, msilo, ...).

But maybe the functions of the module should be renamed to use the prefix ts_ instead of t_.

/O
_______________________________________________
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

--
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Next Kamailio Advanced Trainings 2014 - http://www.asipto.com
Sep 22-25, Berlin, Germany


_______________________________________________
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

--
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Next Kamailio Advanced Trainings 2014 - http://www.asipto.com
Sep 22-25, Berlin, Germany


_______________________________________________
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev