Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
/vimPublic

runtime(leex): add leex filetype support#18832

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
jparise wants to merge3 commits intovim:master
base:master
Choose a base branch
Loading
fromjparise:leex

Conversation

@jparise
Copy link
Contributor

leex is the lexical analyzer generator for Erlang. Its input file format follows a section-based structure and uses the.xrl file extension.

https://www.erlang.org/doc/apps/parsetools/leex.html

This initial work includes file detection, an ftplugin (which inherits the Erlang configuration), and a syntax definition.

leex is the lexical analyzer generator for Erlang. Its input file formatfollows a section-based structure and uses the `.xrl` file extension.https://www.erlang.org/doc/apps/parsetools/leex.htmlThis initial work includes file detection, an ftplugin (which inheritsthe Erlang configuration), and a syntax definition.
@jparise
Copy link
ContributorAuthor

@hcs42 this implements what we discussed in#18819 using a newleex filetype that covers the leex syntax and includes the Erlang syntax selectively.

runtime/syntax/kotlin.vim@udalov
runtime/syntax/kdl.vim@imsnif@jiangyinzuo
runtime/syntax/krl.vim@KnoP-01
runtime/syntax/leex.vim@jparise
Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

@hcs42 I wasn't sure if you wanted to maintain these files, as well. I'm happy to reassign them to you or co-maintain them, any way you prefer.

" Maintainer: Jon Parise <jon@indelible.org>
" Last Change: 2025 Nov 28
" Filenames: *.xrl
" URL: https://www.erlang.org/doc/apps/parsetools/leex.html
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

This header field is used for the upstream repo or download. I'd just add this as a comment below the header block or in a "References: " field likecs.vim etc.

jparise reacted with thumbs up emoji
Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

That sounds good. I've also seenRepository: used for upstream repos.

finish
endif

syninclude@Erlangsyntax/erlang.vim
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

There's probably exceptions in the repo but this cluster should also use theleex prefix - so,@leexErlang.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Good idea. I'll fix up some of my other projects to do this, too. 🙂

Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

@dkearns I also see that many syntax files use<sfile>:p:h/... to explicitly reference the runtime-included file. Because I'm not referencing any specific Erlang syntax elements, I think this could go either way here (because it's presumably fine if a higher-precedence runtimepatherlang syntax is used). Any preference?

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@dkearnsdkearnsdkearns left review comments

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jparise@dkearns

[8]ページ先頭

©2009-2025 Movatter.jp