Skip to content

Commit

Permalink
Change uppercase DEBUG variables to lowercase debug
Browse files Browse the repository at this point in the history
This is needed to avoid conflicting with the DEBUG CPP token.

I also changed one instance of debug to a 'parameter'. I left others as
is because I didn't do the analysis needed to confirm that they're never
set from elsewhere (because they are public to their modules).

Fixes ESCOMP#534
  • Loading branch information
billsacks committed Oct 28, 2018
1 parent db532bb commit 6eb7917
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions src/biogeophys/SurfaceRadiationMod.F90
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ module SurfaceRadiationMod
implicit none
private

logical :: DEBUG = .false. ! for debugging this module
logical, parameter :: debug = .false. ! for debugging this module

!
! !PUBLIC MEMBER FUNCTIONS:
Expand Down Expand Up @@ -977,7 +977,7 @@ subroutine SurfaceRadiation(bounds, num_nourbanp, filter_nourbanp, &
g = patch%gridcell(p)

if(elai(p)==0.0_r8.and.fabd(p,1)>0._r8)then
if ( DEBUG ) write(iulog,*) 'absorption without LAI',elai(p),tlai(p),fabd(p,1),p
if ( debug ) write(iulog,*) 'absorption without LAI',elai(p),tlai(p),fabd(p,1),p
endif
! Solar incident

Expand Down
8 changes: 4 additions & 4 deletions src/utils/clmfates_interfaceMod.F90
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ module CLMFatesInterfaceMod
! developer will at least question its usage (RGK)
private :: hlm_bounds_to_fates_bounds

logical :: DEBUG = .false.
logical :: debug = .false.

character(len=*), parameter, private :: sourcefile = &
__FILE__
Expand Down Expand Up @@ -365,7 +365,7 @@ subroutine init(this, bounds_proc )
! Check through FATES parameters to see if all have been set
call set_fates_ctrlparms('check_allset')

if(DEBUG)then
if(debug)then
write(iulog,*) 'clm_fates%init(): allocating for ',nclumps,' threads'
end if

Expand Down Expand Up @@ -400,15 +400,15 @@ subroutine init(this, bounds_proc )
s = s + 1
collist(s) = c
this%f2hmap(nc)%hsites(c) = s
if(DEBUG)then
if(debug)then
write(iulog,*) 'clm_fates%init(): thread',nc,': found column',c,'with lu',l
write(iulog,*) 'LU type:', lun%itype(l)
end if
endif

enddo

if(DEBUG)then
if(debug)then
write(iulog,*) 'clm_fates%init(): thread',nc,': allocated ',s,' sites'
end if

Expand Down
2 changes: 1 addition & 1 deletion src/utils/clmfates_paraminterfaceMod.F90
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ module CLMFatesParamInterfaceMod
private :: SetParameterDimensions
private :: GetUsedDimensionSizes

logical :: DEBUG = .false.
logical :: debug = .false.

character(len=*), parameter, private :: sourcefile = &
__FILE__
Expand Down

0 comments on commit 6eb7917

Please sign in to comment.