traxys/tree-sitter
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(rust): remove redundant String clones

Browse source
This commit is contained in:
Will Lillis 2025-08-08 22:49:58 -04:00
parent 9f2dd2f377
commit 3d9f4d1bd3
2 changed files with 6 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

3
crates/cli/src/tests/query_test.rs
View file

@ -821,8 +821,7 @@ fn test_query_matches_with_many_overlapping_results() {
// .foo(bar(BAZ))
// .foo(bar(BAZ))
// ...
let mut source = "a".to_string();
source += &"\n .foo(bar(BAZ))".repeat(count);
let source = format!("a{}", "\n .foo(bar(BAZ))".repeat(count));
assert_query_matches(
&language,

10
crates/generate/src/build_tables/build_parse_table.rs
View file

@ -855,9 +855,9 @@ impl<'a> ParseTableBuilder<'a> {
for symbol in preceding_symbols {
conflict_error
.symbol_sequence
.push(self.symbol_name(symbol).to_string());
.push(self.symbol_name(symbol));
}
conflict_error.conflicting_lookahead = self.symbol_name(&conflicting_lookahead).to_string();
conflict_error.conflicting_lookahead = self.symbol_name(&conflicting_lookahead);
let interpretations = conflicting_items
.iter()
@ -865,7 +865,7 @@ impl<'a> ParseTableBuilder<'a> {
let preceding_symbols = preceding_symbols
.iter()
.take(preceding_symbols.len() - item.step_index as usize)
.map(|symbol| self.symbol_name(symbol).to_string())
.map(|symbol| self.symbol_name(symbol))
.collect::<Vec<_>>();
let variable_name = self.syntax_grammar.variables[item.variable_index as usize]
@ -876,7 +876,7 @@ impl<'a> ParseTableBuilder<'a> {
.production
.steps
.iter()
.map(|step| self.symbol_name(&step.symbol).to_string())
.map(|step| self.symbol_name(&step.symbol))
.collect::<Vec<_>>();
let precedence = match item.precedence() {
@ -892,7 +892,7 @@ impl<'a> ParseTableBuilder<'a> {
production_step_symbols,
step_index: item.step_index,
done: item.is_done(),
conflicting_lookahead: self.symbol_name(&conflicting_lookahead).to_string(),
conflicting_lookahead: self.symbol_name(&conflicting_lookahead),
precedence,
associativity,
}