1
Fork 0

run rustfmt on various folders

This commit is contained in:
Srinivas Reddy Thatiparthy 2016-10-16 15:41:01 +05:30
parent 6dc035ed91
commit 54e320d4bc
No known key found for this signature in database
GPG key ID: 091C58F4BFC36571
5 changed files with 26 additions and 39 deletions

View file

@ -25,7 +25,9 @@ pub fn run_silent(cmd: &mut Command) {
}; };
if !status.success() { if !status.success() {
fail(&format!("command did not execute successfully: {:?}\n\ fail(&format!("command did not execute successfully: {:?}\n\
expected success, got: {}", cmd, status)); expected success, got: {}",
cmd,
status));
} }
} }
@ -65,7 +67,9 @@ pub fn output(cmd: &mut Command) -> String {
}; };
if !output.status.success() { if !output.status.success() {
panic!("command did not execute successfully: {:?}\n\ panic!("command did not execute successfully: {:?}\n\
expected success, got: {}", cmd, output.status); expected success, got: {}",
cmd,
output.status);
} }
String::from_utf8(output.stdout).unwrap() String::from_utf8(output.stdout).unwrap()
} }

View file

@ -57,11 +57,7 @@ impl<T> RawVec<T> {
pub fn new() -> Self { pub fn new() -> Self {
unsafe { unsafe {
// !0 is usize::MAX. This branch should be stripped at compile time. // !0 is usize::MAX. This branch should be stripped at compile time.
let cap = if mem::size_of::<T>() == 0 { let cap = if mem::size_of::<T>() == 0 { !0 } else { 0 };
!0
} else {
0
};
// heap::EMPTY doubles as "unallocated" and "zero-sized allocation" // heap::EMPTY doubles as "unallocated" and "zero-sized allocation"
RawVec { RawVec {
@ -209,11 +205,7 @@ impl<T> RawVec<T> {
let (new_cap, ptr) = if self.cap == 0 { let (new_cap, ptr) = if self.cap == 0 {
// skip to 4 because tiny Vec's are dumb; but not if that would cause overflow // skip to 4 because tiny Vec's are dumb; but not if that would cause overflow
let new_cap = if elem_size > (!0) / 8 { let new_cap = if elem_size > (!0) / 8 { 1 } else { 4 };
1
} else {
4
};
let ptr = heap::allocate(new_cap * elem_size, align); let ptr = heap::allocate(new_cap * elem_size, align);
(new_cap, ptr) (new_cap, ptr)
} else { } else {
@ -347,7 +339,7 @@ impl<T> RawVec<T> {
let elem_size = mem::size_of::<T>(); let elem_size = mem::size_of::<T>();
// Nothing we can really do about these checks :( // Nothing we can really do about these checks :(
let required_cap = used_cap.checked_add(needed_extra_cap) let required_cap = used_cap.checked_add(needed_extra_cap)
.expect("capacity overflow"); .expect("capacity overflow");
// Cannot overflow, because `cap <= isize::MAX`, and type of `cap` is `usize`. // Cannot overflow, because `cap <= isize::MAX`, and type of `cap` is `usize`.
let double_cap = self.cap * 2; let double_cap = self.cap * 2;
// `double_cap` guarantees exponential growth. // `double_cap` guarantees exponential growth.

View file

@ -35,12 +35,8 @@ fn main() {
// that the feature set used by std is the same across all // that the feature set used by std is the same across all
// targets, which means we have to build the alloc_jemalloc crate // targets, which means we have to build the alloc_jemalloc crate
// for targets like emscripten, even if we don't use it. // for targets like emscripten, even if we don't use it.
if target.contains("rumprun") || if target.contains("rumprun") || target.contains("bitrig") || target.contains("openbsd") ||
target.contains("bitrig") || target.contains("msvc") || target.contains("emscripten") {
target.contains("openbsd") ||
target.contains("msvc") ||
target.contains("emscripten")
{
println!("cargo:rustc-cfg=dummy_jemalloc"); println!("cargo:rustc-cfg=dummy_jemalloc");
return; return;
} }
@ -64,16 +60,16 @@ fn main() {
// only msvc returns None for ar so unwrap is okay // only msvc returns None for ar so unwrap is okay
let ar = build_helper::cc2ar(compiler.path(), &target).unwrap(); let ar = build_helper::cc2ar(compiler.path(), &target).unwrap();
let cflags = compiler.args() let cflags = compiler.args()
.iter() .iter()
.map(|s| s.to_str().unwrap()) .map(|s| s.to_str().unwrap())
.collect::<Vec<_>>() .collect::<Vec<_>>()
.join(" "); .join(" ");
let mut stack = src_dir.join("../jemalloc") let mut stack = src_dir.join("../jemalloc")
.read_dir() .read_dir()
.unwrap() .unwrap()
.map(|e| e.unwrap()) .map(|e| e.unwrap())
.collect::<Vec<_>>(); .collect::<Vec<_>>();
while let Some(entry) = stack.pop() { while let Some(entry) = stack.pop() {
let path = entry.path(); let path = entry.path();
if entry.file_type().unwrap().is_dir() { if entry.file_type().unwrap().is_dir() {
@ -155,10 +151,10 @@ fn main() {
run(&mut cmd); run(&mut cmd);
run(Command::new("make") run(Command::new("make")
.current_dir(&build_dir) .current_dir(&build_dir)
.arg("build_lib_static") .arg("build_lib_static")
.arg("-j") .arg("-j")
.arg(env::var("NUM_JOBS").expect("NUM_JOBS was not set"))); .arg(env::var("NUM_JOBS").expect("NUM_JOBS was not set")));
if target.contains("windows") { if target.contains("windows") {
println!("cargo:rustc-link-lib=static=jemalloc"); println!("cargo:rustc-link-lib=static=jemalloc");

View file

@ -221,11 +221,7 @@ mod imp {
HEAP_REALLOC_IN_PLACE_ONLY, HEAP_REALLOC_IN_PLACE_ONLY,
ptr as LPVOID, ptr as LPVOID,
size as SIZE_T) as *mut u8; size as SIZE_T) as *mut u8;
if new.is_null() { if new.is_null() { old_size } else { size }
old_size
} else {
size
}
} else { } else {
old_size old_size
} }

View file

@ -302,9 +302,8 @@ mod tests {
let arena = Wrap(TypedArena::new()); let arena = Wrap(TypedArena::new());
let result = arena.alloc_outer(|| { let result =
Outer { inner: arena.alloc_inner(|| Inner { value: 10 }) } arena.alloc_outer(|| Outer { inner: arena.alloc_inner(|| Inner { value: 10 }) });
});
assert_eq!(result.inner.value, 10); assert_eq!(result.inner.value, 10);
} }