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

Recommend autoload instead of require #13923

Closed
wants to merge 1 commit into from
Closed

Recommend autoload instead of require #13923

wants to merge 1 commit into from

Conversation

Rufflewind
Copy link
Contributor

Lazy loading with autoload will result in faster startup time for Emacs
users and is generally the recommended approach for major modes.

Lazy loading with autoload will result in faster startup time for Emacs
users and is generally the recommended approach for major modes.
bors added a commit that referenced this pull request May 4, 2014
Lazy loading with autoload will result in faster startup time for Emacs
users and is generally the recommended approach for major modes.
@bors bors closed this May 5, 2014
@Rufflewind Rufflewind deleted the patch-2 branch May 7, 2014 23:36
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 13, 2023
…s, r=lnicola

Revert "Use ZWNJ to prevent VSCode from forming ligatures between hints and code"

Doesn't actually seem to fix the bug, it seems to be theme dependent and I am not sure what causes it
Reverts rust-lang/rust-analyzer#13886
bors pushed a commit to rust-lang-ci/rust that referenced this pull request Jan 28, 2025
…ang#13929)

Some lifetimes in function return types are not bound to concrete
content and can be set arbitrarily. Clippy should not propose to replace
them by the default `'_` lifetime if such a lifetime cannot be
determined unambigously.

I added a field to the `LifetimeChecker` and `Usage` to flag lifetimes
that cannot be replaced by default ones, but it feels a bit hacky.

Fix rust-lang#13923

changelog: [`needless_lifetimes`]: remove false positives by checking
that lifetimes can indeed be elided
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