--- trunk/src/selection/SelectionCompiler.cpp 2010/05/10 17:28:26 1442 +++ branches/development/src/selection/SelectionCompiler.cpp 2013/05/17 14:41:42 1875 @@ -35,8 +35,9 @@ * * [1] Meineke, et al., J. Comp. Chem. 26, 252-271 (2005). * [2] Fennell & Gezelter, J. Chem. Phys. 124, 234104 (2006). - * [3] Sun, Lin & Gezelter, J. Chem. Phys. 128, 24107 (2008). - * [4] Vardeman & Gezelter, in progress (2009). + * [3] Sun, Lin & Gezelter, J. Chem. Phys. 128, 234107 (2008). + * [4] Kuang & Gezelter, J. Chem. Phys. 133, 164101 (2010). + * [5] Vardeman, Stocker & Gezelter, J. Chem. Theory Comput. 7, 834 (2011). */ #include "selection/SelectionCompiler.hpp" @@ -176,7 +177,7 @@ namespace OpenMD { continue; } - if (ltoken.size() == 0) { + if (ltoken.empty()) { return commandExpected(); } @@ -440,7 +441,7 @@ namespace OpenMD { bool SelectionCompiler::compileExpression() { /** todo */ - int i = 1; + unsigned int i = 1; int tokCommand = atokenCommand[0].tok; if (tokCommand == Token::define) { i = 2; @@ -572,6 +573,7 @@ namespace OpenMD { // fall into the code and below and just add the token case Token::all: case Token::none: + case Token::hull: return addTokenToPostfix(tokenNext()); case Token::leftparen: tokenNext(); @@ -664,7 +666,7 @@ namespace OpenMD { bool SelectionCompiler::isNameValid(const std::string& name) { int nbracket = 0; int ndot = 0; - for (int i =0 ; i < name.size(); ++i) { + for (unsigned int i = 0 ; i < name.size(); ++i) { switch(name[i]) { case '[' :