Hello,
I noticed that erlang module is exporting couple of config variable
classes (pseudo-variables) with various names. I would suggest to prefix
them with 'erl' or something similar to know they are related and
specific to erlang module. Many modules do so already (e.g., tm uses
$T...). You used some generic names that could be confusing at some
point with other components (e.g., 'tuple' is also used a lot in
presence, 'pid' is something common any process and kamailio has also
some variable related to it).
Another note: documenting PVs in README is ok, but then need to be also
documented in the wiki cookbook:
- http://www.kamailio.org/wiki/cookbooks/devel/pseudovariables
Cheers,
Daniel
--
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Kamailio World Conference, May 27-29, 2015
Berlin, Germany - http://www.kamailioworld.com
Hello,
I am planning to branch the code for upcoming major release on next
Monday, May 25. Testing went fine so far in my side and I don't expect
new consistent changes. Any fixes afterwards will be backported as usual
to a stable branch. From that moment, master branch will be open again
for new features.
If someone else has different proposal, let's discuss it on sr-dev
mailing list.
Cheers,
Daniel
--
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Kamailio World Conference, May 27-29, 2015
Berlin, Germany - http://www.kamailioworld.com