Should be fixed now. Just to be 100% safe - executing the clang-format in the Kamailio root will pick up the correct format, right?
Cheers,
Henning
-----Original Message----- From: Henning Westerholt Sent: Donnerstag, 18. Mai 2023 13:30 To: Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org Cc: Victor Seva linuxmaniac@torreviejawireless.org Subject: RE: [sr-dev] Re: git:master:61833193: dispatcher: fix force send socket functionality, was not working in some TCP scenarios
Hi Victor,
I was merging a patch that was finally verified yesterday evening in a test setup. Thanks also for the other reply for the thread. But your point is valid. Maybe we can use some git hooks or something, but it might have some unintended side effects.
Let me have a look to fix the format for that commit.
Cheers,
Henning
-----Original Message----- From: Victor Seva linuxmaniac@torreviejawireless.org Sent: Donnerstag, 18. Mai 2023 13:15 To: sr-dev@lists.kamailio.org Subject: [sr-dev] Re: git:master:61833193: dispatcher: fix force send socket functionality, was not working in some TCP scenarios
Hi,
On 18/5/23 12:54, Henning Westerholt wrote:
Module: kamailio Branch: master Commit: 6183319381573e42b882d05ae1748539f7547d8c URL: https://github.com/kamailio/kamailio/commit/6183319381573e42b882d05ae1748539...
We have to find a way to enforce clang-format. First commit after the huge merge... now it doesn't pass the check.
diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index f0322a6b57..e8d7886ec9 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -1869,7 +1869,8 @@ static inline int ds_push_dst( break; } if(sock) {
LM_DBG("set send socket to %.*s\n", sock->sock_str.len, sock->sock_str.s);
LM_DBG("set send socket to %.*s\n", sock->sock_str.len,sock->sock_str.s); set_force_socket(msg, sock); } return 0;