On Sep 23, 2013, at 10:20 AM, G M <[email protected]> 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: > > locale > array > deque > new > string > utility > vector > __bit_reference > __split_buffer > locale_win32.h > > 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. > > 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. > > Thanks > <inline_x.diff>
Committed revision 191987. Thanks, Howard _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
