[Asterisk-code-review] Fix/Update clang-RAII macro implementation (asterisk[13])
Diederik de Groot
asteriskteam at digium.com
Mon Apr 20 13:07:56 CDT 2015
Hello Matt Jordan, George Joseph,
I'd like you to reexamine a change. Please visit
https://gerrit.asterisk.org/159
to look at the new patch set (#3).
Change subject: Fix/Update clang-RAII macro implementation
......................................................................
Fix/Update clang-RAII macro implementation
- When you need to refer to 'variable XXX' outside a block, it needs
to declared as '__block XXX', otherwise it will not be available with-
in the block, making updating that variable hard to do, and ast_free
lead to issues.
- Removed the #error message
because it creates complications when compiling external projects
against asterisk For example when using a different compiler then the
one used to compile asterisk. The warning/error should be generated
during the configure process not the compilation process
ASTERISK-24917
Change-Id: I12091228090e90831bf2b498293858f46ea7a8c2
---
M include/asterisk/utils.h
1 file changed, 2 insertions(+), 8 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/59/159/3
--
To view, visit https://gerrit.asterisk.org/159
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I12091228090e90831bf2b498293858f46ea7a8c2
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Diederik de Groot <dkgroot at talon.nl>
Gerrit-Reviewer: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
More information about the asterisk-code-review
mailing list