vim/YouCompleteMe
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Forcing the use of move in one location

Browse Source
This commit is contained in:
Strahinja Val Markovic 2012-08-06 20:02:46 -07:00
parent ddca4d0cf9
commit f17e45bdc3
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
cpp/ycm/CompletionData.cpp
View File

@ -18,6 +18,7 @@
#include "CompletionData.h"
#include <boost/algorithm/string/erase.hpp>
#include <boost/move/move.hpp>
namespace
{
@ -132,7 +133,7 @@ std::string OptionalChunkToString( CXCompletionString completion_string,
// the parameter BUT if this code is compiled in C++11 mode a move constructor
// can be called on the passed-in value. This is not possible if we accept the
// param by const ref.
std::string RemoveTwoUnderscores( std::string text )
std::string RemoveTwoConsecutiveUnderscores( std::string text )
{
boost::erase_all( text, "__" );
return text;
@ -184,7 +185,8 @@ CompletionData::CompletionData( const CXCompletionResult &completion_result )
// ANY C++ identifier with two consecutive underscores in it is
// compiler-reserved.
everything_except_return_type_ =
RemoveTwoUnderscores( everything_except_return_type_ );
RemoveTwoConsecutiveUnderscores(
boost::move( everything_except_return_type_ ) );
detailed_info_.append( return_type_ )
.append( " " )