diff --git a/applications/utilities/mesh/conversion/fluent3DMeshToFoam/fluent3DMeshToFoam.L b/applications/utilities/mesh/conversion/fluent3DMeshToFoam/fluent3DMeshToFoam.L index ca276a0348..8898cded2c 100644 --- a/applications/utilities/mesh/conversion/fluent3DMeshToFoam/fluent3DMeshToFoam.L +++ b/applications/utilities/mesh/conversion/fluent3DMeshToFoam/fluent3DMeshToFoam.L @@ -221,6 +221,7 @@ redundantBlock {space}({comment}|{unknownPeriodicFace}|{periodicFace endOfSection {space}")"{space} +/* balance "-quoted for editor */ /* ------------------------------------------------------------------------ *\ ----- Exclusive start states ----- @@ -693,7 +694,7 @@ endOfSection {space}")"{space} {lbrac}{label} { Warning << "Found unknown block of type: " - << Foam::string(YYText())(1, YYLeng()-1) << nl + << std::string(YYText()).substr(1, YYLeng()-1) << nl << " on line " << lineNo << endl; yy_push_state(ignoreBlock); diff --git a/applications/utilities/mesh/manipulation/setSet/setSet.C b/applications/utilities/mesh/manipulation/setSet/setSet.C index 4a64e6f801..2f5b6a5cca 100644 --- a/applications/utilities/mesh/manipulation/setSet/setSet.C +++ b/applications/utilities/mesh/manipulation/setSet/setSet.C @@ -903,10 +903,10 @@ int main(int argc, char *argv[]) } // Strip off anything after # - string::size_type i = rawLine.find_first_of("#"); + string::size_type i = rawLine.find('#'); if (i != string::npos) { - rawLine = rawLine(0, i); + rawLine.resize(i); } if (rawLine.empty()) diff --git a/applications/utilities/preProcessing/changeDictionary/changeDictionary.C b/applications/utilities/preProcessing/changeDictionary/changeDictionary.C index ea80da840f..f9be71c75f 100644 --- a/applications/utilities/preProcessing/changeDictionary/changeDictionary.C +++ b/applications/utilities/preProcessing/changeDictionary/changeDictionary.C @@ -254,7 +254,7 @@ bool merge if (key[0] == '~') { - word eraseKey = key(1, key.size()-1); + const word eraseKey = key.substr(1); if (thisDict.remove(eraseKey)) { // Mark thisDict entry as having been match for wildcard @@ -325,7 +325,7 @@ bool merge if (key[0] == '~') { - word eraseKey = key(1, key.size()-1); + const word eraseKey = key.substr(1); // List of indices into thisKeys labelList matches diff --git a/src/OSspecific/POSIX/POSIX.C b/src/OSspecific/POSIX/POSIX.C index d9b2e0b3a4..3a922ed016 100644 --- a/src/OSspecific/POSIX/POSIX.C +++ b/src/OSspecific/POSIX/POSIX.C @@ -88,7 +88,7 @@ static inline bool isBackupName(const Foam::fileName& name) { return false; } - else if (name[name.size()-1] == '~') + else if (name.back() == '~') { return true; } diff --git a/src/OSspecific/POSIX/printStack.C b/src/OSspecific/POSIX/printStack.C index bee02686b3..4031b91d77 100644 --- a/src/OSspecific/POSIX/printStack.C +++ b/src/OSspecific/POSIX/printStack.C @@ -40,12 +40,11 @@ namespace Foam // * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * // -string pOpen(const string &cmd, label line=0) +string pOpen(const string& cmd, label line=0) { - string res = "\n"; + string res; FILE *cmdPipe = popen(cmd.c_str(), "r"); - if (cmdPipe) { char *buf = nullptr; @@ -54,8 +53,7 @@ string pOpen(const string &cmd, label line=0) for (label cnt = 0; cnt <= line; cnt++) { size_t linecap = 0; - ssize_t linelen; - linelen = ::getline(&buf, &linecap, cmdPipe); + ssize_t linelen = ::getline(&buf, &linecap, cmdPipe); if (linelen < 0) { @@ -65,6 +63,11 @@ string pOpen(const string &cmd, label line=0) if (cnt == line) { res = string(buf); + // Trim trailing newline + if (res.size()) + { + res.resize(res.size()-1); + } break; } } @@ -77,7 +80,7 @@ string pOpen(const string &cmd, label line=0) pclose(cmdPipe); } - return res.substr(0, res.size() - 1); + return res; } diff --git a/src/OpenFOAM/db/dictionary/dictionaryIO.C b/src/OpenFOAM/db/dictionary/dictionaryIO.C index 836add62f9..f1605e50fa 100644 --- a/src/OpenFOAM/db/dictionary/dictionaryIO.C +++ b/src/OpenFOAM/db/dictionary/dictionaryIO.C @@ -130,7 +130,7 @@ bool Foam::dictionary::read(Istream& is) bool Foam::dictionary::substituteKeyword(const word& keyword, bool mergeEntry) { - const word varName = keyword(1, keyword.size()-1); + const word varName = keyword.substr(1); // Lookup the variable name in the given dictionary const entry* ePtr = lookupEntryPtr(varName, true, true); diff --git a/src/OpenFOAM/db/functionObjects/functionObjectList/functionObjectList.C b/src/OpenFOAM/db/functionObjects/functionObjectList/functionObjectList.C index 397262386b..37db0a3067 100644 --- a/src/OpenFOAM/db/functionObjects/functionObjectList/functionObjectList.C +++ b/src/OpenFOAM/db/functionObjects/functionObjectList/functionObjectList.C @@ -210,7 +210,7 @@ bool Foam::functionObjectList::readFunctionObject { if (argLevel == 0) { - funcName = funcNameArgs(start, i - start); + funcName = funcNameArgs.substr(start, i - start); start = i+1; } ++argLevel; @@ -226,7 +226,7 @@ bool Foam::functionObjectList::readFunctionObject Tuple2 ( argName, - funcNameArgs(start, i - start) + funcNameArgs.substr(start, i - start) ) ); namedArg = false; @@ -235,7 +235,10 @@ bool Foam::functionObjectList::readFunctionObject { args.append ( - string::validate(funcNameArgs(start, i - start)) + string::validate + ( + funcNameArgs.substr(start, i - start) + ) ); } start = i+1; @@ -252,7 +255,11 @@ bool Foam::functionObjectList::readFunctionObject } else if (c == '=') { - argName = string::validate(funcNameArgs(start, i - start)); + argName = string::validate + ( + funcNameArgs.substr(start, i - start) + ); + start = i+1; namedArg = true; } diff --git a/src/OpenFOAM/dimensionSet/dimensionSetIO.C b/src/OpenFOAM/dimensionSet/dimensionSetIO.C index 768acc8efc..032555cd25 100644 --- a/src/OpenFOAM/dimensionSet/dimensionSetIO.C +++ b/src/OpenFOAM/dimensionSet/dimensionSetIO.C @@ -140,7 +140,7 @@ void Foam::dimensionSet::tokeniser::splitWord(const word& w) { if (i > start) { - word subWord = w(start, i-start); + const word subWord = w.substr(start, i-start); if (isdigit(subWord[0]) || subWord[0] == token::SUBTRACT) { push(token(readScalar(IStringStream(subWord)()))); @@ -166,7 +166,7 @@ void Foam::dimensionSet::tokeniser::splitWord(const word& w) } if (start < w.size()) { - word subWord = w(start, w.size()-start); + const word subWord = w.substr(start); if (isdigit(subWord[0]) || subWord[0] == token::SUBTRACT) { push(token(readScalar(IStringStream(subWord)()))); @@ -524,9 +524,9 @@ Foam::Istream& Foam::dimensionSet::read do { word symbolPow = nextToken.wordToken(); - if (symbolPow[symbolPow.size()-1] == token::END_SQR) + if (symbolPow.back() == token::END_SQR) { - symbolPow = symbolPow(0, symbolPow.size()-1); + symbolPow.resize(symbolPow.size()-1); continueParsing = false; } @@ -537,8 +537,8 @@ Foam::Istream& Foam::dimensionSet::read size_t index = symbolPow.find('^'); if (index != string::npos) { - word symbol = symbolPow(0, index); - word exp = symbolPow(index+1, symbolPow.size()-index+1); + const word symbol = symbolPow.substr(0, index); + const word exp = symbolPow.substr(index+1); scalar exponent = readScalar(IStringStream(exp)()); dimensionedScalar s; @@ -580,9 +580,9 @@ Foam::Istream& Foam::dimensionSet::read { // Read first five dimensions exponents_[dimensionSet::MASS] = nextToken.number(); - for (int Dimension=1; Dimension> exponents_[Dimension]; + is >> exponents_[d]; } // Read next token diff --git a/src/OpenFOAM/matrices/solution/solution.C b/src/OpenFOAM/matrices/solution/solution.C index 5abac0fa3e..982c3398b6 100644 --- a/src/OpenFOAM/matrices/solution/solution.C +++ b/src/OpenFOAM/matrices/solution/solution.C @@ -76,16 +76,13 @@ void Foam::solution::read(const dictionary& dict) const word& e = entryNames[i]; scalar value = readScalar(relaxDict.lookup(e)); - if (e(0, 1) == "p") + if (e.startsWith("p")) { fieldRelaxDict_.add(e, value); } - else if (e.length() >= 3) + else if (e.startsWith("rho")) { - if (e(0, 3) == "rho") - { - fieldRelaxDict_.add(e, value); - } + fieldRelaxDict_.add(e, value); } } diff --git a/src/combustionModels/FSD/reactionRateFlameAreaModels/reactionRateFlameArea/reactionRateFlameAreaNew.C b/src/combustionModels/FSD/reactionRateFlameAreaModels/reactionRateFlameArea/reactionRateFlameAreaNew.C index 2154d2cc94..83222a222b 100644 --- a/src/combustionModels/FSD/reactionRateFlameAreaModels/reactionRateFlameArea/reactionRateFlameAreaNew.C +++ b/src/combustionModels/FSD/reactionRateFlameAreaModels/reactionRateFlameArea/reactionRateFlameAreaNew.C @@ -34,16 +34,15 @@ Foam::autoPtr Foam::reactionRateFlameArea::New const combustionModel& combModel ) { - word reactionRateFlameAreaType + const word modelType ( dict.lookup("reactionRateFlameArea") ); Info<< "Selecting reaction rate flame area correlation " - << reactionRateFlameAreaType << endl; + << modelType << endl; - auto cstrIter = - dictionaryConstructorTablePtr_->cfind(reactionRateFlameAreaType); + auto cstrIter = dictionaryConstructorTablePtr_->cfind(modelType); if (!cstrIter.found()) { @@ -51,15 +50,13 @@ Foam::autoPtr Foam::reactionRateFlameArea::New ( dict ) << "Unknown reactionRateFlameArea type " - << reactionRateFlameAreaType << nl << nl + << modelType << nl << nl << "Valid reaction rate flame area types :" << endl << dictionaryConstructorTablePtr_->sortedToc() << exit(FatalIOError); } - const label tempOpen = reactionRateFlameAreaType.find('<'); - - const word className = reactionRateFlameAreaType(0, tempOpen); + const word className = modelType.substr(0, modelType.find('<')); return autoPtr (cstrIter()(className, dict, mesh, combModel)); diff --git a/src/combustionModels/psiCombustionModel/psiCombustionModel/psiCombustionModelNew.C b/src/combustionModels/psiCombustionModel/psiCombustionModel/psiCombustionModelNew.C index ca7d0616b8..50e69d177d 100644 --- a/src/combustionModels/psiCombustionModel/psiCombustionModel/psiCombustionModelNew.C +++ b/src/combustionModels/psiCombustionModel/psiCombustionModel/psiCombustionModelNew.C @@ -35,7 +35,7 @@ Foam::combustionModels::psiCombustionModel::New const word& phaseName ) { - const word combModelName + const word modelType ( IOdictionary ( @@ -51,21 +51,21 @@ Foam::combustionModels::psiCombustionModel::New ).lookup("combustionModel") ); - Info<< "Selecting combustion model " << combModelName << endl; + Info<< "Selecting combustion model " << modelType << endl; - auto cstrIter = dictionaryConstructorTablePtr_->cfind(combModelName); + auto cstrIter = dictionaryConstructorTablePtr_->cfind(modelType); if (!cstrIter.found()) { FatalErrorInFunction << "Unknown psiCombustionModel type " - << combModelName << endl << endl - << "Valid combustionModel types :" << endl + << modelType << nl << nl + << "Valid combustionModel types :" << nl << dictionaryConstructorTablePtr_->sortedToc() << exit(FatalError); } - const word className = combModelName(0, combModelName.find('<')); + const word className = modelType.substr(0, modelType.find('<')); return autoPtr ( diff --git a/src/combustionModels/rhoCombustionModel/rhoCombustionModel/rhoCombustionModelNew.C b/src/combustionModels/rhoCombustionModel/rhoCombustionModel/rhoCombustionModelNew.C index f6ffadc0b6..fb03177606 100644 --- a/src/combustionModels/rhoCombustionModel/rhoCombustionModel/rhoCombustionModelNew.C +++ b/src/combustionModels/rhoCombustionModel/rhoCombustionModel/rhoCombustionModelNew.C @@ -35,7 +35,7 @@ Foam::combustionModels::rhoCombustionModel::New const word& phaseName ) { - const word combTypeName + const word modelType ( IOdictionary ( @@ -51,23 +51,21 @@ Foam::combustionModels::rhoCombustionModel::New ).lookup("combustionModel") ); - Info<< "Selecting combustion model " << combTypeName << endl; + Info<< "Selecting combustion model " << modelType << endl; - auto cstrIter = dictionaryConstructorTablePtr_->cfind(combTypeName); + auto cstrIter = dictionaryConstructorTablePtr_->cfind(modelType); if (!cstrIter.found()) { FatalErrorInFunction << "Unknown rhoCombustionModel type " - << combTypeName << endl << endl - << "Valid combustionModel types :" << endl + << modelType << nl << nl + << "Valid combustionModel types :" << nl << dictionaryConstructorTablePtr_->sortedToc() << exit(FatalError); } - const label tempOpen = combTypeName.find('<'); - - const word className = combTypeName(0, tempOpen); + const word className = modelType.substr(0, modelType.find('<')); return autoPtr ( diff --git a/src/meshTools/edgeMesh/edgeMeshFormats/nas/NASedgeFormat.C b/src/meshTools/edgeMesh/edgeMeshFormats/nas/NASedgeFormat.C index 7bd0acab4f..7bcc2d3243 100644 --- a/src/meshTools/edgeMesh/edgeMeshFormats/nas/NASedgeFormat.C +++ b/src/meshTools/edgeMesh/edgeMeshFormats/nas/NASedgeFormat.C @@ -74,7 +74,7 @@ bool Foam::fileFormats::NASedgeFormat::read // Check if character 72 is continuation if (line.size() > 72 && line[72] == '+') { - line = line.substr(0, 72); + line.resize(72); while (true) { @@ -87,7 +87,7 @@ bool Foam::fileFormats::NASedgeFormat::read } else { - line += buf.substr(8, buf.size()-8); + line += buf.substr(8); break; } } diff --git a/src/meshTools/edgeMesh/edgeMeshFormats/obj/OBJedgeFormat.C b/src/meshTools/edgeMesh/edgeMeshFormats/obj/OBJedgeFormat.C index 1261b971cf..e0af5e0e76 100644 --- a/src/meshTools/edgeMesh/edgeMeshFormats/obj/OBJedgeFormat.C +++ b/src/meshTools/edgeMesh/edgeMeshFormats/obj/OBJedgeFormat.C @@ -117,9 +117,9 @@ bool Foam::fileFormats::OBJedgeFormat::read(const fileName& filename) string line = this->getLineNoComment(is); // handle continuations - if (line[line.size()-1] == '\\') + if (line.back() == '\\') { - line.substr(0, line.size()-1); + line.resize(line.size()-1); line += this->getLineNoComment(is); } diff --git a/src/regionModels/thermalBaffleModels/derivedFvPatchFields/thermalBaffle/thermalBaffleFvPatchScalarField.C b/src/regionModels/thermalBaffleModels/derivedFvPatchFields/thermalBaffle/thermalBaffleFvPatchScalarField.C index ca1f1511c0..58f7840398 100644 --- a/src/regionModels/thermalBaffleModels/derivedFvPatchFields/thermalBaffle/thermalBaffleFvPatchScalarField.C +++ b/src/regionModels/thermalBaffleModels/derivedFvPatchFields/thermalBaffle/thermalBaffleFvPatchScalarField.C @@ -191,9 +191,8 @@ void thermalBaffleFvPatchScalarField::createPatchMesh() dicts[bottomPatchID].add("inGroups", inGroups); dicts[bottomPatchID].add("sampleMode", mpp.sampleModeNames_[mpp.mode()]); - const label sepPos = coupleGroup.find('_'); - - const word coupleGroupSlave = coupleGroup(0, sepPos) + "_slave"; + const word coupleGroupSlave = + coupleGroup.substr(0, coupleGroup.find('_')) + "_slave"; inGroups[0] = coupleGroupSlave; dicts[topPatchID].add("coupleGroup", coupleGroupSlave); diff --git a/src/sampling/sampledSurface/readers/ensight/ensightSurfaceReader.C b/src/sampling/sampledSurface/readers/ensight/ensightSurfaceReader.C index 2c21271939..38a51d6141 100644 --- a/src/sampling/sampledSurface/readers/ensight/ensightSurfaceReader.C +++ b/src/sampling/sampledSurface/readers/ensight/ensightSurfaceReader.C @@ -186,7 +186,7 @@ void Foam::ensightSurfaceReader::readCase(IFstream& is) // surfaceName.****.fieldName // This is not parser friendly - simply take remainder of buffer label iPos = iss.stdStream().tellg(); - fieldFileName = buffer(iPos, buffer.size() - iPos); + fieldFileName = buffer.substr(iPos); size_t p0 = fieldFileName.find_first_not_of(' '); if (p0 == string::npos) { diff --git a/src/surfMesh/surfaceFormats/nas/NASsurfaceFormat.C b/src/surfMesh/surfaceFormats/nas/NASsurfaceFormat.C index 7f64549e9c..3bcc035188 100644 --- a/src/surfMesh/surfaceFormats/nas/NASsurfaceFormat.C +++ b/src/surfMesh/surfaceFormats/nas/NASsurfaceFormat.C @@ -89,12 +89,12 @@ bool Foam::fileFormats::NASsurfaceFormat::read string line; is.getLine(line); - // Ansa extension - if (line.substr(0, 10) == "$ANSA_NAME") + // ANSA extension + if (line.startsWith("$ANSA_NAME")) { - string::size_type sem0 = line.find (';', 0); - string::size_type sem1 = line.find (';', sem0+1); - string::size_type sem2 = line.find (';', sem1+1); + string::size_type sem0 = line.find(';', 0); + string::size_type sem1 = line.find(';', sem0+1); + string::size_type sem2 = line.find(';', sem1+1); if ( @@ -111,14 +111,11 @@ bool Foam::fileFormats::NASsurfaceFormat::read string rawName; is.getLine(rawName); - if (rawName[rawName.size()-1] == '\r') + if (rawName.back() == '\r') { - rawName = rawName.substr(1, rawName.size()-2); - } - else - { - rawName = rawName.substr(1, rawName.size()-1); + rawName.resize(rawName.size()-1); } + rawName = rawName.substr(1); string::stripInvalid(rawName); ansaName = rawName; @@ -132,11 +129,7 @@ bool Foam::fileFormats::NASsurfaceFormat::read // Hypermesh extension // $HMNAME COMP 1"partName" - if - ( - line.substr(0, 12) == "$HMNAME COMP" - && line.find ('"') != string::npos - ) + if (line.startsWith("$HMNAME COMP") && line.find('"') != string::npos) { label groupId = readLabel ( @@ -165,7 +158,7 @@ bool Foam::fileFormats::NASsurfaceFormat::read // Check if character 72 is continuation if (line.size() > 72 && line[72] == '+') { - line = line.substr(0, 72); + line.resize(72); while (true) { @@ -178,7 +171,7 @@ bool Foam::fileFormats::NASsurfaceFormat::read } else { - line += buf.substr(8, buf.size()-8); + line += buf.substr(8); break; } } diff --git a/src/surfMesh/surfaceFormats/obj/OBJsurfaceFormat.C b/src/surfMesh/surfaceFormats/obj/OBJsurfaceFormat.C index 708f7654e9..692f2ee3bc 100644 --- a/src/surfMesh/surfaceFormats/obj/OBJsurfaceFormat.C +++ b/src/surfMesh/surfaceFormats/obj/OBJsurfaceFormat.C @@ -82,9 +82,9 @@ bool Foam::fileFormats::OBJsurfaceFormat::read string line = this->getLineNoComment(is); // handle continuations - if (line[line.size()-1] == '\\') + if (line.back() == '\\') { - line.substr(0, line.size()-1); + line.resize(line.size()-1); line += this->getLineNoComment(is); } diff --git a/src/surfMesh/surfaceFormats/tri/TRIsurfaceFormatCore.C b/src/surfMesh/surfaceFormats/tri/TRIsurfaceFormatCore.C index 97b4112f9b..385ad872ab 100644 --- a/src/surfMesh/surfaceFormats/tri/TRIsurfaceFormatCore.C +++ b/src/surfMesh/surfaceFormats/tri/TRIsurfaceFormatCore.C @@ -87,9 +87,9 @@ bool Foam::fileFormats::TRIsurfaceFormatCore::read string line = this->getLineNoComment(is); // handle continuations ? - // if (line[line.size()-1] == '\\') + // if (line.back() == '\\') // { - // line.substr(0, line.size()-1); + // line.resize(line.size()-1); // line += this->getLineNoComment(is); // } @@ -128,14 +128,14 @@ bool Foam::fileFormats::TRIsurfaceFormatCore::read // ie, instead of having 0xFF, skip 0 and leave xFF to // get read as a word and name it "zoneFF" - char zero; - lineStream >> zero; + char zeroChar; + lineStream >> zeroChar; - word rawName(lineStream); - word name("zone" + rawName(1, rawName.size()-1)); + const word rawName(lineStream); + const word name("zone" + rawName.substr(1)); - HashTable