[Asterisk-code-review] main/manager.c: Bugfix sort action manager by alphabetically (asterisk[13])

Rodrigo Ramirez Norambuena asteriskteam at digium.com
Wed May 13 10:46:21 CDT 2015


Rodrigo Ramirez Norambuena has uploaded a new change for review.

  https://gerrit.asterisk.org/459

Change subject: main/manager.c: Bugfix sort action_manager by alphabetically
......................................................................

main/manager.c: Bugfix sort action_manager by alphabetically

Fix the alphabetic order added on ast_manager_register_struct. The order
for struct manager_action added is not working, this change fixes the
problem.

Change-Id: I149da0cd06c3c4445d7516cc303358e9f26f8b4b
---
M main/manager.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/59/459/1

diff --git a/main/manager.c b/main/manager.c
index 9aa8ecf..035115f 100644
--- a/main/manager.c
+++ b/main/manager.c
@@ -6755,9 +6755,9 @@
 			return -1;
 		}
 		if (ret > 0) { /* Insert these alphabetically */
-			prev = cur;
 			break;
 		}
+		prev = cur;
 	}
 
 	ao2_t_ref(act, +1, "action object added to list");

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I149da0cd06c3c4445d7516cc303358e9f26f8b4b
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Rodrigo Ramirez Norambuena <decipher.hk at gmail.com>



More information about the asterisk-code-review mailing list