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

boost::array instead of std::vector in nodelist

Browse Source 
This should provide a minor performance boost & memory reduction.
This commit is contained in:
Strahinja Val Markovic 2013-03-01 20:10:12 -08:00
parent dae3dcdd9e
commit c8a9d0590e
2 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cpp/ycm/LetterNodeListMap.cpp
View File

@ -39,8 +39,6 @@ int IndexForChar( char letter ) {
LetterNodeListMap::LetterNodeListMap() { LetterNodeListMap::LetterNodeListMap() {
letters_.resize( kNumLetters );
for ( uint i = 0; i < letters_.size(); ++i ) { for ( uint i = 0; i < letters_.size(); ++i ) {
letters_[ i ] = NULL; letters_[ i ] = NULL;
} }

5
cpp/ycm/LetterNodeListMap.h
View File

@ -18,9 +18,9 @@
#ifndef LETTERNODELISTMAP_H_BRK2UMC1 #ifndef LETTERNODELISTMAP_H_BRK2UMC1
#define LETTERNODELISTMAP_H_BRK2UMC1 #define LETTERNODELISTMAP_H_BRK2UMC1
#include <vector>
#include <list> #include <list>
#include <boost/utility.hpp> #include <boost/utility.hpp>
#include <boost/array.hpp>
#define NUM_LETTERS 128 #define NUM_LETTERS 128
@ -47,8 +47,7 @@ public:
bool HasLetter( char letter ) const; bool HasLetter( char letter ) const;
private: private:
boost::array< std::list< LetterNode * >*, NUM_LETTERS > letters_;
std::vector< std::list< LetterNode * >* > letters_;
}; };
} // namespace YouCompleteMe } // namespace YouCompleteMe