[Asterisk-code-review] func_vmcount: Add support for multiple mailboxes (asterisk[master])

[Sean Bright]

Attention is currently required from: N A.
Sean Bright has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/16490 )

Change subject: func_vmcount: Add support for multiple mailboxes
......................................................................


Patch Set 1:

(1 comment)

File funcs/func_vmcount.c:

https://gerrit.asterisk.org/c/asterisk/+/16490/comment/aabfb720_b8522e89 
PS1, Line 91: 	while ((mailbox = strsep(&args.vmbox, "|"))) {
Also, whatever delimiter ends up being appropriate, make sure that you are tolerant of zero or more leading delimiters, zero or more trailing delimiters, and empty items in the list (e.g. foo at default|||bar at default)



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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I9108528300795fd5b607efa9d4dd7b74be031813
Gerrit-Change-Number: 16490
Gerrit-PatchSet: 1
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Sean Bright <sean at seanbright.com>
Gerrit-Attention: N A <mail at interlinked.x10host.com>
Gerrit-Comment-Date: Mon, 20 Sep 2021 14:05:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210920/c4ddc2e2/attachment.html>