Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#2219 closed Defect (fixed)

Suspected wrong declaration of ztrans in src/ICE/icevar.F90

Reported by: kmogensen Owned by: systeam
Priority: low Milestone:
Component: SI3 Version: release-4.0
Severity: major Keywords:
Cc: Branch review:
MP ready?: Task progress:



In line 798 in src/ICE/icevar.F90 I read the following:

   INTEGER, PARAMETER ::   ztrans = 0.25_wp


I suspect that this variable should have been declared as REAL(wp).


Commit History (1)


fix bug #2219

Change History (4)

comment:1 Changed 3 years ago by clem

In 10589:

fix bug #2219

comment:2 Changed 3 years ago by clem

  • Resolution set to fixed
  • Status changed from new to closed

yes indeed. I do not know what the compiler is doing with such a thing. I hope the value remains 0.25.
It must be corrected asap, you are right. Done at r10589

comment:3 Changed 3 years ago by kmogensen

For info: Gfortran was complaining about the declaration which is why I noticed it. So I suspect that for that particular compiler it value would be 0 rather than 0.25.

comment:4 Changed 3 years ago by molines

Just a small comment :
A real parameter should have a name starting with pp_ for readibily So in this case it might be pp_trans for example !

In fact I think it is very pleasant to read the code when it follows the coding rules …


Note: See TracTickets for help on using tickets.