[Asterisk-code-review] lock: Remove ast mutex logger from header. (asterisk[master])
Corey Farrell
asteriskteam at digium.com
Fri Oct 19 08:27:22 CDT 2018
Hello Richard Mudgett, Jenkins2,
I'd like you to reexamine a change. Please visit
https://gerrit.asterisk.org/10519
to look at the new patch set (#2).
Change subject: lock: Remove __ast_mutex_logger from header.
......................................................................
lock: Remove __ast_mutex_logger from header.
__ast_mutex_logger used the variable `canlog` without accepting it as a
argument. Replace this macro with ast_mutex_logger which correctly
takes canlog as the first argument. This will prevent confusion when
working with lock.c code, many of the functions declare a canlog
variable and in some cases it appeared to be unused.
ast_mutex_logger is defined in lock.c as it is a private implementation
detail.
Change-Id: I83b372cb0654c5c18eadc512f65a57fa6c2e9853
---
M include/asterisk/lock.h
M main/lock.c
M utils/extconf.c
3 files changed, 73 insertions(+), 64 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/19/10519/2
--
To view, visit https://gerrit.asterisk.org/10519
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I83b372cb0654c5c18eadc512f65a57fa6c2e9853
Gerrit-Change-Number: 10519
Gerrit-PatchSet: 2
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2 (1000185)
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181019/fa389eba/attachment.html>
More information about the asterisk-code-review
mailing list