Thank you for the your interest in Kamailio and the module in development. I did a quick high level look today and have a few comments:
- can you give a small description what your module actually does? :-)
- a huge part of your code is related to the asn1 functionality. Some of this is library code that could be maybe placed elsewhere, some of them is generated from a ASN1 definition. If you finished the development we should discuss how to deal with this. Maybe something can be also just add as a build dependency and don't need to be included.
- there is a cmake file inside, do you use cmake for something? In Kamailio we use standard make for building
- you just pushed a change to the htable module, this should be a separate pull request as its isolated from your module
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.