diff --git a/Applications/DataExplorer/DataView/StratView/StratScene.cpp b/Applications/DataExplorer/DataView/StratView/StratScene.cpp
index ad39310169fcc619e970c65fcabfef8cd21dd92d..6ad50d170bfa038758b7eea8b2d675661719e591 100644
--- a/Applications/DataExplorer/DataView/StratView/StratScene.cpp
+++ b/Applications/DataExplorer/DataView/StratView/StratScene.cpp
@@ -58,7 +58,7 @@ StratScene::StratScene(GeoLib::StationBorehole* station,
addDepthLabels(station->getProfile(), stratBarOffset + stratBarBounds.width());
- if (station->getSoilNames().size() > 0)
+ if (!station->getSoilNames().empty())
addSoilNameLabels(station->getSoilNames(), station->getProfile(), stratBarOffset +
(stratBarBounds.width() / 2));
}
diff --git a/Applications/DataExplorer/VtkVis/VtkPointsSource.cpp b/Applications/DataExplorer/VtkVis/VtkPointsSource.cpp
index 23552d614fc77298c299ea563ba68c16c6265c2d..8c5ee2b0a51f0f390c42be19d581cfee9921756c 100644
--- a/Applications/DataExplorer/VtkVis/VtkPointsSource.cpp
+++ b/Applications/DataExplorer/VtkVis/VtkPointsSource.cpp
@@ -45,7 +45,7 @@ void VtkPointsSource::PrintSelf( ostream& os, vtkIndent indent )
{
this->Superclass::PrintSelf(os,indent);
- if (_points->size() == 0)
+ if (_points->empty())
return;
os << indent << "== VtkPointsSource ==" << "\n";
diff --git a/Applications/DataExplorer/VtkVis/VtkPolylinesSource.cpp b/Applications/DataExplorer/VtkVis/VtkPolylinesSource.cpp
index 747da52d917a1fd96c032677311ed30a2d3439fe..eb162fe53950719ee11e9a6560f6ddedf2126ad9 100644
--- a/Applications/DataExplorer/VtkVis/VtkPolylinesSource.cpp
+++ b/Applications/DataExplorer/VtkVis/VtkPolylinesSource.cpp
@@ -51,7 +51,7 @@ void VtkPolylinesSource::PrintSelf( ostream& os, vtkIndent indent )
{
this->Superclass::PrintSelf(os,indent);
- if (_polylines->size() == 0)
+ if (_polylines->empty())
return;
for (auto _polyline : *_polylines)
@@ -77,7 +77,7 @@ int VtkPolylinesSource::RequestData( vtkInformation* request,
if (!_polylines)
return 0;
- if (_polylines->size() == 0)
+ if (_polylines->empty())
{
ERR("VtkPolylineSource::RequestData(): Size of polyline vector is 0");
return 0;
diff --git a/Applications/DataExplorer/VtkVis/VtkStationSource.cpp b/Applications/DataExplorer/VtkVis/VtkStationSource.cpp
index 18813e7719912915c6b4318822e86f50f7e494f5..ee10fdded3dccd2c83b80c5ff333623a14bcd7f5 100644
--- a/Applications/DataExplorer/VtkVis/VtkStationSource.cpp
+++ b/Applications/DataExplorer/VtkVis/VtkStationSource.cpp
@@ -48,7 +48,7 @@ void VtkStationSource::PrintSelf( ostream& os, vtkIndent indent )
{
this->Superclass::PrintSelf(os,indent);
- if (_stations->size() == 0)
+ if (_stations->empty())
return;
os << indent << "== VtkStationSource ==" << "\n";
diff --git a/Applications/DataExplorer/VtkVis/VtkSurfacesSource.cpp b/Applications/DataExplorer/VtkVis/VtkSurfacesSource.cpp
index 28f4362f1e3dc6edb1c007acc0ad50fd93e296bb..27b12c9feec3c42c64283eb2d9b7ec54d07e10ab 100644
--- a/Applications/DataExplorer/VtkVis/VtkSurfacesSource.cpp
+++ b/Applications/DataExplorer/VtkVis/VtkSurfacesSource.cpp
@@ -48,7 +48,7 @@ void VtkSurfacesSource::PrintSelf( ostream& os, vtkIndent indent )
{
this->Superclass::PrintSelf(os,indent);
- if (_surfaces->size() == 0)
+ if (_surfaces->empty())
return;
os << indent << "== VtkSurfacesSource ==" << "\n";
diff --git a/Applications/DataExplorer/VtkVis/VtkVisTabWidget.cpp b/Applications/DataExplorer/VtkVis/VtkVisTabWidget.cpp
index 867dec7385d4cef092689cec9daf2a29adb1fcfd..4c3e61bb59c237d8428fb38e5fc56e64202a5b9b 100644
--- a/Applications/DataExplorer/VtkVis/VtkVisTabWidget.cpp
+++ b/Applications/DataExplorer/VtkVis/VtkVisTabWidget.cpp
@@ -319,7 +319,7 @@ void VtkVisTabWidget::buildProportiesDialog(VtkVisPipelineItem* item)
QList<QVariant> values = i.value();
VtkAlgorithmPropertyVectorEdit* vectorEdit;
- if (values.size() > 0)
+ if (!values.empty())
{
QList<QString> valuesAsString;
foreach (QVariant variant, values)
diff --git a/Applications/DataExplorer/mainwindow.cpp b/Applications/DataExplorer/mainwindow.cpp
index 21c11b6f59e158997601dceeea561b26ca8d1d48..905df83a4af9b245f4f7d6c66101e9fa6ad0cbcb 100644
--- a/Applications/DataExplorer/mainwindow.cpp
+++ b/Applications/DataExplorer/mainwindow.cpp
@@ -746,7 +746,7 @@ void MainWindow::loadPetrelFiles()
this, "Select surface data file(s) to import", "", "Petrel files (*)");
QStringList well_path_file_names = QFileDialog::getOpenFileNames(
this, "Select well path data file(s) to import", "", "Petrel files (*)");
- if (sfc_file_names.size() != 0 || well_path_file_names.size() != 0)
+ if (!sfc_file_names.empty() || !well_path_file_names.empty())
{
QStringList::const_iterator it = sfc_file_names.begin();
std::list<std::string> sfc_files;
@@ -1284,7 +1284,7 @@ QString MainWindow::getLastUsedDir()
QSettings settings;
QString fileName("");
QStringList files = settings.value("recentFileList").toStringList();
- if (files.size() != 0)
+ if (!files.empty())
return QFileInfo(files[0]).absolutePath();
else
return QDir::homePath();
diff --git a/Applications/FileIO/PetrelInterface.cpp b/Applications/FileIO/PetrelInterface.cpp
index a8b264fc5a6271fee21668e8bf9fbefcd704aa3d..60a507b864a8300ad459e3a032be7397ebf4baeb 100644
--- a/Applications/FileIO/PetrelInterface.cpp
+++ b/Applications/FileIO/PetrelInterface.cpp
@@ -67,11 +67,11 @@ PetrelInterface::PetrelInterface(std::list<std::string> &sfc_fnames,
// store data in GEOObject
geo_obj->addPointVec(std::unique_ptr<std::vector<GeoLib::Point*>>(pnt_vec),
_unique_name);
- if (well_vec->size() > 0)
+ if (!well_vec->empty())
geo_obj->addStationVec(
std::unique_ptr<std::vector<GeoLib::Point*>>(well_vec),
_unique_name);
- if (ply_vec->size() > 0)
+ if (!ply_vec->empty())
geo_obj->addPolylineVec(
std::unique_ptr<std::vector<GeoLib::Polyline*>>(ply_vec),
_unique_name);
diff --git a/GeoLib/IO/Legacy/OGSIOVer4.cpp b/GeoLib/IO/Legacy/OGSIOVer4.cpp
index 6dc1e1e06813128fceda4e2c76be97fb3a01b3fd..8ee236a70762392a50a95d934e5d2b5e784adbb6 100644
--- a/GeoLib/IO/Legacy/OGSIOVer4.cpp
+++ b/GeoLib/IO/Legacy/OGSIOVer4.cpp
@@ -187,9 +187,9 @@ std::string readPolyline(std::istream &in,
{ // read the point ids
in >> line;
if (type != 100)
- while (!in.eof() && !in.fail() && line.size() != 0
- && (line.find('#') == std::string::npos)
- && (line.find('$') == std::string::npos))
+ while (!in.eof() && !in.fail() && !line.empty() &&
+ (line.find('#') == std::string::npos) &&
+ (line.find('$') == std::string::npos))
{
auto pnt_id(BaseLib::str2number<std::size_t>(line));
if (!zero_based_indexing)
@@ -219,7 +219,7 @@ std::string readPolyline(std::istream &in,
line = path + line;
readPolylinePointVector(line, pnt_vec, ply, path, errors);
} // subkeyword found
- } while (line.find('#') == std::string::npos && line.size() != 0 && in);
+ } while (line.find('#') == std::string::npos && !line.empty() && in);
if (type != 100)
{
@@ -322,9 +322,9 @@ std::string readSurface(std::istream &in,
if (line.find("$POLYLINES") != std::string::npos) // subkeyword found
{ // read the name of the polyline(s)
in >> line;
- while (!in.eof() && !in.fail() && line.size() != 0
- && (line.find('#') == std::string::npos)
- && (line.find('$') == std::string::npos))
+ while (!in.eof() && !in.fail() && !line.empty() &&
+ (line.find('#') == std::string::npos) &&
+ (line.find('$') == std::string::npos))
{
// we did read the name of a polyline -> search the id for polyline
auto it(ply_vec_names.find(line));
@@ -355,7 +355,7 @@ std::string readSurface(std::istream &in,
}
// empty line or a keyword is found
}
- } while (line.find('#') == std::string::npos && line.size() != 0 && in);
+ } while (line.find('#') == std::string::npos && !line.empty() && in);
if (!name.empty())
sfc_names.insert(std::pair<std::string,std::size_t>(name,sfc_vec.size()));
diff --git a/GeoLib/StationBorehole.cpp b/GeoLib/StationBorehole.cpp
index e0ed2b2047895849dd96088c1066ba4b5098526f..57671ab879f46aad59a7bb925e5d30e1cfb8e502 100644
--- a/GeoLib/StationBorehole.cpp
+++ b/GeoLib/StationBorehole.cpp
@@ -140,7 +140,7 @@ int StationBorehole::addLayer(std::list<std::string> fields, StationBorehole* bo
int StationBorehole::addStratigraphy(const std::vector<Point*> &profile, const std::vector<std::string> &soil_names)
{
- if (((profile.size()-1) == soil_names.size()) && (soil_names.size()>0))
+ if (((profile.size() - 1) == soil_names.size()) && (!soil_names.empty()))
{
this->_profilePntVec.push_back(profile[0]);
std::size_t nLayers = soil_names.size();
diff --git a/ProcessLib/GroundwaterFlow/GroundwaterFlowFEM.h b/ProcessLib/GroundwaterFlow/GroundwaterFlowFEM.h
index 0fb418c1f621808313ceb34f8cb7c13b17ea534a..d9734be0888bb5b56d5a0f9a8be1b3dd2ff660d2 100644
--- a/ProcessLib/GroundwaterFlow/GroundwaterFlowFEM.h
+++ b/ProcessLib/GroundwaterFlow/GroundwaterFlowFEM.h
@@ -190,7 +190,7 @@ public:
std::vector<double> const& getIntPtDarcyVelocityX(
std::vector<double>& /*cache*/) const override
{
- assert(_darcy_velocities.size() > 0);
+ assert(!_darcy_velocities.empty());
return _darcy_velocities[0];
}
diff --git a/ProcessLib/HT/CreateHTProcess.cpp b/ProcessLib/HT/CreateHTProcess.cpp
index 29076a86289e3049e94813bc336e1de387f71bea..987df379781e414f366560c0f468661a9b721b2b 100644
--- a/ProcessLib/HT/CreateHTProcess.cpp
+++ b/ProcessLib/HT/CreateHTProcess.cpp
@@ -139,7 +139,7 @@ std::unique_ptr<Process> createHTProcess(
std::vector<double> const b =
//! \ogs_file_param{prj__processes__process__HT__specific_body_force}
config.getConfigParameter<std::vector<double>>("specific_body_force");
- assert(b.size() > 0 && b.size() < 4);
+ assert(!b.empty() && b.size() < 4);
bool const has_gravity = MathLib::toVector(b).norm() > 0;
if (has_gravity)
std::copy_n(b.data(), b.size(), specific_body_force.data());
diff --git a/ProcessLib/HT/HTFEM.h b/ProcessLib/HT/HTFEM.h
index f2cbd909e58eb2696a82a20bf71015fea645275e..92ff83a8aaa995c68237c8b9455c751ad6253ce1 100644
--- a/ProcessLib/HT/HTFEM.h
+++ b/ProcessLib/HT/HTFEM.h
@@ -278,7 +278,7 @@ public:
std::vector<double> const& getIntPtDarcyVelocityX(
std::vector<double>& /*cache*/) const override
{
- assert(_darcy_velocities.size() > 0);
+ assert(!_darcy_velocities.empty());
return _darcy_velocities[0];
}
diff --git a/ProcessLib/HeatConduction/HeatConductionFEM.h b/ProcessLib/HeatConduction/HeatConductionFEM.h
index 1afceed27c72ee287af4bacb2ddacbbf39e9a267..1dc74f69749763d78eee31d073304c1e49ee5b1e 100644
--- a/ProcessLib/HeatConduction/HeatConductionFEM.h
+++ b/ProcessLib/HeatConduction/HeatConductionFEM.h
@@ -171,7 +171,7 @@ public:
std::vector<double> const& getIntPtHeatFluxX(
std::vector<double>& /*cache*/) const override
{
- assert(_heat_fluxes.size() > 0);
+ assert(!_heat_fluxes.empty());
return _heat_fluxes[0];
}
diff --git a/ProcessLib/HydroMechanics/HydroMechanicsFEM.h b/ProcessLib/HydroMechanics/HydroMechanicsFEM.h
index 591cffa4b36ff56cd53ef99627105c3f6ce836b5..1914d9f8ff5c242cfc76c5ceccaf825014b85de0 100644
--- a/ProcessLib/HydroMechanics/HydroMechanicsFEM.h
+++ b/ProcessLib/HydroMechanics/HydroMechanicsFEM.h
@@ -539,7 +539,7 @@ public:
std::vector<double> const& getIntPtDarcyVelocityX(
std::vector<double>& /*cache*/) const override
{
- assert(_darcy_velocities.size() > 0);
+ assert(!_darcy_velocities.empty());
return _darcy_velocities[0];
}
diff --git a/ProcessLib/LiquidFlow/LiquidFlowLocalAssembler.h b/ProcessLib/LiquidFlow/LiquidFlowLocalAssembler.h
index d45f13017ee177600a76f56cf0147011fa6b60c6..c6adbf2bf321e66780af45afb02accca38680ec8 100644
--- a/ProcessLib/LiquidFlow/LiquidFlowLocalAssembler.h
+++ b/ProcessLib/LiquidFlow/LiquidFlowLocalAssembler.h
@@ -116,7 +116,7 @@ public:
std::vector<double> const& getIntPtDarcyVelocityX(
std::vector<double>& /*cache*/) const override
{
- assert(_darcy_velocities.size() > 0);
+ assert(!_darcy_velocities.empty());
return _darcy_velocities[0];
}
diff --git a/ProcessLib/RichardsFlow/RichardsFlowFEM.h b/ProcessLib/RichardsFlow/RichardsFlowFEM.h
index 759c633880b7e5cf360f9575d67e09418d65b2fd..caf5a78f60533223ae539f9068584dd42d8a4c7a 100644
--- a/ProcessLib/RichardsFlow/RichardsFlowFEM.h
+++ b/ProcessLib/RichardsFlow/RichardsFlowFEM.h
@@ -235,14 +235,14 @@ public:
std::vector<double> const& getIntPtSaturation(
std::vector<double>& /*cache*/) const override
{
- assert(_saturation.size() > 0);
+ assert(!_saturation.empty());
return _saturation;
}
std::vector<double> const& getIntPtDarcyVelocityX(
std::vector<double>& /*cache*/) const override
{
- assert(_darcy_velocities.size() > 0);
+ assert(!_darcy_velocities.empty());
return _darcy_velocities[0];
}
diff --git a/ProcessLib/TES/TESReactionAdaptor.cpp b/ProcessLib/TES/TESReactionAdaptor.cpp
index edd977ca082c123d1427a1fd798ad4037ddd805c..85bb8cdb16bcddff2f677579ada82b4434edc6b7 100644
--- a/ProcessLib/TES/TESReactionAdaptor.cpp
+++ b/ProcessLib/TES/TESReactionAdaptor.cpp
@@ -65,7 +65,7 @@ TESFEMReactionAdaptorAdsorption::TESFEMReactionAdaptorAdsorption(
assert(dynamic_cast<Adsorption::AdsorptionReaction const*>(
data.ap.react_sys.get()) != nullptr &&
"Reactive system has wrong type.");
- assert(_bounds_violation.size() != 0);
+ assert(!_bounds_violation.empty());
}
ReactionRate
diff --git a/ProcessLib/TwoPhaseFlowWithPP/CreateTwoPhaseFlowWithPPProcess.cpp b/ProcessLib/TwoPhaseFlowWithPP/CreateTwoPhaseFlowWithPPProcess.cpp
index 03375787171a34c465f5c70acbbc1b8131f00a0e..8bd46d221fb931d9d53a2892092e080dbaa4d2d8 100644
--- a/ProcessLib/TwoPhaseFlowWithPP/CreateTwoPhaseFlowWithPPProcess.cpp
+++ b/ProcessLib/TwoPhaseFlowWithPP/CreateTwoPhaseFlowWithPPProcess.cpp
@@ -58,7 +58,7 @@ std::unique_ptr<Process> createTwoPhaseFlowWithPPProcess(
std::vector<double> const b =
//! \ogs_file_param{prj__processes__process__TWOPHASE_FLOW_PP__specific_body_force}
config.getConfigParameter<std::vector<double>>("specific_body_force");
- assert(b.size() > 0 && b.size() < 4);
+ assert(!b.empty() && b.size() < 4);
Eigen::VectorXd specific_body_force(b.size());
bool const has_gravity = MathLib::toVector(b).norm() > 0;
if (has_gravity)
diff --git a/ProcessLib/TwoPhaseFlowWithPP/TwoPhaseFlowWithPPLocalAssembler.h b/ProcessLib/TwoPhaseFlowWithPP/TwoPhaseFlowWithPPLocalAssembler.h
index 8915b94b9d24c81b373f02f3a270edb4634783d6..9c53d8678c665a555f84731e16705ff95c8e4db1 100644
--- a/ProcessLib/TwoPhaseFlowWithPP/TwoPhaseFlowWithPPLocalAssembler.h
+++ b/ProcessLib/TwoPhaseFlowWithPP/TwoPhaseFlowWithPPLocalAssembler.h
@@ -137,14 +137,14 @@ public:
std::vector<double> const& getIntPtSaturation(
std::vector<double>& /*cache*/) const override
{
- assert(_saturation.size() > 0);
+ assert(!_saturation.empty());
return _saturation;
}
std::vector<double> const& getIntPtWetPressure(
std::vector<double>& /*cache*/) const override
{
- assert(_pressure_wet.size() > 0);
+ assert(!_pressure_wet.empty());
return _pressure_wet;
}
diff --git a/ProcessLib/TwoPhaseFlowWithPrho/CreateTwoPhaseFlowWithPrhoProcess.cpp b/ProcessLib/TwoPhaseFlowWithPrho/CreateTwoPhaseFlowWithPrhoProcess.cpp
index 62288cf12b38b1d9311a70e6e0688fcd048c077d..2e794cf91d4c086817c4c4013bd00c83b1987fa6 100644
--- a/ProcessLib/TwoPhaseFlowWithPrho/CreateTwoPhaseFlowWithPrhoProcess.cpp
+++ b/ProcessLib/TwoPhaseFlowWithPrho/CreateTwoPhaseFlowWithPrhoProcess.cpp
@@ -58,7 +58,7 @@ std::unique_ptr<Process> createTwoPhaseFlowWithPrhoProcess(
std::vector<double> const b =
//! \ogs_file_param{prj__processes__process__TWOPHASE_FLOW_PRHO__specific_body_force}
config.getConfigParameter<std::vector<double>>("specific_body_force");
- assert(b.size() > 0 && b.size() < 4);
+ assert(!b.empty() && b.size() < 4);
Eigen::VectorXd specific_body_force(b.size());
bool const has_gravity = MathLib::toVector(b).norm() > 0;
if (has_gravity)
diff --git a/ProcessLib/TwoPhaseFlowWithPrho/TwoPhaseFlowWithPrhoLocalAssembler.h b/ProcessLib/TwoPhaseFlowWithPrho/TwoPhaseFlowWithPrhoLocalAssembler.h
index acd3f3c46e955f03f43b32eae353bc9631260f67..28c994d8f84c7f46d78b9e159c3f7f802d855aa0 100644
--- a/ProcessLib/TwoPhaseFlowWithPrho/TwoPhaseFlowWithPrhoLocalAssembler.h
+++ b/ProcessLib/TwoPhaseFlowWithPrho/TwoPhaseFlowWithPrhoLocalAssembler.h
@@ -140,14 +140,14 @@ public:
std::vector<double> const& getIntPtSaturation(
std::vector<double>& /*cache*/) const override
{
- assert(_saturation.size() > 0);
+ assert(!_saturation.empty());
return _saturation;
}
std::vector<double> const& getIntPtNonWettingPressure(
std::vector<double>& /*cache*/) const override
{
- assert(_pressure_nonwetting.size() > 0);
+ assert(!_pressure_nonwetting.empty());
return _pressure_nonwetting;
}