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
I'm using neovim, and installed fortls using Mason.
In this code ...
real pure function sind(x)
real, intent(in), value :: x
sind =sin(x*((4.0*atan(1.0))/180.0))
end function
Going to the function name sind in the line real pure function sind(x) and renaming it using the symbol rename method in neovim( with fortls LSP) only changes the function name, and everywhere it's used, which is good, but does not change the internal variable name sind in the line sind = sin(x*((4.0*atan(1.0))/180.0))
I had to manually rename it.
It would be nice if the variable name sind was also renamed automatically.
I'm not sure if this was intended, or if it is a bug, but if it would be easy to support this feature, then I humbly request this to be supported.
Thanks.
The text was updated successfully, but these errors were encountered:
This is not intended. I can't tell for sure without looking into this, but there will be complications with the intrinsic function sind. The bug probably is related to other bugs that are open here with user functions overwriting intrinsic functions. Fixing this category of issues is not a high priority right now, but I am more than happy to review a pull request about this.
I'm using neovim, and installed fortls using Mason.
In this code ...
Going to the function name
sind
in the linereal pure function sind(x)
and renaming it using the symbol rename method in neovim( with fortls LSP) only changes the function name, and everywhere it's used, which is good, but does not change the internal variable namesind
in the linesind = sin(x*((4.0*atan(1.0))/180.0))
I had to manually rename it.
It would be nice if the variable name
sind
was also renamed automatically.I'm not sure if this was intended, or if it is a bug, but if it would be easy to support this feature, then I humbly request this to be supported.
Thanks.
The text was updated successfully, but these errors were encountered: