You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It seems to me fprettify does not provide an automated edit of keyword elseif to yield else if.
Inspired by a discussion in the fortitude project (link) I observe fprettify (version 0.3.7, fetched from PyPI) apparently doesn't edit elseif while e.g., endif and enddo routinely yield end if and end do, respectively.
My test file:
! test.f90program test
implicit noneinteger:: i
do i =1, 8if (i == 3) thenprint*, "first branch"elseif (i == 4) thenprint*, "second branch"endifenddoend program test
Tests run:
fprettify test.f90, i.e. default parameters: endif and enddo are split, elseif remains elseif
fprettify --whitespace-intrinsics False test.f90: as expected, endif and enddo are equally retained as such
fprettify --whitespace-intrinsics True test.f90: endif and enddo are edited, but elseif is not
fprettify --whitespace-logical True test.f90: endif and enddo are edited, but elseif is not
Is there a different flag or level more suitable for the task I should use instead?
The text was updated successfully, but these errors were encountered:
It seems to me fprettify does not provide an automated edit of keyword
elseif
to yieldelse if
.Inspired by a discussion in the fortitude project (link) I observe fprettify (version 0.3.7, fetched from PyPI) apparently doesn't edit
elseif
while e.g.,endif
andenddo
routinely yieldend if
andend do
, respectively.My test file:
Tests run:
fprettify test.f90
, i.e. default parameters:endif
andenddo
are split,elseif
remainselseif
fprettify --whitespace-intrinsics False test.f90
: as expected,endif
andenddo
are equally retained as suchfprettify --whitespace-intrinsics True test.f90
:endif
andenddo
are edited, butelseif
is notfprettify --whitespace-logical True test.f90
:endif
andenddo
are edited, butelseif
is notIs there a different flag or level more suitable for the task I should use instead?
The text was updated successfully, but these errors were encountered: