Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Empty spaces in fortran lexer keyword recognition #1188

Closed
Anteru opened this issue Aug 31, 2019 · 0 comments
Closed

Empty spaces in fortran lexer keyword recognition #1188

Anteru opened this issue Aug 31, 2019 · 0 comments
Labels
A-lexing area: changes to individual lexers changelog-update Items which need to get mentioned in the changelog S-minor severity: minor X-imported imported from Bitbucket

Comments

@Anteru
Copy link
Collaborator

Anteru commented Aug 31, 2019

(Original issue 1484 created by anphi on 2018-11-14T20:37:47.258741+00:00)

Fortran allows arbitrary empty space to be embedded in keywords.
For example, it is legal to write "e n d d o"

While probably nobody would do this, and we don't want to request the lexer to handle that, some people run together certain multi-word keywords, such as "enddo" instead of "end do". The current Fortran lexer does not recognize such them. Could you please make any inter-word space for keywords optional, so that both versions are matched?

@Anteru Anteru added T-feature type: a new feature X-imported imported from Bitbucket S-minor severity: minor labels Aug 31, 2019
@birkenfeld birkenfeld added the A-lexing area: changes to individual lexers label Dec 1, 2019
Anteru pushed a commit that referenced this issue Feb 12, 2021
…rd spaces. (#1677)

* Fix for issue #1188.
* I took the list of keywords affected by this issue from the VIM fortran lexer.
* All the affeced keywords are ending keywords, that is, "end" followed by another keyword without space.
@Anteru Anteru added changelog-update Items which need to get mentioned in the changelog and removed T-feature type: a new feature labels Feb 12, 2021
@Anteru Anteru closed this as completed Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-lexing area: changes to individual lexers changelog-update Items which need to get mentioned in the changelog S-minor severity: minor X-imported imported from Bitbucket
Projects
None yet
Development

No branches or pull requests

2 participants