[PATCHES] pam_wrapper: fix some clang/FreeBSD picky-developer warnings
Andreas Schneider
asn at samba.org
Thu Nov 23 14:10:04 UTC 2017
On Wednesday, 22 November 2017 22:24:08 CET Uri Simchoni via samba-technical
wrote:
> Hi,
Hi Uri,
> This patch set contains patches for upstream pam_wrapper, concerning
> fixes to FreeBSD picky-developer build. An accompanying patch has been
> reviewed and fixed by Andreas already.
>
> About the conversion from a variable to a macro - yes that's contrary to
> what they teach us (particularly in C++ school), but one has to remember
> that the reason for deprecation of macros is that tools don't handle
> them intelligently - well, here's a case where at least a particular
> compiler handles macros more intelligently than a variable... (tried
> const without the static - in C++ that should have done the trick, no
> luck either :( )
>
> Please review and maybe push...
thanks for the patches. I pushed them upstream.
Andreas
--
Andreas Schneider GPG-ID: CC014E3D
Samba Team asn at samba.org
www.samba.org
More information about the samba-technical
mailing list