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

emacs mode: ']', ')', and '}' close previous open paren/brace/bracket #10262

Closed
pnkfelix opened this issue Nov 4, 2013 · 4 comments
Closed

Comments

@pnkfelix
Copy link
Member

pnkfelix commented Nov 4, 2013

Here is the illustrative example:

fn foo() {
    { bar('}'); }
    { bar(']'); }
    { bar(')'); }
}

The characters within ' characters above are incorrectly treated by src/etc/rust-mode.el as closing the opening paren of the invocation of bar. This confuses Emacs in a number of ways (e.g. obviously paren matching; but also the auto-indentation gets confused too, which is more annoying).

@pnkfelix
Copy link
Member Author

pnkfelix commented Nov 4, 2013

Blocks #8793

@pnkfelix
Copy link
Member Author

pnkfelix commented Nov 4, 2013

probably strongly related to #10190, though I cannot tell for sure if this is a duplicate of that.

johnwalker pushed a commit to johnwalker/rust that referenced this issue Jan 2, 2014
Single quotes are now treated like strings, fixing
issues with stringy brackets.
@shepmaster
Copy link
Member

Using open braces instead is also a problem. This is the natural indenting for that:

fn foo() {
    { bar('{'); }
      { bar('['); }
        { bar('('); }
        }

I'd expect / hope that it's same root cause for both.

@steveklabnik
Copy link
Member

Moved to rust-lang/rust-mode#6

matthiaskrgr pushed a commit to matthiaskrgr/rust that referenced this issue Mar 21, 2024
matthiaskrgr pushed a commit to matthiaskrgr/rust that referenced this issue Mar 21, 2024
Don't emit `doc_markdown` lint for missing backticks if it's inside a quote

Fixes rust-lang#10262.

changelog: Don't emit `doc_markdown` lint for missing backticks if it's inside a quote
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 a pull request may close this issue.

3 participants