[Asterisk-code-review] Message.c: Clear message channel frames on cleanup (asterisk[13])

Jonathan Rose asteriskteam at digium.com
Wed May 13 16:12:45 CDT 2015


Jonathan Rose has posted comments on this change.

Change subject: Message.c: Clear message channel frames on cleanup
......................................................................


Patch Set 1:

Testing was done by using a Digium phone to perform numerous voicemail retrievals.

Without the patch, the queue would eventually hit 128 and reset back down to 64 (all of the queued frames were null frames) triggering the warning message

With the patch, the queue never reaches the limit (or even close) since it gets dumped with every message and I could sit around refreshing my voicemail all the live long day without having to be assaulted by grotesque warnings on my CLI.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idf18df73ccd8c220be38743335b5c79c2a4c0d0f
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Jonathan Rose <jrose at digium.com>
Gerrit-Reviewer: Jonathan Rose <jrose at digium.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list