bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

use && for fn() even in legacy mode to work around mem leak

Browse source
This commit is contained in:
Niko Matsakis 2012-09-19 06:56:29 -07:00
parent 8d4928f780
commit 53abb5edbd
1 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

14
src/rustc/middle/ty.rs
View file

@ -1090,12 +1090,22 @@ pure fn mach_sty(cfg: @session::config, t: t) -> sty {
}
fn default_arg_mode_for_ty(tcx: ctxt, ty: ty::t) -> ast::rmode {
if ty::type_is_immediate(ty) {
return if ty::type_is_immediate(ty) {
ast::by_val
} else if tcx.legacy_modes {
} else if tcx.legacy_modes || type_is_fn(ty) {
// ^^^^^^^^^^^^^^
// FIXME(#2202) --- We retain by-ref by default to workaround a memory
// leak that otherwise results when @fn is upcast to &fn.
ast::by_ref
} else {
ast::by_copy
};
fn type_is_fn(ty: t) -> bool {
match get(ty).sty {
ty_fn(*) => true,
_ => false
}
}
}