<div dir="ltr">Wow, I didn't realize you could just flip the order to silence the warning...  Good to know.<div><br></div><div>LGTM</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 23, 2013 at 7:20 AM, G M <span dir="ltr"><<a href="mailto:gmisocpp@gmail.com" target="_blank">gmisocpp@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><p>Hi Everyone</p><p>Attached is a patch that changes all references to "x inline" to "inline x" where x = _libcpp_always_inline or _libcpp_inline_visibility macros.</p>
<p>The patch touches these files:</p>
<div>locale<br>array<br>deque<br>new<br>string<br>utility<br>vector<br>__bit_reference<br>__split_buffer</div><div>locale_win32.h</div><div> </div><div>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 etc.</div>

<div> </div><div>It is intended to make libcxx more consistent with itself and to prevent the 1000 or so</div><div>"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.</div>

<div> </div><div>Prefer "inline [other inline related keyword]" over "[other related keyword] inline".</div><div>After this patch, libcxx should be consistent to this pattern.</div><div> </div><div>Thanks</div>

</div>
</blockquote></div><br></div>