andras/reconcile
andras/reconcile
1
0
Fork 0
Code Issues Pull requests 6 Projects Releases Packages Wiki Activity Actions

Mini perf optimisation

Browse source
This commit is contained in:
Andras Schmelczer 2026-03-10 20:40:14 +00:00
commit 408ce5268f
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/operation_transformation/edited_text.rs
View file

@ -435,6 +435,7 @@ where
) -> Result<EditedText<'a, T>, DiffError> { ) -> Result<EditedText<'a, T>, DiffError> {
let mut operations: Vec<Operation<T>> = Vec::with_capacity(diff.len()); let mut operations: Vec<Operation<T>> = Vec::with_capacity(diff.len());
let mut order = 0; let mut order = 0;
let text_length = original_text.chars().count();
for item in diff { for item in diff {
match item { match item {
@ -443,7 +444,6 @@ where
let length = usize::try_from(length).expect("length must fit in usize"); let length = usize::try_from(length).expect("length must fit in usize");
// Validate that the range doesn't exceed the original text // Validate that the range doesn't exceed the original text
let text_length = original_text.chars().count();
if order + length > text_length { if order + length > text_length {
return Err(DiffError::LengthExceedsOriginal { return Err(DiffError::LengthExceedsOriginal {
position: order, position: order,
@ -466,7 +466,6 @@ where
usize::try_from(-length).expect("negative length must fit in usize"); usize::try_from(-length).expect("negative length must fit in usize");
// Validate that the delete range doesn't exceed the original text // Validate that the delete range doesn't exceed the original text
let text_length = original_text.chars().count();
if order + length > text_length { if order + length > text_length {
return Err(DiffError::LengthExceedsOriginal { return Err(DiffError::LengthExceedsOriginal {
position: order, position: order,