View Issue Details

IDProjectCategoryView StatusLast Update
0001730OpenFOAM[All Projects] Bugpublic2015-06-03 09:49
ReporteralexeymAssigned Tohenry 
PrioritynormalSeveritytrivialReproducibilityalways
Status resolvedResolutionfixed 
Product Version 
Fixed in Version 
Summary0001730: rhoCentralFoam does not use time controls read in readTimeControls.H
DescriptionLine 49 of rhoCentralFoam.C is

#include "readTimeControls.H"

yet none of the variables created in readTimeControls.H is used in rhoCentralFoam.C.
Steps To Reproduce1. Build rhoCentralFoam.C with -Wall -Wextra flags.
2. Get following warnings:

In file included from rhoCentralFoam.C:49:
$FOAM_SRC/finiteVolume/lnInclude/readTimeControls.H:38:8: warning: unused variable 'maxDeltaT' [-Wunused-variable]
scalar maxDeltaT =
       ^
$FOAM_SRC/finiteVolume/lnInclude/readTimeControls.H:35:8: warning: unused variable 'maxCo' [-Wunused-variable]
scalar maxCo =
       ^
$FOAM_SRC/finiteVolume/lnInclude/readTimeControls.H:32:12: warning: unused variable 'adjustTimeStep' [-Wunused-variable]
const bool adjustTimeStep =
           ^

3. Compare variables in warnings with the ones defined in readTimeControls.H
Additional InformationIf line 49 is commented out, solver compiles without warnings.
TagsNo tags attached.

Activities

alexeym

2015-06-02 21:32

reporter   ~0004883

The same thing with:
1. rhoCentralFoam.C (line 49)
2. sonicFoam.C (line 64)

henry

2015-06-03 09:48

manager   ~0004885

Resolved by commit a244c4111aa7a2c11d439c4747dfb9d15068359b

Issue History

Date Modified Username Field Change
2015-06-02 21:10 alexeym New Issue
2015-06-02 21:32 alexeym Note Added: 0004883
2015-06-03 09:48 henry Note Added: 0004885
2015-06-03 09:48 henry Status new => resolved
2015-06-03 09:48 henry Resolution open => fixed
2015-06-03 09:48 henry Assigned To => henry