[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