[Asterisk-code-review] main/taskprocessor: Increase max name length of taskprocessors (...asterisk[13])

Friendly Automation asteriskteam at digium.com
Tue Mar 26 08:01:31 CDT 2019


Friendly Automation has submitted this change and it was merged. ( https://gerrit.asterisk.org/c/asterisk/+/11166 )

Change subject: main/taskprocessor: Increase max name length of taskprocessors
......................................................................

main/taskprocessor: Increase max name length of taskprocessors

Since the new names went in, the maximum taskprocessor name is too
short.  This patch increases the name field to a length to better
handle the new names.

Change-Id: I32f32d6926f25c8ef5a91303fd2988d2c2858877
---
M include/asterisk/taskprocessor.h
M main/taskprocessor.c
2 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  Sean Bright: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/include/asterisk/taskprocessor.h b/include/asterisk/taskprocessor.h
index 5278595..2f49e47 100644
--- a/include/asterisk/taskprocessor.h
+++ b/include/asterisk/taskprocessor.h
@@ -57,7 +57,7 @@
 struct ast_taskprocessor;
 
 /*! \brief Suggested maximum taskprocessor name length (less null terminator). */
-#define AST_TASKPROCESSOR_MAX_NAME	45
+#define AST_TASKPROCESSOR_MAX_NAME	70
 
 /*! Default taskprocessor high water level alert trigger */
 #define AST_TASKPROCESSOR_HIGH_WATER_LEVEL 500
diff --git a/main/taskprocessor.c b/main/taskprocessor.c
index 969c985..270ce74 100644
--- a/main/taskprocessor.c
+++ b/main/taskprocessor.c
@@ -511,8 +511,8 @@
 	struct ao2_container *sorted_tps;
 	struct ast_taskprocessor *tps;
 	struct ao2_iterator iter;
-#define FMT_HEADERS		"%-45s %10s %10s %10s %10s %10s\n"
-#define FMT_FIELDS		"%-45s %10lu %10lu %10lu %10lu %10lu\n"
+#define FMT_HEADERS		"%-70s %10s %10s %10s %10s %10s\n"
+#define FMT_FIELDS		"%-70s %10lu %10lu %10lu %10lu %10lu\n"
 
 	switch (cmd) {
 	case CLI_INIT:

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/11166
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: I32f32d6926f25c8ef5a91303fd2988d2c2858877
Gerrit-Change-Number: 11166
Gerrit-PatchSet: 1
Gerrit-Owner: Matthew Fredrickson <creslin at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190326/71a15d12/attachment-0001.html>


More information about the asterisk-code-review mailing list