[Asterisk-code-review] main/asterisk.c: Remove silly usage of RAII VAR. (asterisk[master])

Jenkins2 asteriskteam at digium.com
Wed Feb 14 12:55:19 CST 2018


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

Change subject: main/asterisk.c: Remove silly usage of RAII_VAR.
......................................................................

main/asterisk.c: Remove silly usage of RAII_VAR.

Change-Id: I7e2996397fbd3c3a6a69dd805c38448ddfc34ae9
---
M main/asterisk.c
1 file changed, 2 insertions(+), 1 deletion(-)

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



diff --git a/main/asterisk.c b/main/asterisk.c
index 0214159..71bff57 100644
--- a/main/asterisk.c
+++ b/main/asterisk.c
@@ -1028,7 +1028,7 @@
 
 static void publish_fully_booted(void)
 {
-	RAII_VAR(struct ast_json *, json_object, NULL, ast_json_unref);
+	struct ast_json *json_object;
 	int uptime = 0;
 	int lastreloaded = 0;
 	struct timeval tmp;
@@ -1049,6 +1049,7 @@
 			"Uptime", uptime,
 			"LastReload", lastreloaded);
 	ast_manager_publish_event("FullyBooted", EVENT_FLAG_SYSTEM, json_object);
+	ast_json_unref(json_object);
 }
 
 static void ast_run_atexits(int run_cleanups)

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I7e2996397fbd3c3a6a69dd805c38448ddfc34ae9
Gerrit-Change-Number: 8206
Gerrit-PatchSet: 1
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett 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/20180214/800edda3/attachment.html>


More information about the asterisk-code-review mailing list