Auto merge of #26456 - Manishearth:rollup, r=Manishearth
- Successful merges: #26410, #26432, #26434, #26439, #26451, #26452 - Failed merges:
This commit is contained in:
commit
40d19bf641
@ -1,12 +1,14 @@
|
||||
# The Rust Programming Language
|
||||
|
||||
Rust is a systems programming language that is fast, memory safe and
|
||||
multithreaded, but does not employ a garbage collector or otherwise
|
||||
impose significant runtime overhead.
|
||||
Rust is a fast systems programming language that guarantees
|
||||
memory safety and offers painless concurrency ([no data races]).
|
||||
It does not employ a garbage collector and has minimal runtime overhead.
|
||||
|
||||
This repo contains the code for `rustc`, the Rust compiler, as well
|
||||
as standard libraries, tools and documentation for Rust.
|
||||
|
||||
[no data races]: http://blog.rust-lang.org/2015/04/10/Fearless-Concurrency.html
|
||||
|
||||
## Quick Start
|
||||
|
||||
Read ["Installing Rust"] from [The Book].
|
||||
|
@ -77,8 +77,7 @@ ERR_IDX_GEN = $(RPATH_VAR2_T_$(CFG_BUILD)_H_$(CFG_BUILD)) $(ERR_IDX_GEN_EXE)
|
||||
|
||||
D := $(S)src/doc
|
||||
|
||||
# FIXME (#25705) eventually may want to put error-index target back here.
|
||||
DOC_TARGETS := trpl style
|
||||
DOC_TARGETS := trpl style error-index
|
||||
COMPILER_DOC_TARGETS :=
|
||||
DOC_L10N_TARGETS :=
|
||||
|
||||
|
@ -17,6 +17,7 @@ extern crate serialize as rustc_serialize;
|
||||
use std::collections::BTreeMap;
|
||||
use std::fs::{read_dir, File};
|
||||
use std::io::{Read, Write};
|
||||
use std::env;
|
||||
use std::path::Path;
|
||||
use std::error::Error;
|
||||
|
||||
@ -106,7 +107,8 @@ r##"<!DOCTYPE html>
|
||||
}
|
||||
|
||||
fn main_with_result() -> Result<(), Box<Error>> {
|
||||
let metadata_dir = get_metadata_dir();
|
||||
let build_arch = try!(env::var("CFG_BUILD"));
|
||||
let metadata_dir = get_metadata_dir(&build_arch);
|
||||
let err_map = try!(load_all_errors(&metadata_dir));
|
||||
try!(render_error_page(&err_map, Path::new("doc/error-index.html")));
|
||||
Ok(())
|
||||
|
@ -512,6 +512,11 @@ impl<T: ?Sized> PartialOrd for *mut T {
|
||||
#[unstable(feature = "unique", reason = "needs an RFC to flesh out design")]
|
||||
pub struct Unique<T: ?Sized> {
|
||||
pointer: NonZero<*const T>,
|
||||
// NOTE: this marker has no consequences for variance, but is necessary
|
||||
// for dropck to understand that we logically own a `T`.
|
||||
//
|
||||
// For details, see:
|
||||
// https://github.com/rust-lang/rfcs/blob/master/text/0769-sound-generic-drop.md#phantom-data
|
||||
_marker: PhantomData<T>,
|
||||
}
|
||||
|
||||
|
@ -730,9 +730,11 @@ struct ListNode {
|
||||
This type cannot have a well-defined size, because it needs to be arbitrarily
|
||||
large (since we would be able to nest `ListNode`s to any depth). Specifically,
|
||||
|
||||
size of `ListNode` = 1 byte for `head`
|
||||
+ 1 byte for the discriminant of the `Option`
|
||||
+ size of `ListNode`
|
||||
```plain
|
||||
size of `ListNode` = 1 byte for `head`
|
||||
+ 1 byte for the discriminant of the `Option`
|
||||
+ size of `ListNode`
|
||||
```
|
||||
|
||||
One way to fix this is by wrapping `ListNode` in a `Box`, like so:
|
||||
|
||||
|
@ -331,7 +331,7 @@ pub mod builtin {
|
||||
|
||||
/// A macro which expands to the line number on which it was invoked.
|
||||
///
|
||||
/// The expanded expression has type `usize`, and the returned line is not
|
||||
/// The expanded expression has type `u32`, and the returned line is not
|
||||
/// the invocation of the `line!()` macro itself, but rather the first macro
|
||||
/// invocation leading up to the invocation of the `line!()` macro.
|
||||
///
|
||||
@ -346,7 +346,7 @@ pub mod builtin {
|
||||
|
||||
/// A macro which expands to the column number on which it was invoked.
|
||||
///
|
||||
/// The expanded expression has type `usize`, and the returned column is not
|
||||
/// The expanded expression has type `u32`, and the returned column is not
|
||||
/// the invocation of the `column!()` macro itself, but rather the first macro
|
||||
/// invocation leading up to the invocation of the `column!()` macro.
|
||||
///
|
||||
|
@ -14,24 +14,18 @@
|
||||
//! currently always a crate name.
|
||||
|
||||
use std::collections::BTreeMap;
|
||||
use std::env;
|
||||
use std::path::PathBuf;
|
||||
use std::fs::{read_dir, create_dir_all, OpenOptions, File};
|
||||
use std::io::{Read, Write};
|
||||
use std::fs::{remove_file, create_dir_all, File};
|
||||
use std::io::Write;
|
||||
use std::error::Error;
|
||||
use rustc_serialize::json::{self, as_json};
|
||||
use rustc_serialize::json::as_json;
|
||||
|
||||
use codemap::Span;
|
||||
use ext::base::ExtCtxt;
|
||||
use diagnostics::plugin::{ErrorMap, ErrorInfo};
|
||||
|
||||
pub use self::Uniqueness::*;
|
||||
|
||||
// Default metadata directory to use for extended error JSON.
|
||||
const ERROR_METADATA_DIR_DEFAULT: &'static str = "tmp/extended-errors";
|
||||
|
||||
// The name of the environment variable that sets the metadata dir.
|
||||
const ERROR_METADATA_VAR: &'static str = "ERROR_METADATA_DIR";
|
||||
const ERROR_METADATA_PREFIX: &'static str = "tmp/extended-errors";
|
||||
|
||||
/// JSON encodable/decodable version of `ErrorInfo`.
|
||||
#[derive(PartialEq, RustcDecodable, RustcEncodable)]
|
||||
@ -61,84 +55,32 @@ impl ErrorLocation {
|
||||
}
|
||||
}
|
||||
|
||||
/// Type for describing the uniqueness of a set of error codes, as returned by `check_uniqueness`.
|
||||
pub enum Uniqueness {
|
||||
/// All errors in the set checked are unique according to the metadata files checked.
|
||||
Unique,
|
||||
/// One or more errors in the set occur in another metadata file.
|
||||
/// This variant contains the first duplicate error code followed by the name
|
||||
/// of the metadata file where the duplicate appears.
|
||||
Duplicate(String, String)
|
||||
/// Get the directory where metadata for a given `prefix` should be stored.
|
||||
///
|
||||
/// See `output_metadata`.
|
||||
pub fn get_metadata_dir(prefix: &str) -> PathBuf {
|
||||
PathBuf::from(ERROR_METADATA_PREFIX).join(prefix)
|
||||
}
|
||||
|
||||
/// Get the directory where metadata files should be stored.
|
||||
pub fn get_metadata_dir() -> PathBuf {
|
||||
match env::var(ERROR_METADATA_VAR) {
|
||||
Ok(v) => From::from(v),
|
||||
Err(_) => From::from(ERROR_METADATA_DIR_DEFAULT)
|
||||
}
|
||||
/// Map `name` to a path in the given directory: <directory>/<name>.json
|
||||
fn get_metadata_path(directory: PathBuf, name: &str) -> PathBuf {
|
||||
directory.join(format!("{}.json", name))
|
||||
}
|
||||
|
||||
/// Get the path where error metadata for the set named by `name` should be stored.
|
||||
fn get_metadata_path(name: &str) -> PathBuf {
|
||||
get_metadata_dir().join(format!("{}.json", name))
|
||||
}
|
||||
|
||||
/// Check that the errors in `err_map` aren't present in any metadata files in the
|
||||
/// metadata directory except the metadata file corresponding to `name`.
|
||||
pub fn check_uniqueness(name: &str, err_map: &ErrorMap) -> Result<Uniqueness, Box<Error>> {
|
||||
let metadata_dir = get_metadata_dir();
|
||||
let metadata_path = get_metadata_path(name);
|
||||
|
||||
// Create the error directory if it does not exist.
|
||||
try!(create_dir_all(&metadata_dir));
|
||||
|
||||
// Check each file in the metadata directory.
|
||||
for entry in try!(read_dir(&metadata_dir)) {
|
||||
let path = try!(entry).path();
|
||||
|
||||
// Skip any existing file for this set.
|
||||
if path == metadata_path {
|
||||
continue;
|
||||
}
|
||||
|
||||
// Read the metadata file into a string.
|
||||
let mut metadata_str = String::new();
|
||||
try!(
|
||||
File::open(&path).and_then(|mut f|
|
||||
f.read_to_string(&mut metadata_str))
|
||||
);
|
||||
|
||||
// Parse the JSON contents.
|
||||
let metadata: ErrorMetadataMap = try!(json::decode(&metadata_str));
|
||||
|
||||
// Check for duplicates.
|
||||
for err in err_map.keys() {
|
||||
let err_code = err.as_str();
|
||||
if metadata.contains_key(err_code) {
|
||||
return Ok(Duplicate(
|
||||
err_code.to_string(),
|
||||
path.to_string_lossy().into_owned()
|
||||
));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Ok(Unique)
|
||||
}
|
||||
|
||||
/// Write metadata for the errors in `err_map` to disk, to a file corresponding to `name`.
|
||||
pub fn output_metadata(ecx: &ExtCtxt, name: &str, err_map: &ErrorMap)
|
||||
/// Write metadata for the errors in `err_map` to disk, to a file corresponding to `prefix/name`.
|
||||
///
|
||||
/// For our current purposes the prefix is the target architecture and the name is a crate name.
|
||||
/// If an error occurs steps will be taken to ensure that no file is created.
|
||||
pub fn output_metadata(ecx: &ExtCtxt, prefix: &str, name: &str, err_map: &ErrorMap)
|
||||
-> Result<(), Box<Error>>
|
||||
{
|
||||
let metadata_path = get_metadata_path(name);
|
||||
// Create the directory to place the file in.
|
||||
let metadata_dir = get_metadata_dir(prefix);
|
||||
try!(create_dir_all(&metadata_dir));
|
||||
|
||||
// Open the dump file.
|
||||
let mut dump_file = try!(OpenOptions::new()
|
||||
.write(true)
|
||||
.create(true)
|
||||
.open(&metadata_path)
|
||||
);
|
||||
// Open the metadata file.
|
||||
let metadata_path = get_metadata_path(metadata_dir, name);
|
||||
let mut metadata_file = try!(File::create(&metadata_path));
|
||||
|
||||
// Construct a serializable map.
|
||||
let json_map = err_map.iter().map(|(k, &ErrorInfo { description, use_site })| {
|
||||
@ -150,6 +92,10 @@ pub fn output_metadata(ecx: &ExtCtxt, name: &str, err_map: &ErrorMap)
|
||||
(key, value)
|
||||
}).collect::<ErrorMetadataMap>();
|
||||
|
||||
try!(write!(&mut dump_file, "{}", as_json(&json_map)));
|
||||
Ok(())
|
||||
// Write the data to the file, deleting it if the write fails.
|
||||
let result = write!(&mut metadata_file, "{}", as_json(&json_map));
|
||||
if result.is_err() {
|
||||
try!(remove_file(&metadata_path));
|
||||
}
|
||||
Ok(try!(result))
|
||||
}
|
||||
|
@ -10,6 +10,7 @@
|
||||
|
||||
use std::cell::RefCell;
|
||||
use std::collections::BTreeMap;
|
||||
use std::env;
|
||||
|
||||
use ast;
|
||||
use ast::{Ident, Name, TokenTree};
|
||||
@ -20,6 +21,8 @@ use parse::token;
|
||||
use ptr::P;
|
||||
use util::small_vector::SmallVector;
|
||||
|
||||
use diagnostics::metadata::output_metadata;
|
||||
|
||||
// Maximum width of any line in an extended error description (inclusive).
|
||||
const MAX_DESCRIPTION_WIDTH: usize = 80;
|
||||
|
||||
@ -154,7 +157,7 @@ pub fn expand_build_diagnostic_array<'cx>(ecx: &'cx mut ExtCtxt,
|
||||
token_tree: &[TokenTree])
|
||||
-> Box<MacResult+'cx> {
|
||||
assert_eq!(token_tree.len(), 3);
|
||||
let (_crate_name, name) = match (&token_tree[0], &token_tree[2]) {
|
||||
let (crate_name, name) = match (&token_tree[0], &token_tree[2]) {
|
||||
(
|
||||
// Crate name.
|
||||
&ast::TtToken(_, token::Ident(ref crate_name, _)),
|
||||
@ -164,9 +167,18 @@ pub fn expand_build_diagnostic_array<'cx>(ecx: &'cx mut ExtCtxt,
|
||||
_ => unreachable!()
|
||||
};
|
||||
|
||||
// FIXME (#25705): we used to ensure error code uniqueness and
|
||||
// output error description JSON metadata here, but the approach
|
||||
// employed was too brittle.
|
||||
// Output error metadata to `tmp/extended-errors/<target arch>/<crate name>.json`
|
||||
let target_triple = env::var("CFG_COMPILER_HOST_TRIPLE")
|
||||
.ok().expect("unable to determine target arch from $CFG_COMPILER_HOST_TRIPLE");
|
||||
|
||||
with_registered_diagnostics(|diagnostics| {
|
||||
if let Err(e) = output_metadata(ecx, &target_triple, crate_name, &diagnostics) {
|
||||
ecx.span_bug(span, &format!(
|
||||
"error writing metadata for triple `{}` and crate `{}`, error: {}, cause: {:?}",
|
||||
target_triple, crate_name, e.description(), e.cause()
|
||||
));
|
||||
}
|
||||
});
|
||||
|
||||
// Construct the output expression.
|
||||
let (count, expr) =
|
||||
|
14
src/test/compile-fail/issue-18058.rs
Normal file
14
src/test/compile-fail/issue-18058.rs
Normal file
@ -0,0 +1,14 @@
|
||||
// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
|
||||
// file at the top-level directory of this distribution and at
|
||||
// http://rust-lang.org/COPYRIGHT.
|
||||
//
|
||||
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
|
||||
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
|
||||
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
|
||||
// option. This file may not be copied, modified, or distributed
|
||||
// except according to those terms.
|
||||
|
||||
impl Undefined {}
|
||||
//~^ ERROR use of undeclared type name `Undefined`
|
||||
|
||||
fn main() {}
|
27
src/test/run-pass/issue-24227.rs
Normal file
27
src/test/run-pass/issue-24227.rs
Normal file
@ -0,0 +1,27 @@
|
||||
// Copyright 2014 The Rust Project Developers. See the COPYRIGHT
|
||||
// file at the top-level directory of this distribution and at
|
||||
// http://rust-lang.org/COPYRIGHT.
|
||||
//
|
||||
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
|
||||
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
|
||||
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
|
||||
// option. This file may not be copied, modified, or distributed
|
||||
// except according to those terms.
|
||||
|
||||
// This resulted in an ICE. Test for future-proofing
|
||||
// Issue #24227
|
||||
|
||||
#![allow(unused)]
|
||||
|
||||
struct Foo<'a> {
|
||||
x: &'a u8
|
||||
}
|
||||
|
||||
impl<'a> Foo<'a> {
|
||||
fn foo() {
|
||||
let mut tmp: Self;
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
fn main() {}
|
Loading…
x
Reference in New Issue
Block a user