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

Fixes bug related to interaction with emacs's native compilation #721

Closed
wants to merge 2 commits into from

Conversation

psibi
Copy link
Contributor

@psibi psibi commented Jul 22, 2022

Description

This change specifically fixes this bug for me:
ardumont/markdown-toc#51

IIUC, # is used to inform the byte-compiler that the expression can
be byte-compiled. But in this case there is nothing to compile since
it's a symbol.

Related Issue

ardumont/markdown-toc#51

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves an existing feature)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the CONTRIBUTING.md document.
  • I have updated the documentation in the README.md file if necessary.
  • I have added an entry to CHANGES.md.
  • I have added tests to cover my changes.
  • All new and existing tests passed (using make test).

psibi added 2 commits July 22, 2022 17:23
This change specifically fixes this bug for me:
ardumont/markdown-toc#51

IIUC, `#` is used to inform the byte-compiler that the expression can
be byte-compiled. But in this case there is nothing to compile since
it's a symbol.
@psibi
Copy link
Contributor Author

psibi commented Jul 26, 2022

This doesn't seem to be the right fix. The bug has returned back. If I restart emacs, it doesn't happen again. This needs more fiddling.

@psibi psibi closed this Jul 26, 2022
@syohex
Copy link
Collaborator

syohex commented Jul 28, 2022

I suppose this is same as #578. Workaround is #578 (comment)

@psibi
Copy link
Contributor Author

psibi commented Jul 28, 2022

@syohex Thanks, I didn't know about that. Is there a more proper fix for it instead of opting out of native compilation for markdown mode ?

@syohex
Copy link
Collaborator

syohex commented Jul 28, 2022

Is there a more proper fix for it instead of opting out of native compilation for markdown mode ?

I have no idea yet.

@psibi
Copy link
Contributor Author

psibi commented Jul 28, 2022

Cool, thanks for the update. The workaround would keep me sane. :-)

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.

2 participants