Skip to content

Commit 13f5628

Browse files
committed
Inline format-args
This makes the code a bit easier to read and smaller. Some of it was done with this command, and later fixed by hand: ``` cargo clippy --workspace --allow-dirty --fix --benches --tests --bins -- -A clippy::all -W clippy::uninlined_format_args ```
1 parent e7bd19d commit 13f5628

File tree

17 files changed

+47
-55
lines changed

17 files changed

+47
-55
lines changed

regex-capi/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ description = """
1111
A C API for Rust's regular expression library.
1212
"""
1313
workspace = ".."
14-
edition = "2018"
14+
edition = "2021"
1515

1616
[lib]
1717
name = "rure"

regex-capi/src/error.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ ffi_fn! {
6161
ffi_fn! {
6262
fn rure_error_message(err: *mut Error) -> *const c_char {
6363
let err = unsafe { &mut *err };
64-
let cmsg = match CString::new(format!("{}", err)) {
64+
let cmsg = match CString::new(format!("{err}")) {
6565
Ok(msg) => msg,
6666
Err(err) => {
6767
// I guess this can probably happen if the regex itself has a

regex-capi/src/macros.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,7 @@ macro_rules! ffi_fn {
1818
} else {
1919
"UNABLE TO SHOW RESULT OF PANIC.".to_owned()
2020
};
21-
let _ = writeln!(
22-
&mut io::stderr(),
23-
"panic unwind caught, aborting: {:?}",
24-
msg);
21+
let _ = writeln!(&mut io::stderr(), "panic unwind caught, aborting: {msg:?}");
2522
unsafe { abort() }
2623
}
2724
}

regex-capi/src/rure.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ ffi_fn! {
8282
let re = rure_compile(
8383
pat, len, RURE_DEFAULT_FLAGS, ptr::null(), &mut err);
8484
if err.is_err() {
85-
let _ = writeln!(&mut io::stderr(), "{}", err);
85+
let _ = writeln!(&mut io::stderr(), "{err}");
8686
let _ = writeln!(
8787
&mut io::stderr(), "aborting from rure_compile_must");
8888
unsafe { abort() }
@@ -579,7 +579,7 @@ ffi_fn! {
579579
let mut err = Error::new(ErrorKind::None);
580580
let esc = rure_escape(pat, len, &mut err);
581581
if err.is_err() {
582-
let _ = writeln!(&mut io::stderr(), "{}", err);
582+
let _ = writeln!(&mut io::stderr(), "{err}");
583583
let _ = writeln!(
584584
&mut io::stderr(), "aborting from rure_escape_must");
585585
unsafe { abort() }

regex-lite/src/hir/parse.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -593,8 +593,7 @@ impl<'a> Parser<'a> {
593593
'u' => 4,
594594
'U' => 8,
595595
unk => unreachable!(
596-
"invalid start of fixed length hexadecimal number {}",
597-
unk
596+
"invalid start of fixed length hexadecimal number {unk}"
598597
),
599598
};
600599
if !self.bump_and_bump_space() {
@@ -720,7 +719,7 @@ impl<'a> Parser<'a> {
720719
'?' => (0, Some(1)),
721720
'*' => (0, None),
722721
'+' => (1, None),
723-
unk => unreachable!("unrecognized repetition operator '{}'", unk),
722+
unk => unreachable!("unrecognized repetition operator '{unk}'"),
724723
};
725724
let mut greedy = true;
726725
if self.bump() && self.char() == '?' {
@@ -1216,7 +1215,7 @@ impl<'a> Parser<'a> {
12161215
'd' | 'D' => posix_class("digit").unwrap(),
12171216
's' | 'S' => posix_class("space").unwrap(),
12181217
'w' | 'W' => posix_class("word").unwrap(),
1219-
unk => unreachable!("invalid Perl class \\{}", unk),
1218+
unk => unreachable!("invalid Perl class \\{unk}"),
12201219
});
12211220
if ch.is_ascii_uppercase() {
12221221
class.negate();

regex-lite/src/nfa.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ impl core::fmt::Debug for NFA {
136136
writeln!(f, "NFA(")?;
137137
writeln!(f, "pattern: {}", self.pattern)?;
138138
for (sid, state) in self.states.iter().enumerate() {
139-
writeln!(f, "{:07?}: {:?}", sid, state)?;
139+
writeln!(f, "{sid:07?}: {state:?}")?;
140140
}
141141
writeln!(f, ")")?;
142142
Ok(())
@@ -206,14 +206,14 @@ impl core::fmt::Debug for State {
206206
fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result {
207207
match *self {
208208
State::Char { target, ch } => {
209-
write!(f, "{:?} => {:?}", ch, target)
209+
write!(f, "{ch:?} => {target:?}")
210210
}
211211
State::Ranges { target, ref ranges } => {
212212
for (i, &(start, end)) in ranges.iter().enumerate() {
213213
if i > 0 {
214214
write!(f, ", ")?;
215215
}
216-
write!(f, "{:?}-{:?} => {:?}", start, end, target)?;
216+
write!(f, "{start:?}-{end:?} => {target:?}")?;
217217
}
218218
Ok(())
219219
}
@@ -225,18 +225,18 @@ impl core::fmt::Debug for State {
225225
if i > 0 {
226226
write!(f, ", ")?;
227227
}
228-
write!(f, "{:?}", sid)?;
228+
write!(f, "{sid:?}")?;
229229
}
230230
write!(f, ")")
231231
}
232232
State::Goto { target, look: None } => {
233-
write!(f, "goto({:?})", target)
233+
write!(f, "goto({target:?})")
234234
}
235235
State::Goto { target, look: Some(look) } => {
236-
write!(f, "{:?} => {:?}", look, target)
236+
write!(f, "{look:?} => {target:?}")
237237
}
238238
State::Capture { target, slot } => {
239-
write!(f, "capture(slot={:?}) => {:?}", slot, target,)
239+
write!(f, "capture(slot={slot:?}) => {target:?}")
240240
}
241241
State::Fail => write!(f, "FAIL"),
242242
State::Match => {

regex-lite/src/string.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1798,7 +1798,7 @@ impl<'h> Captures<'h> {
17981798
.nfa()
17991799
.static_explicit_captures_len()
18001800
.expect("number of capture groups can vary in a match");
1801-
assert_eq!(N, len, "asked for {} groups, but must ask for {}", N, len);
1801+
assert_eq!(N, len, "asked for {N} groups, but must ask for {len}");
18021802
let mut matched = self.iter().flatten();
18031803
let whole_match = matched.next().expect("a match").as_str();
18041804
let group_matches = [0; N].map(|_| {
@@ -1965,7 +1965,7 @@ impl<'h> core::fmt::Debug for Captures<'h> {
19651965
fn fmt(&self, f: &mut core::fmt::Formatter) -> core::fmt::Result {
19661966
write!(f, "{}", self.0)?;
19671967
if let Some(name) = self.1 {
1968-
write!(f, "/{:?}", name)?;
1968+
write!(f, "/{name:?}")?;
19691969
}
19701970
Ok(())
19711971
}
@@ -2013,7 +2013,7 @@ impl<'h> core::ops::Index<usize> for Captures<'h> {
20132013
fn index(&self, i: usize) -> &str {
20142014
self.get(i)
20152015
.map(|m| m.as_str())
2016-
.unwrap_or_else(|| panic!("no group at index '{}'", i))
2016+
.unwrap_or_else(|| panic!("no group at index '{i}'"))
20172017
}
20182018
}
20192019

@@ -2039,7 +2039,7 @@ impl<'h, 'n> core::ops::Index<&'n str> for Captures<'h> {
20392039
fn index<'a>(&'a self, name: &'n str) -> &'a str {
20402040
self.name(name)
20412041
.map(|m| m.as_str())
2042-
.unwrap_or_else(|| panic!("no group named '{}'", name))
2042+
.unwrap_or_else(|| panic!("no group named '{name}'"))
20432043
}
20442044
}
20452045

regex-lite/tests/string.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,7 @@ fn run_test(re: &Regex, test: &RegexTest) -> TestResult {
2323
Ok(hay) => hay,
2424
Err(err) => {
2525
return TestResult::fail(&format!(
26-
"haystack is not valid UTF-8: {}",
27-
err
26+
"haystack is not valid UTF-8: {err}",
2827
));
2928
}
3029
};
@@ -45,7 +44,7 @@ fn run_test(re: &Regex, test: &RegexTest) -> TestResult {
4544
.map(|caps| testify_captures(&caps));
4645
TestResult::captures(it)
4746
}
48-
name => TestResult::fail(&format!("unrecognized test name: {}", name)),
47+
name => TestResult::fail(&format!("unrecognized test name: {name}")),
4948
}
5049
}
5150

regex-test/lib.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -153,17 +153,17 @@ impl RegexTests {
153153
/// The given group name is assigned to all loaded tests.
154154
pub fn load_slice(&mut self, group_name: &str, data: &[u8]) -> Result<()> {
155155
let data = std::str::from_utf8(&data).with_context(|| {
156-
format!("data in {} is not valid UTF-8", group_name)
156+
format!("data in {group_name} is not valid UTF-8")
157157
})?;
158158
let mut index = 1;
159159
let mut tests: RegexTests =
160160
toml::from_str(&data).with_context(|| {
161-
format!("error decoding TOML for '{}'", group_name)
161+
format!("error decoding TOML for '{group_name}'")
162162
})?;
163163
for t in &mut tests.tests {
164164
t.group = group_name.to_string();
165165
if t.name.is_empty() {
166-
t.name = format!("{}", index);
166+
t.name = format!("{index}");
167167
index += 1;
168168
}
169169
t.full_name = format!("{}/{}", t.group, t.name);
@@ -1103,7 +1103,7 @@ impl RegexTestFailureKind {
11031103
let mut buf = String::new();
11041104
match *self {
11051105
RegexTestFailureKind::UserFailure { ref why } => {
1106-
write!(buf, "failed by implementor because: {}", why)?;
1106+
write!(buf, "failed by implementor because: {why}")?;
11071107
}
11081108
RegexTestFailureKind::IsMatch => {
11091109
if test.is_match() {
@@ -1142,13 +1142,13 @@ impl RegexTestFailureKind {
11421142
write!(buf, "expected regex to NOT compile, but it did")?;
11431143
}
11441144
RegexTestFailureKind::CompileError { ref err } => {
1145-
write!(buf, "expected regex to compile, failed: {}", err)?;
1145+
write!(buf, "expected regex to compile, failed: {err}")?;
11461146
}
11471147
RegexTestFailureKind::UnexpectedPanicCompile(ref msg) => {
1148-
write!(buf, "got unexpected panic while compiling:\n{}", msg)?;
1148+
write!(buf, "got unexpected panic while compiling:\n{msg}")?;
11491149
}
11501150
RegexTestFailureKind::UnexpectedPanicSearch(ref msg) => {
1151-
write!(buf, "got unexpected panic while searching:\n{}", msg)?;
1151+
write!(buf, "got unexpected panic while searching:\n{msg}")?;
11521152
}
11531153
}
11541154
Ok(buf)

src/error.rs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,7 @@ impl core::fmt::Display for Error {
7171
Error::Syntax(ref err) => err.fmt(f),
7272
Error::CompiledTooBig(limit) => write!(
7373
f,
74-
"Compiled regex exceeds size limit of {} bytes.",
75-
limit
74+
"Compiled regex exceeds size limit of {limit} bytes.",
7675
),
7776
}
7877
}
@@ -88,9 +87,9 @@ impl core::fmt::Debug for Error {
8887
Error::Syntax(ref err) => {
8988
let hr: String = core::iter::repeat('~').take(79).collect();
9089
writeln!(f, "Syntax(")?;
91-
writeln!(f, "{}", hr)?;
92-
writeln!(f, "{}", err)?;
93-
writeln!(f, "{}", hr)?;
90+
writeln!(f, "{hr}")?;
91+
writeln!(f, "{err}")?;
92+
writeln!(f, "{hr}")?;
9493
write!(f, ")")?;
9594
Ok(())
9695
}

0 commit comments

Comments
 (0)