Skip to content

update match for textmate grammar for underscore prefixed elements - #6317

Closed
bnjjj wants to merge 1 commit into
rust-lang:masterfrom
bnjjj:fix_6311
Closed

update match for textmate grammar for underscore prefixed elements #6317
bnjjj wants to merge 1 commit into
rust-lang:masterfrom
bnjjj:fix_6311

Conversation

@bnjjj

@bnjjj bnjjj commented Oct 21, 2020

Copy link
Copy Markdown
Contributor

close #6311

Do not consider function or macro prefixed by _ as an unused function/macro

…st-lang#6311

Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>
@lnicola

lnicola commented Oct 22, 2020

Copy link
Copy Markdown
Member

r? @dustypomerleau

@lnicola

lnicola commented Oct 22, 2020

Copy link
Copy Markdown
Member

Ah, never mind, I just noticed #6320.

bors Bot added a commit that referenced this pull request Oct 22, 2020
6320: Textmate grammar: prevent line comments from breaking block comments (closes #6281) r=matklad a=dustypomerleau

Fixes #6281.

Previously, line comments were able to break block comments by essentially commenting out the closing `*/`, resulting in a never-ending comment. This PR splits block comments into a separate repository group to fix this problem.

Since the comment scopes also include ignored parameters and inferred types, I've added the change proposed by @bnjjj in #6317, in order to close #6311 as well.

6321: Fix opening module documentation opening parent documentation instead r=matklad a=zacps

The whole path/URL joining code is kind of ugly which is what led to
this, but at the same time I don't really want to rewrite it right
now...

Fixes #6286

Co-authored-by: Dusty Pomerleau <dustypomerleau@users.noreply.github.com>
Co-authored-by: Zac Pullar-Strecker <zacmps@gmail.com>
@lnicola

lnicola commented Oct 22, 2020

Copy link
Copy Markdown
Member

Subsumed by #6320.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Syntax highlighting consider __ident as a comment inside macro arguments

2 participants