[Asterisk-code-review] Remove const cast from leaf functions. (asterisk[master])

Joerg Sonnenberger asteriskteam at digium.com
Tue Jun 2 12:31:46 CDT 2015


Joerg Sonnenberger has uploaded a new change for review.

  https://gerrit.asterisk.org/564

Change subject: Remove const cast from leaf functions.
......................................................................

Remove const cast from leaf functions.

app_control_register_rule and app_control_unregister_rule lock/unlock
the queue, which is a mutating operation according to the
ao2_lock/_unlock prototype. Depending on the specific (implicit) casts
in SCOPED_LOCK and RAII_VAR, the compiler may warn or not. As the only
callers of those functions do not have the const, get consistent results
by just dropping it.

Change-Id: Ib9e6296155a39bc5d627142a3828180c3cfe8fbb
---
M res/stasis/control.c
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/64/564/1

diff --git a/res/stasis/control.c b/res/stasis/control.c
index 0f8b540..a61e961 100644
--- a/res/stasis/control.c
+++ b/res/stasis/control.c
@@ -135,7 +135,7 @@
 }
 
 static void app_control_register_rule(
-	const struct stasis_app_control *control,
+	struct stasis_app_control *control,
 	struct app_control_rules *list, struct stasis_app_control_rule *obj)
 {
 	SCOPED_AO2LOCK(lock, control->command_queue);
@@ -143,7 +143,7 @@
 }
 
 static void app_control_unregister_rule(
-	const struct stasis_app_control *control,
+	struct stasis_app_control *control,
 	struct app_control_rules *list, struct stasis_app_control_rule *obj)
 {
 	struct stasis_app_control_rule *rule;

-- 
To view, visit https://gerrit.asterisk.org/564
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib9e6296155a39bc5d627142a3828180c3cfe8fbb
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Joerg Sonnenberger <joerg at bec.de>



More information about the asterisk-code-review mailing list