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

remove oldsmallintmap #5095

Closed
wants to merge 1 commit into from
Closed

remove oldsmallintmap #5095

wants to merge 1 commit into from

Conversation

thestinger
Copy link
Contributor

Closes #4738.

The std::smallintmap module doesn't use an implicit @ box so I just added that explicitly for now to make porting simple.

@nikomatsakis
Copy link
Contributor

@bors retry

bors added a commit that referenced this pull request Feb 25, 2013
Closes #4738.

The `std::smallintmap` module doesn't use an implicit @ box so I just added that explicitly for now to make porting simple.
@bors bors closed this Feb 26, 2013
oli-obk pushed a commit to oli-obk/rust that referenced this pull request May 2, 2020
Fixes issue 5095

fixes rust-lang#5095.

- [x] Followed [lint naming conventions][lint_naming]
- [x] Added passing UI tests (including committed `.stderr` file)
- [x] `cargo test` passes locally
- [x] Executed `cargo dev update_lints`
- [x] Added lint documentation
- [x] Run `cargo dev fmt`

[lint_naming]: https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints

changelog: (internal) warn about collapsible `span_lint_and_then` calls.
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.

4 participants