> Thanks for the pull request. Just inventing a new patch category is
> not the right approach, sorry.
>
> Please just use "core" for the core part and the other module parts as
> individual commits per module with the respective module name as
> prefix.


No problem, will do.

> One small question, why do you add this additional quotes and space
> around the TIME_T_INT_FMT #define?

You mean this, right?

LM_DBG("converting datetime value %" TIME_T_INT_FMT " to str\n", VAL_TIME(v));

Below wouldn't work:
LM_DBG("converting datetime value %TIME_T_INT_FMT to str\n", VAL_TIME(v));

Or maybe I didn't understand your question?

Kind regards,
Seb


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2894#issuecomment-949915253
_______________________________________________
Kamailio (SER) - Development Mailing List
[email protected]
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to