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

debuginfo: Add a test case for issue #12886. #13556

Merged
merged 1 commit into from
Apr 16, 2014

Conversation

michaelwoerister
Copy link
Member

This is a test case verifying that issue #12886 was indeed fixed by PR #13441 from last week.
Fixes #12886.

@michaelwoerister
Copy link
Member Author

Thanks @alexcrichton.
Also thanks, @jdm, for providing the test case.

bors added a commit that referenced this pull request Apr 16, 2014
…chton

This is a test case verifying that issue #12886 was indeed fixed by PR #13441 from last week.
Fixes #12886.
@bors bors closed this Apr 16, 2014
@bors bors merged commit 7c042cd into rust-lang:master Apr 16, 2014
@michaelwoerister michaelwoerister deleted the various-fixes branch February 7, 2022 09:28
notriddle pushed a commit to notriddle/rust that referenced this pull request Nov 10, 2022
docs: fix adoc links

Correct rust-lang#13536 with adoc link syntax
flip1995 pushed a commit to flip1995/rust that referenced this pull request Nov 28, 2024
…ang#13691)

In preparation of rust-lang#13556, I want to remove the dependency on
`clippy_config`, as I don't think that we want to publish that for
outside consumers. To do this the 2 dependecies on `clippy_config` had
to be removed:

1. The MSRV implementation was in `clippy_config`, but was required in
`qualify_min_const`. I think exposing the MSRV infrastructure and the
MSRVs we defined might also be helpful for `clippy_utils` users. I don't
see why it should not be able to live in `clippy_utils` from a technical
point of few.
2. The `create_disallowed_map` function that took in a
`clippy_utils::types::DisallowedPath` is moved to the `DisallowedPath`
implementation. This also fits there and is only useful for Clippy and
not in `clippy_utils` for external consumers.

`clippy_config` now depends in `clippy_utils`, so the dependecy just got
reversed. But having the `clippy_utils` crate as the base of the
dependency tree in Clippy makes sense.

changelog: none
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.

Line-based breakpoints set dozens of unwanted ones
3 participants