Skip to content

Commit

Permalink
Changed intent for G in tidal_forcing_init()
Browse files Browse the repository at this point in the history
- The recent introduction of pass_var() calls on some fields in
  tidal_forcing_init() should have caused compile-time errors
  because pass_var() requires the G%Domain argument to be intent(inout).
- Very worryingly, none of the compilers on c3 complained. Nor did gnu
  complain on the "trusty" image but I happened to still be using the older
  "precise" image for testing my scripts and the compiler there
  caught the mistake.
  • Loading branch information
adcroft committed Jun 21, 2017
1 parent f348179 commit 379c6b0
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/parameterizations/lateral/MOM_tidal_forcing.F90
Original file line number Diff line number Diff line change
Expand Up @@ -105,10 +105,10 @@ module MOM_tidal_forcing
contains

subroutine tidal_forcing_init(Time, G, param_file, CS)
type(time_type), intent(in) :: Time
type(ocean_grid_type), intent(in) :: G !< The ocean's grid structure
type(param_file_type), intent(in) :: param_file !< A structure to parse for run-time parameters
type(tidal_forcing_CS), pointer :: CS
type(time_type), intent(in) :: Time
type(ocean_grid_type), intent(inout) :: G !< The ocean's grid structure
type(param_file_type), intent(in) :: param_file !< A structure to parse for run-time parameters
type(tidal_forcing_CS), pointer :: CS

! This subroutine allocates space for the static variables used
! by this module. The metrics may be effectively 0, 1, or 2-D arrays,
Expand Down

0 comments on commit 379c6b0

Please sign in to comment.