]> git.lizzy.rs Git - rust.git/commitdiff
auto merge of #19388 : nick29581/rust/rc-show, r=alexcrichto
authorbors <bors@rust-lang.org>
Thu, 1 Jan 2015 06:36:24 +0000 (06:36 +0000)
committerbors <bors@rust-lang.org>
Thu, 1 Jan 2015 06:36:24 +0000 (06:36 +0000)
r? @huonw or @alexcrichton

Apparently, we have previously rejected an RFC like this. However, since then we removed `{:?}` and so without this debugging gets really difficult as soon as there is a RefCell anywhere, so I believe there is more benefit to adding these impls than there was before. By using "try_borrow" we can avoid panicing in `Show` (I think).

@ huon in response to a comment in #19254: I noticed that `drop()` checks for the ptr being null, so I checked here too. Now I am checking for both, if you're confident I can change to only checking `strong()`.

1  2 
src/librustc_resolve/lib.rs

Simple merge