[patch] [libcxx] inline consistency
rnk at google.com
Mon Sep 23 09:58:49 PDT 2013
Wow, I didn't realize you could just flip the order to silence the
warning... Good to know.
On Mon, Sep 23, 2013 at 7:20 AM, G M <gmisocpp at gmail.com> wrote:
> Hi Everyone
> Attached is a patch that changes all references to "x inline" to "inline
> x" where x = _libcpp_always_inline or _libcpp_inline_visibility macros.
> The patch touches these files:
> There is no intended functionality change and it is expected that
> reversing the position of the inline keyword with regard to the other
> keywords does not change the meaning of anything, least not for apple/Linux
> It is intended to make libcxx more consistent with itself and to prevent
> the 1000 or so
> "inline.cpp(3) : warning C4141: 'inline' : used more than once" warnings
> that MS's cl.exe compiler emits without this patch, i.e. if inline is not
> the first keyword before a function name etc.
> Prefer "inline [other inline related keyword]" over "[other related
> keyword] inline".
> After this patch, libcxx should be consistent to this pattern.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits