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

bootstrap compiler ignores unknown names #154

Closed
tohava opened this issue Aug 26, 2010 · 1 comment
Closed

bootstrap compiler ignores unknown names #154

tohava opened this issue Aug 26, 2010 · 1 comment

Comments

@tohava
Copy link
Contributor

tohava commented Aug 26, 2010

// BEGIN CODE
// -- rust --

fn main() {
log "doing";
this_does_nothing_what_the;
log "boing";
}
// END CODE

The following code compiles, wouldn't it be wiser to have the compiler report an error about "this_does_nothing_what_the"?

@graydon
Copy link
Contributor

graydon commented Sep 30, 2010

Closed by 74cb950.

oli-obk pushed a commit to oli-obk/rust that referenced this issue Jul 19, 2017
partially update for latest rustc nightly
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
Add IP v6 group membership options to OpenBSD build
Aaron1011 pushed a commit to Aaron1011/rust that referenced this issue Oct 26, 2020
…ft-606eee5

Bump cranelift from `fe081d3` to `606eee5`
bors pushed a commit to rust-lang-ci/rust that referenced this issue Oct 1, 2021
…nt-type

Change vectors to be generic over element type.
antoyo added a commit to antoyo/rust that referenced this issue Jun 7, 2022
Fix error related to var tracking assignments
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Apr 10, 2024
Clean up tests/ui by removing `does-nothing.rs`

In [a previous PR](rust-lang#123297 (comment)), it was suggested that this test be removed:

> it's testing a basic diagnostic for an unknown variable (added over a decade ago for rust-lang#154) that is already covered by probably dozens or hundreds of other tests.

It was then suggested that [opening a new PR](rust-lang#123563 (comment)) for this would be more organized.

I'm setting this as a draft, as:

1. The tests/ui directory is rather disorganized, a large quantity of tests are not even contained inside their own directories. This PR could turn into "clean up the UI tests directory", if I were to place everything into categories (for example, everything related to CLI flags could get placed in a cli directory).
2. This will have a merge conflict with rust-lang#123563 should that get merged. I trust that _this time_, I won't run into [The Incident](rust-lang#123297 (comment)) while rebasing. Edit: Yay, I did it properly!
rust-timer added a commit to rust-lang-ci/rust that referenced this issue Apr 10, 2024
Rollup merge of rust-lang#123568 - Oneirical:delete-tests, r=wesleywiser

Clean up tests/ui by removing `does-nothing.rs`

In [a previous PR](rust-lang#123297 (comment)), it was suggested that this test be removed:

> it's testing a basic diagnostic for an unknown variable (added over a decade ago for rust-lang#154) that is already covered by probably dozens or hundreds of other tests.

It was then suggested that [opening a new PR](rust-lang#123563 (comment)) for this would be more organized.

I'm setting this as a draft, as:

1. The tests/ui directory is rather disorganized, a large quantity of tests are not even contained inside their own directories. This PR could turn into "clean up the UI tests directory", if I were to place everything into categories (for example, everything related to CLI flags could get placed in a cli directory).
2. This will have a merge conflict with rust-lang#123563 should that get merged. I trust that _this time_, I won't run into [The Incident](rust-lang#123297 (comment)) while rebasing. Edit: Yay, I did it properly!
celinval pushed a commit to celinval/rust-dev that referenced this issue Jun 4, 2024
* Improve some regression tests and add some more.

* Add more assertions to a test.

* Revert changes to trait regressions.

* Add comments explaining dereferencing raw address test.
This issue was closed.
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

No branches or pull requests

2 participants