Rooba
/
lapce
mirror of https://github.com/lapce/lapce.git
1
1
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Only allocate one new string.

This commit is contained in:
Philip Daniels 2022-05-26 22:18:54 +01:00
parent 479e1a532c
commit abfada5641
1 changed files with 13 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
lapce-data/src/data.rs
View File

@ -2248,25 +2248,22 @@ pub fn jump_to_location(
}
fn get_name_for_new_file(&self) -> String {
let mut i = 0;
loop {
i += 1;
let potential_name = format!("Untitled-{}", i);
// Checking just the current scratch_docs rather than all the different document
// collections seems to be the right thing to do. The user may have genuine 'new N'
// files tucked away somewhere in their workspace.
if self.scratch_docs.values().any(|doc| match doc.content() {
const PREFIX: &str = "Untitled-";
let new_num = self.scratch_docs
.values()
.filter_map(|doc| match doc.content() {
BufferContent::Scratch(_, existing_name) => {
*existing_name == potential_name
// The unwraps are safe because scratch docs are always
// titled the same format and the user cannot change the name.
let num_part = existing_name.strip_prefix(PREFIX).unwrap();
let num = num_part.parse::<i32>().unwrap();
Some(num)
}
_ => false,
}) {
continue;
}
_ => None,
})
.max().unwrap_or(0) + 1;
return potential_name;
}
return format!("{}{}", PREFIX, new_num);
}
pub fn new_file(&mut self, ctx: &mut EventCtx, config: &Config) {