[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