Skip to content

Commit 76f7a9d

Browse files
committed
Address CodeQL complaints
1 parent b4c9765 commit 76f7a9d

File tree

3 files changed

+2
-5
lines changed

3 files changed

+2
-5
lines changed

SU2_CFD/include/gradients/computeMetrics.hpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
#include <vector>
3333
#include <algorithm>
3434
#include <limits>
35+
#include <cmath>
3536
#include "../../../Common/include/parallelization/omp_structure.hpp"
3637
#include "../../../Common/include/linear_algebra/blas_structure.hpp"
3738
#include "../../../Common/include/toolboxes/geometry_toolbox.hpp"
@@ -171,7 +172,7 @@ void setPositiveDefiniteMetrics(CGeometry& geometry, const CConfig& config,
171172
/*--- Make positive definite by taking absolute value of eigenvalues ---*/
172173
/*--- Handle NaN and very small values that could cause numerical issues ---*/
173174
for (auto iDim = 0; iDim < nDim; iDim++) {
174-
if (EigVal[iDim] != EigVal[iDim]) {
175+
if (std::isnan(EigVal[iDim])) {
175176
/*--- NaN detected, set to small positive value ---*/
176177
EigVal[iDim] = eps;
177178
} else {
@@ -268,8 +269,6 @@ void normalizeMetrics(CGeometry& geometry, const CConfig& config,
268269
ScalarType A[nDim][nDim], EigVec[nDim][nDim], EigVal[nDim], work[nDim];
269270

270271
for (auto iPoint = 0ul; iPoint < nPointDomain; ++iPoint) {
271-
auto nodes = geometry.nodes;
272-
273272
/*--- Decompose metric ---*/
274273
Tensor::get(metric, iPoint, iSensor, A, nDim);
275274
CBlasStructure::EigenDecomposition(A, EigVec, EigVal, nDim, work);

SU2_CFD/src/output/CFlowOutput.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4132,7 +4132,6 @@ void CFlowOutput::LoadMeshAdaptationOutputs(const CConfig* config, const CSolver
41324132
unsigned long iPoint) {
41334133

41344134
const auto* Node_Flow = solver[FLOW_SOL]->GetNodes();
4135-
const auto* Node_Geo = geometry->nodes;
41364135
if(config->GetCompute_Metric()) {
41374136
// Common metric components for both 2D and 3D
41384137
SetVolumeOutputValue("METRIC_XX", iPoint, Node_Flow->GetMetric(iPoint, 0));

SU2_CFD/src/output/filewriter/CParaviewXMLFileWriter.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -384,7 +384,6 @@ void CParaviewXMLFileWriter::WriteData(string val_filename){
384384
/*--- Resize buffer to accommodate tensor data ---*/
385385
dataBufferFloat.resize(myPoint*NTENSOR);
386386

387-
float val = 0.0;
388387
for (iPoint = 0; iPoint < myPoint; iPoint++) {
389388
dataBufferFloat[iPoint*NTENSOR + 0] = (float)dataSorter->GetData(VarCounter+0,iPoint); // XX
390389
dataBufferFloat[iPoint*NTENSOR + 1] = (float)dataSorter->GetData(VarCounter+2,iPoint); // YY

0 commit comments

Comments
 (0)