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

Fixing a bug with word-boundary ratio calculation

Browse Source
This commit is contained in:
Strahinja Val Markovic 2012-06-24 15:11:58 -07:00
parent a8f095ec0a
commit adb31abef2
2 changed files with 15 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cpp/ycm/Result.cpp
View File

@ -36,11 +36,11 @@ int NumWordBoundaryCharMatches( const std::string &query,
while ( j < query.size() && i < word_boundary_chars.size() ) while ( j < query.size() && i < word_boundary_chars.size() )
{ {
if ( toupper( query[ j ] ) == toupper( word_boundary_chars[ i ] ) ) if ( toupper( query[ j ] ) == toupper( word_boundary_chars[ i ] ) )
++i;
++j; ++j;
++i;
} }
return i; return j;
} }
} // unnamed namespace } // unnamed namespace

12
cpp/ycm/tests/Completer_test.cpp
View File

@ -116,6 +116,18 @@ TEST( CompleterTest, CompleteMatchForWordBoundaryCharsWins )
TEST( CompleterTest, RatioUtilizationTieBreak ) TEST( CompleterTest, RatioUtilizationTieBreak )
{ {
EXPECT_THAT( Completer( Candidates(
"aGaaFooBarQux",
"aBaafbq" ) ).CandidatesForQuery( "fbq" ),
ElementsAre( "aGaaFooBarQux",
"aBaafbq" ) );
EXPECT_THAT( Completer( Candidates(
"aFooBarQux",
"afbq" ) ).CandidatesForQuery( "fbq" ),
ElementsAre( "aFooBarQux",
"afbq" ) );
EXPECT_THAT( Completer( Candidates( EXPECT_THAT( Completer( Candidates(
"FooBarQux", "FooBarQux",
"FooBarQuxZaa" ) ).CandidatesForQuery( "fbq" ), "FooBarQuxZaa" ) ).CandidatesForQuery( "fbq" ),