From 3d0f10e1e814f1ce2e13465c70f1f5c5d70883a0 Mon Sep 17 00:00:00 2001 From: Sean Bright Date: Fri, 7 Nov 2025 17:17:28 -0500 Subject: [PATCH] bridge_simple: Increase code verbosity for clarity. There's no actual problem here, but I can see how it might by confusing. Resolves: #1444 --- bridges/bridge_simple.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/bridges/bridge_simple.c b/bridges/bridge_simple.c index 4e27b9f540..a7ce3a000c 100644 --- a/bridges/bridge_simple.c +++ b/bridges/bridge_simple.c @@ -157,11 +157,13 @@ static int simple_bridge_join(struct ast_bridge *bridge, struct ast_bridge_chann /* The ast_channel_hold_state() and ast_channel_name() accessors need to be * called with the associated channel lock held. */ - if ((unhold_c1 = ast_channel_hold_state(c1) == AST_CONTROL_HOLD)) { + unhold_c1 = (ast_channel_hold_state(c1) == AST_CONTROL_HOLD); + if (unhold_c1) { ast_debug(1, "Channel %s simulating UNHOLD for bridge simple join.\n", ast_channel_name(c1)); } - if ((unhold_c0 = ast_channel_hold_state(c0) == AST_CONTROL_HOLD)) { + unhold_c0 = (ast_channel_hold_state(c0) == AST_CONTROL_HOLD); + if (unhold_c0) { ast_debug(1, "Channel %s simulating UNHOLD for bridge simple join.\n", ast_channel_name(c0)); }