On 17 Jan 2015, at 14:43, Robert Boisvert <rdboisv...@gmail.com> wrote:

> Olle,
> 
> Please address your concerns to me so I can look into them.
Hi!
Thanks for a quick response!
- LM_DBG and LM_ERR needs to be terminated with \n
I did fix a few - but there's a lot more fixing to be done.

See how mohqueue stacks error messages on top of each other here where they 
should be on one line each.

 0(92166) DEBUG: <core> [sr_module.c:731]: find_mod_export(): <db_query_lock> 
in module <db_mysql> not found
 0(92166) ERROR: <core> [db.c:73]: db_check_api(): module db_mysql does not 
export db_use_table function. Please check if module is loaded.
 0(92166) ERROR: mohqueue [mohq.c:236]: init_db(): Unable to bind DB API using 
mysql://kamailio:kamailiorw@localhost/kamailio 0(92166) ERROR: <core> 
[sr_module.c:945]: init_mod(): Error while initializing module mohqueue 
(/usr/local/lib64/kamailio/modules/mohqueue.so)
ERROR: error while initializing modules
 0(92166) DEBUG: xprint [xprint.c:174]: destroy(): destroy module ...


It may be my personal opinion, but comparing the source code formatting in the 
mohqueue
source code files with other modules, it is not very well formatted. 

Regards,
/O

> 
> Thanks,
> Bob
> 
> On Sat, Jan 17, 2015 at 6:34 AM, Olle E. Johansson <o...@edvina.net> wrote:
> Module: kamailio
> Branch: master
> Commit: 6daf77d666ff0fcb1bd8df4bb3c63dc0fdac9656
> URL: 
> https://github.com/kamailio/kamailio/commit/6daf77d666ff0fcb1bd8df4bb3c63dc0fdac9656
> 
> Author: Olle E. Johansson <o...@edvina.net>
> Committer: Olle E. Johansson <o...@edvina.net>
> Date: 2015-01-17T12:33:45+01:00
> 
> mohqueue Remove SVN IDs, add some \n at end of error messages
> 
> More cleanup of LM_DBG and LM_ERR needed, as well as code formatting
> 
> ---
> 
> Modified: modules/mohqueue/mohq.c
> Modified: modules/mohqueue/mohq.h
> Modified: modules/mohqueue/mohq_common.h
> Modified: modules/mohqueue/mohq_db.c
> Modified: modules/mohqueue/mohq_db.h
> Modified: modules/mohqueue/mohq_funcs.c
> Modified: modules/mohqueue/mohq_funcs.h
> Modified: modules/mohqueue/mohq_locks.c
> Modified: modules/mohqueue/mohq_locks.h
> 
> ---
> 
> Diff:  
> https://github.com/kamailio/kamailio/commit/6daf77d666ff0fcb1bd8df4bb3c63dc0fdac9656.diff
> Patch: 
> https://github.com/kamailio/kamailio/commit/6daf77d666ff0fcb1bd8df4bb3c63dc0fdac9656.patch
> 
> 
> _______________________________________________
> sr-dev mailing list
> sr-dev@lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
> 
> _______________________________________________
> sr-dev mailing list
> sr-dev@lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

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

Reply via email to