mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use from instead of into in unescaping code.

Browse Source 
The `T` type in these functions took me some time to understand, and I
find the explicit `T` in the use of `from` makes the code easier to
read, as does the `u8` annotation in `scan_escape`.
This commit is contained in:
Nicholas Nethercote 2024-01-23 10:37:27 +11:00
parent 4b4bdb575b
commit ef1e2228cf
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
compiler/rustc_lexer/src/unescape.rs
View File

@ -222,7 +222,7 @@ fn scan_escape<T: From<u8> + From<char>>(
mode: Mode,
) -> Result<T, EscapeError> {
// Previous character was '\\', unescape what follows.
let res = match chars.next().ok_or(EscapeError::LoneSlash)? {
let res: u8 = match chars.next().ok_or(EscapeError::LoneSlash)? {
'"' => b'"',
'n' => b'\n',
'r' => b'\r',
@ -249,10 +249,10 @@ fn scan_escape<T: From<u8> + From<char>>(
value as u8
}
'u' => return scan_unicode(chars, mode.is_unicode_escape_disallowed()).map(Into::into),
'u' => return scan_unicode(chars, mode.is_unicode_escape_disallowed()).map(T::from),
_ => return Err(EscapeError::InvalidEscape),
};
Ok(res.into())
Ok(T::from(res))
}
fn scan_unicode(
@ -366,7 +366,7 @@ fn unescape_non_raw_common<F, T: From<u8> + From<char>>(src: &str, mode: Mode, c
}
'"' => Err(EscapeError::EscapeOnlyChar),
'\r' => Err(EscapeError::BareCarriageReturn),
_ => ascii_check(c, chars_should_be_ascii).map(Into::into),
_ => ascii_check(c, chars_should_be_ascii).map(T::from),
};
let end = src.len() - chars.as_str().len();
callback(start..end, res);