[Asterisk-code-review] stasis channels.c: Remove a very silly RAII VAR(). (asterisk[13])

Jenkins2 asteriskteam at digium.com
Tue Nov 7 08:09:08 CST 2017


Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/7019 )

Change subject: stasis_channels.c: Remove a very silly RAII_VAR().
......................................................................

stasis_channels.c: Remove a very silly RAII_VAR().

Change-Id: I28b458b3c1a442c4ef0be7b4986a95ea4149e14f
---
M main/stasis_channels.c
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Sean Bright: Looks good to me, but someone else must approve
  Corey Farrell: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/main/stasis_channels.c b/main/stasis_channels.c
index d46a8dd..0479bd9 100644
--- a/main/stasis_channels.c
+++ b/main/stasis_channels.c
@@ -889,7 +889,7 @@
 	const struct ast_channel_snapshot *snapshot,
 	const struct stasis_message_sanitizer *sanitize)
 {
-	RAII_VAR(struct ast_json *, json_chan, NULL, ast_json_unref);
+	struct ast_json *json_chan;
 
 	if (snapshot == NULL
 		|| (sanitize && sanitize->channel_snapshot
@@ -918,7 +918,7 @@
 		"creationtime", ast_json_timeval(snapshot->creationtime, NULL),
 		"language", snapshot->language);
 
-	return ast_json_ref(json_chan);
+	return json_chan;
 }
 
 int ast_channel_snapshot_cep_equal(

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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: merged
Gerrit-Change-Id: I28b458b3c1a442c4ef0be7b4986a95ea4149e14f
Gerrit-Change-Number: 7019
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171107/17bbfff1/attachment-0001.html>


More information about the asterisk-code-review mailing list