Auto merge of #24186 - richo:pad-pointers, r=alexcrichton

This pads out the printing of pointers to their native width.

Extracted from and rebased on top of #24144
This commit is contained in:
bors
2015-04-11 01:59:18 +00:00
3 changed files with 40 additions and 4 deletions

View File

@@ -847,9 +847,32 @@ impl Display for char {
#[stable(feature = "rust1", since = "1.0.0")]
impl<T> Pointer for *const T {
fn fmt(&self, f: &mut Formatter) -> Result {
let old_width = f.width;
let old_flags = f.flags;
// The alternate flag is already treated by LowerHex as being special-
// it denotes whether to prefix with 0x. We use it to work out whether
// or not to zero extend, and then unconditionally set it to get the
// prefix.
if f.flags & 1 << (FlagV1::Alternate as u32) > 0 {
f.flags |= 1 << (FlagV1::SignAwareZeroPad as u32);
if let None = f.width {
// The formats need two extra bytes, for the 0x
if cfg!(target_pointer_width = "32") {
f.width = Some(10);
} else {
f.width = Some(18);
}
}
}
f.flags |= 1 << (FlagV1::Alternate as u32);
let ret = LowerHex::fmt(&(*self as usize), f);
f.flags &= !(1 << (FlagV1::Alternate as u32));
f.width = old_width;
f.flags = old_flags;
ret
}
}