From: Sean Bright <sean@seanbright.com>
Date: Fri, 7 Nov 2025 22:17:28 +0000 (-0500)
Subject: bridge_simple: Increase code verbosity for clarity.
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=33987ed966178dc49a07cee9c4cab919b992a3a0;p=thirdparty%2Fasterisk.git

bridge_simple: Increase code verbosity for clarity.

There's no actual problem here, but I can see how it might by
confusing.

Resolves: #1444
---

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));
 	}