[Asterisk-code-review] Fix/Update clang-RAII macro implementation (asterisk[13])

[Diederik de Groot]

Diederik de Groot has posted comments on this change.

Change subject: Fix/Update clang-RAII macro implementation
......................................................................


Patch Set 4:

Aggreed. How about all the other clang patches we made before, should they also be cherry picked to master and 11. Should that be done in one big review or on per case. I am not fully clear about the standard/procedure in this case, as those commits was done against svn a couple of weeks ago.

git log |grep -B4 "clang compiler warnings"
Maybe someone with a little more git experience should/could help out ?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I12091228090e90831bf2b498293858f46ea7a8c2
Gerrit-PatchSet: 4
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Diederik de Groot <dkgroot at talon.nl>
Gerrit-Reviewer: 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>
Gerrit-HasComments: No