Skip to content

Fixes for export NetCDF files by Delft-FEWS#1

Open
boyandomhof wants to merge 1 commit into
Deltares:masterfrom
boyandomhof:patch-1
Open

Fixes for export NetCDF files by Delft-FEWS#1
boyandomhof wants to merge 1 commit into
Deltares:masterfrom
boyandomhof:patch-1

Fixes for export NetCDF files by Delft-FEWS

7509956
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 19, 2026 in 50s

Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check failure on line 50 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "nullptr" literal. 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPc7&open=AZ0Gk2zRSrNuz-3DzPc7&pullRequest=1

Check failure on line 724 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 126 to the 25 allowed. 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdq&open=AZ0Gk2zRSrNuz-3DzPdq&pullRequest=1

Check warning on line 1048 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make the type of this parameter a pointer-to-const. The current type of "param_name" is "char *". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdw&open=AZ0Gk2zRSrNuz-3DzPdw&pullRequest=1

Check failure on line 488 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "malloc". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdZ&open=AZ0Gk2zRSrNuz-3DzPdZ&pullRequest=1

Check failure on line 515 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "nullptr" literal. 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdd&open=AZ0Gk2zRSrNuz-3DzPdd&pullRequest=1

Check warning on line 1091 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value stored to 'status' is never read 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPd8&open=AZ0Gk2zRSrNuz-3DzPd8&pullRequest=1

Check failure on line 409 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "malloc". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdR&open=AZ0Gk2zRSrNuz-3DzPdR&pullRequest=1

Check failure on line 232 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "malloc". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdJ&open=AZ0Gk2zRSrNuz-3DzPdJ&pullRequest=1

Check warning on line 1068 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value stored to 'status' is never read 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPd5&open=AZ0Gk2zRSrNuz-3DzPd5&pullRequest=1

Check failure on line 206 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "free". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdG&open=AZ0Gk2zRSrNuz-3DzPdG&pullRequest=1

Check failure on line 505 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "malloc". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdb&open=AZ0Gk2zRSrNuz-3DzPdb&pullRequest=1

Check failure on line 463 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rewrite the code so that you no longer need this "delete". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdU&open=AZ0Gk2zRSrNuz-3DzPdU&pullRequest=1

Check warning on line 1120 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value stored to 'status' is never read 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPeA&open=AZ0Gk2zRSrNuz-3DzPeA&pullRequest=1

Check failure on line 492 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "free". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdY&open=AZ0Gk2zRSrNuz-3DzPdY&pullRequest=1

Check warning on line 232 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the redundant type with "auto". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdB&open=AZ0Gk2zRSrNuz-3DzPdB&pullRequest=1

Check warning on line 329 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Pass large object "times" by reference to const. 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdK&open=AZ0Gk2zRSrNuz-3DzPdK&pullRequest=1

Check failure on line 766 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "nullptr" literal. 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdu&open=AZ0Gk2zRSrNuz-3DzPdu&pullRequest=1

Check failure on line 1158 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "free". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdx&open=AZ0Gk2zRSrNuz-3DzPdx&pullRequest=1

Check failure on line 1086 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "free". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdy&open=AZ0Gk2zRSrNuz-3DzPdy&pullRequest=1

Check failure on line 1155 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Forming reference to null pointer 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPeE&open=AZ0Gk2zRSrNuz-3DzPeE&pullRequest=1

Check failure on line 190 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "nullptr" literal. 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdD&open=AZ0Gk2zRSrNuz-3DzPdD&pullRequest=1

Check failure on line 460 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rewrite the code so that you no longer need this "delete". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdW&open=AZ0Gk2zRSrNuz-3DzPdW&pullRequest=1

Check warning on line 1123 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Value stored to 'status' is never read 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPeB&open=AZ0Gk2zRSrNuz-3DzPeB&pullRequest=1

Check warning on line 195 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "contains" instead of "find". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdE&open=AZ0Gk2zRSrNuz-3DzPdE&pullRequest=1

Check failure on line 178 in packages/src/cf_timeseries.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "malloc". 

See more on https://sonarcloud.io/project/issues?id=Deltares_PlotCFTS&issues=AZ0Gk2zRSrNuz-3DzPdA&open=AZ0Gk2zRSrNuz-3DzPdA&pullRequest=1
View more details on SonarQubeCloud