Skip to content

Commit

Permalink
Use byte literals in libstd
Browse files Browse the repository at this point in the history
  • Loading branch information
nham committed Aug 6, 2014
1 parent dfdea3f commit 3fb78e2
Show file tree
Hide file tree
Showing 13 changed files with 52 additions and 54 deletions.
12 changes: 6 additions & 6 deletions src/libstd/ascii.rs
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ impl Ascii {
/// Check if the character is a space or horizontal tab
#[inline]
pub fn is_blank(&self) -> bool {
self.chr == ' ' as u8 || self.chr == '\t' as u8
self.chr == b' ' || self.chr == b'\t'
}

/// Check if the character is a control character
Expand Down Expand Up @@ -150,7 +150,7 @@ impl Ascii {
/// Checks if the character is lowercase
#[inline]
pub fn is_lowercase(&self) -> bool {
(self.chr - 'a' as u8) < 26
(self.chr - b'a') < 26
}

#[inline]
Expand All @@ -163,7 +163,7 @@ impl Ascii {
/// Checks if the character is uppercase
#[inline]
pub fn is_uppercase(&self) -> bool {
(self.chr - 'A' as u8) < 26
(self.chr - b'A') < 26
}

/// Checks if the character is punctuation
Expand All @@ -175,7 +175,7 @@ impl Ascii {
/// Checks if the character is a valid hex digit
#[inline]
pub fn is_hex(&self) -> bool {
self.is_digit() || ((self.chr | 32u8) - 'a' as u8) < 6
self.is_digit() || ((self.chr | 32u8) - b'a') < 6
}
}

Expand Down Expand Up @@ -792,13 +792,13 @@ mod tests {

#[test]
fn test_to_string() {
let s = Ascii{ chr: 't' as u8 }.to_string();
let s = Ascii{ chr: b't' }.to_string();
assert_eq!(s, "t".to_string());
}

#[test]
fn test_show() {
let c = Ascii { chr: 't' as u8 };
let c = Ascii { chr: b't' };
assert_eq!(format!("{}", c), "t".to_string());
}
}
2 changes: 1 addition & 1 deletion src/libstd/dynamic_lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ impl DynamicLibrary {
}

fn separator() -> u8 {
if cfg!(windows) {';' as u8} else {':' as u8}
if cfg!(windows) {b';'} else {b':'}
}

/// Returns the current search path for dynamic libraries being used by this
Expand Down
14 changes: 7 additions & 7 deletions src/libstd/io/buffered.rs
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,7 @@ impl<W: Writer> LineBufferedWriter<W> {

impl<W: Writer> Writer for LineBufferedWriter<W> {
fn write(&mut self, buf: &[u8]) -> IoResult<()> {
match buf.iter().rposition(|&b| b == '\n' as u8) {
match buf.iter().rposition(|&b| b == b'\n') {
Some(i) => {
try!(self.inner.write(buf.slice_to(i + 1)));
try!(self.inner.flush());
Expand Down Expand Up @@ -524,15 +524,15 @@ mod test {
assert_eq!(writer.get_ref().get_ref(), &[]);
writer.flush().unwrap();
assert_eq!(writer.get_ref().get_ref(), &[0, 1]);
writer.write([0, '\n' as u8, 1, '\n' as u8, 2]).unwrap();
writer.write([0, b'\n', 1, b'\n', 2]).unwrap();
assert_eq!(writer.get_ref().get_ref(),
&[0, 1, 0, '\n' as u8, 1, '\n' as u8]);
&[0, 1, 0, b'\n', 1, b'\n']);
writer.flush().unwrap();
assert_eq!(writer.get_ref().get_ref(),
&[0, 1, 0, '\n' as u8, 1, '\n' as u8, 2]);
writer.write([3, '\n' as u8]).unwrap();
&[0, 1, 0, b'\n', 1, b'\n', 2]);
writer.write([3, b'\n']).unwrap();
assert_eq!(writer.get_ref().get_ref(),
&[0, 1, 0, '\n' as u8, 1, '\n' as u8, 2, 3, '\n' as u8]);
&[0, 1, 0, b'\n', 1, b'\n', 2, 3, b'\n']);
}

#[test]
Expand Down Expand Up @@ -579,7 +579,7 @@ mod test {

#[test]
fn test_chars() {
let buf = [195u8, 159u8, 'a' as u8];
let buf = [195u8, 159u8, b'a'];
let mut reader = BufferedReader::with_capacity(1, BufReader::new(buf));
let mut it = reader.chars();
assert_eq!(it.next(), Some(Ok('ß')));
Expand Down
4 changes: 2 additions & 2 deletions src/libstd/io/fs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1241,8 +1241,8 @@ mod test {
let mut cur = [0u8, .. 2];
for f in files {
let stem = f.filestem_str().unwrap();
let root = stem.as_bytes()[0] - ('0' as u8);
let name = stem.as_bytes()[1] - ('0' as u8);
let root = stem.as_bytes()[0] - b'0';
let name = stem.as_bytes()[1] - b'0';
assert!(cur[root as uint] < name);
cur[root as uint] = name;
}
Expand Down
4 changes: 2 additions & 2 deletions src/libstd/io/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1103,7 +1103,7 @@ pub trait Writer {
/// that the `write` method is used specifically instead.
#[inline]
fn write_line(&mut self, s: &str) -> IoResult<()> {
self.write_str(s).and_then(|()| self.write(['\n' as u8]))
self.write_str(s).and_then(|()| self.write([b'\n']))
}

/// Write a single char, encoded as UTF-8.
Expand Down Expand Up @@ -1442,7 +1442,7 @@ pub trait Buffer: Reader {
/// Additionally, this function can fail if the line of input read is not a
/// valid UTF-8 sequence of bytes.
fn read_line(&mut self) -> IoResult<String> {
self.read_until('\n' as u8).and_then(|line|
self.read_until(b'\n').and_then(|line|
match String::from_utf8(line) {
Ok(s) => Ok(s),
Err(_) => Err(standard_error(InvalidInput)),
Expand Down
12 changes: 6 additions & 6 deletions src/libstd/io/net/ip.rs
Original file line number Diff line number Diff line change
Expand Up @@ -161,12 +161,12 @@ impl<'a> Parser<'a> {
fn parse_digit(c: char, radix: u8) -> Option<u8> {
let c = c as u8;
// assuming radix is either 10 or 16
if c >= '0' as u8 && c <= '9' as u8 {
Some(c - '0' as u8)
} else if radix > 10 && c >= 'a' as u8 && c < 'a' as u8 + (radix - 10) {
Some(c - 'a' as u8 + 10)
} else if radix > 10 && c >= 'A' as u8 && c < 'A' as u8 + (radix - 10) {
Some(c - 'A' as u8 + 10)
if c >= b'0' && c <= b'9' {
Some(c - b'0')
} else if radix > 10 && c >= b'a' && c < b'a' + (radix - 10) {
Some(c - b'a' + 10)
} else if radix > 10 && c >= b'A' && c < b'A' + (radix - 10) {
Some(c - b'A' + 10)
} else {
None
}
Expand Down
2 changes: 1 addition & 1 deletion src/libstd/io/stdio.rs
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,7 @@ pub fn print(s: &str) {
/// `\n` character is printed to the console after the string.
pub fn println(s: &str) {
with_task_stdout(|io| {
io.write(s.as_bytes()).and_then(|()| io.write(['\n' as u8]))
io.write(s.as_bytes()).and_then(|()| io.write([b'\n']))
})
}

Expand Down
2 changes: 1 addition & 1 deletion src/libstd/macros.rs
Original file line number Diff line number Diff line change
Expand Up @@ -502,7 +502,7 @@ pub mod builtin {
///
/// ```
/// let rust = bytes!("r", 'u', "st", 255);
/// assert_eq!(rust[1], 'u' as u8);
/// assert_eq!(rust[1], b'u');
/// assert_eq!(rust[4], 255);
/// ```
#[macro_export]
Expand Down
36 changes: 17 additions & 19 deletions src/libstd/num/strconv.rs
Original file line number Diff line number Diff line change
Expand Up @@ -138,12 +138,10 @@ impl_NumStrConv_Integer!(u64)


// Special value strings as [u8] consts.
static INF_BUF: [u8, ..3] = ['i' as u8, 'n' as u8, 'f' as u8];
static POS_INF_BUF: [u8, ..4] = ['+' as u8, 'i' as u8, 'n' as u8,
'f' as u8];
static NEG_INF_BUF: [u8, ..4] = ['-' as u8, 'i' as u8, 'n' as u8,
'f' as u8];
static NAN_BUF: [u8, ..3] = ['N' as u8, 'a' as u8, 'N' as u8];
static INF_BUF: [u8, ..3] = [b'i', b'n', b'f'];
static POS_INF_BUF: [u8, ..4] = [b'+', b'i', b'n', b'f'];
static NEG_INF_BUF: [u8, ..4] = [b'-', b'i', b'n', b'f'];
static NAN_BUF: [u8, ..3] = [b'N', b'a', b'N'];

/**
* Converts an integral number to its string representation as a byte vector.
Expand Down Expand Up @@ -201,8 +199,8 @@ pub fn int_to_str_bytes_common<T: Int>(num: T, radix: uint, sign: SignFormat, f:
current_digit_signed
};
buf[cur] = match current_digit.to_u8().unwrap() {
i @ 0..9 => '0' as u8 + i,
i => 'a' as u8 + (i - 10),
i @ 0..9 => b'0' + i,
i => b'a' + (i - 10),
};
cur += 1;

Expand All @@ -213,8 +211,8 @@ pub fn int_to_str_bytes_common<T: Int>(num: T, radix: uint, sign: SignFormat, f:

// Decide what sign to put in front
match sign {
SignNeg | SignAll if neg => { f('-' as u8); }
SignAll => { f('+' as u8); }
SignNeg | SignAll if neg => { f(b'-'); }
SignAll => { f(b'+'); }
_ => ()
}

Expand Down Expand Up @@ -350,10 +348,10 @@ pub fn float_to_str_bytes_common<T:NumCast+Zero+One+PartialEq+PartialOrd+Float+
// Decide what sign to put in front
match sign {
SignNeg | SignAll if neg => {
buf.push('-' as u8);
buf.push(b'-');
}
SignAll => {
buf.push('+' as u8);
buf.push(b'+');
}
_ => ()
}
Expand All @@ -368,7 +366,7 @@ pub fn float_to_str_bytes_common<T:NumCast+Zero+One+PartialEq+PartialOrd+Float+
// Now emit the fractional part, if any
deccum = num.fract();
if deccum != _0 || (limit_digits && exact && digit_count > 0) {
buf.push('.' as u8);
buf.push(b'.');
let mut dig = 0u;

// calculate new digits while
Expand Down Expand Up @@ -415,14 +413,14 @@ pub fn float_to_str_bytes_common<T:NumCast+Zero+One+PartialEq+PartialOrd+Float+
// If reached left end of number, have to
// insert additional digit:
if i < 0
|| *buf.get(i as uint) == '-' as u8
|| *buf.get(i as uint) == '+' as u8 {
|| *buf.get(i as uint) == b'-'
|| *buf.get(i as uint) == b'+' {
buf.insert((i + 1) as uint, value2ascii(1));
break;
}

// Skip the '.'
if *buf.get(i as uint) == '.' as u8 { i -= 1; continue; }
if *buf.get(i as uint) == b'.' { i -= 1; continue; }

// Either increment the digit,
// or set to 0 if max and carry the 1.
Expand All @@ -448,14 +446,14 @@ pub fn float_to_str_bytes_common<T:NumCast+Zero+One+PartialEq+PartialOrd+Float+
let mut i = buf_max_i;

// discover trailing zeros of fractional part
while i > start_fractional_digits && *buf.get(i) == '0' as u8 {
while i > start_fractional_digits && *buf.get(i) == b'0' {
i -= 1;
}

// Only attempt to truncate digits if buf has fractional digits
if i >= start_fractional_digits {
// If buf ends with '.', cut that too.
if *buf.get(i) == '.' as u8 { i -= 1 }
if *buf.get(i) == b'.' { i -= 1 }

// only resize buf if we actually remove digits
if i < buf_max_i {
Expand All @@ -465,7 +463,7 @@ pub fn float_to_str_bytes_common<T:NumCast+Zero+One+PartialEq+PartialOrd+Float+
} // If exact and trailing '.', just cut that
else {
let max_i = buf.len() - 1;
if *buf.get(max_i) == '.' as u8 {
if *buf.get(max_i) == b'.' {
buf = Vec::from_slice(buf.slice(0, max_i));
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/libstd/os.rs
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ pub fn env_as_bytes() -> Vec<(Vec<u8>,Vec<u8>)> {
fn env_convert(input: Vec<Vec<u8>>) -> Vec<(Vec<u8>, Vec<u8>)> {
let mut pairs = Vec::new();
for p in input.iter() {
let mut it = p.as_slice().splitn(1, |b| *b == '=' as u8);
let mut it = p.as_slice().splitn(1, |b| *b == b'=');
let key = Vec::from_slice(it.next().unwrap());
let val = Vec::from_slice(it.next().unwrap_or(&[]));
pairs.push((key, val));
Expand Down
6 changes: 3 additions & 3 deletions src/libstd/path/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -351,7 +351,7 @@ pub trait GenericPath: Clone + GenericPathUnsafe {
match self.filename() {
None => None,
Some(name) => Some({
let dot = '.' as u8;
let dot = b'.';
match name.rposition_elem(&dot) {
None | Some(0) => name,
Some(1) if name == b".." => name,
Expand Down Expand Up @@ -398,7 +398,7 @@ pub trait GenericPath: Clone + GenericPathUnsafe {
match self.filename() {
None => None,
Some(name) => {
let dot = '.' as u8;
let dot = b'.';
match name.rposition_elem(&dot) {
None | Some(0) => None,
Some(1) if name == b".." => None,
Expand Down Expand Up @@ -474,7 +474,7 @@ pub trait GenericPath: Clone + GenericPathUnsafe {
assert!(!contains_nul(&extension));

let val = self.filename().and_then(|name| {
let dot = '.' as u8;
let dot = b'.';
let extlen = extension.container_as_bytes().len();
match (name.rposition_elem(&dot), extlen) {
(None, 0) | (Some(0), 0) => None,
Expand Down
2 changes: 1 addition & 1 deletion src/libstd/path/posix.rs
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ impl GenericPath for Path {
match self.sepidx {
None if b"." == self.repr.as_slice() => false,
None => {
self.repr = vec!['.' as u8];
self.repr = vec![b'.'];
self.sepidx = None;
true
}
Expand Down
8 changes: 4 additions & 4 deletions src/libstd/path/windows.rs
Original file line number Diff line number Diff line change
Expand Up @@ -737,12 +737,12 @@ impl Path {
let mut comps = comps;
match (comps.is_some(),prefix) {
(false, Some(DiskPrefix)) => {
if s.as_bytes()[0] >= 'a' as u8 && s.as_bytes()[0] <= 'z' as u8 {
if s.as_bytes()[0] >= b'a' && s.as_bytes()[0] <= b'z' {
comps = Some(vec![]);
}
}
(false, Some(VerbatimDiskPrefix)) => {
if s.as_bytes()[4] >= 'a' as u8 && s.as_bytes()[0] <= 'z' as u8 {
if s.as_bytes()[4] >= b'a' && s.as_bytes()[0] <= b'z' {
comps = Some(vec![]);
}
}
Expand Down Expand Up @@ -1010,7 +1010,7 @@ fn parse_prefix<'a>(mut path: &'a str) -> Option<PathPrefix> {
} else {
// \\?\path
let idx = path.find('\\');
if idx == Some(2) && path.as_bytes()[1] == ':' as u8 {
if idx == Some(2) && path.as_bytes()[1] == b':' {
let c = path.as_bytes()[0];
if c.is_ascii() && (c as char).is_alphabetic() {
// \\?\C:\ path
Expand All @@ -1033,7 +1033,7 @@ fn parse_prefix<'a>(mut path: &'a str) -> Option<PathPrefix> {
}
_ => ()
}
} else if path.len() > 1 && path.as_bytes()[1] == ':' as u8 {
} else if path.len() > 1 && path.as_bytes()[1] == b':' {
// C:
let c = path.as_bytes()[0];
if c.is_ascii() && (c as char).is_alphabetic() {
Expand Down

0 comments on commit 3fb78e2

Please sign in to comment.