initializing vectors with capacity

Author: perryqh

Cargo.lock

@@ -15,6 +15,7 @@ ignore = "0.4.23"
 itertools = "0.13.0"
 path-clean = "1.0.1"
 lazy_static = "1.5.0"
+num_cpus = "1.16.0"
 rayon = "1.10.0"
 regex = "1.11.1"
 serde = { version = "1.0.214", features = ["derive"] }

Cargo.toml

@@ -35,6 +35,8 @@ pub struct ProjectBuilder<'a> {
     pub codeowners_file_path: PathBuf,
 }
 
+const INITIAL_VECTOR_CAPACITY: usize = 1000;
+
 impl<'a> ProjectBuilder<'a> {
     pub fn new(config: &'a Config, base_path: PathBuf, codeowners_file_path: PathBuf) -> Self {
         Self {
@@ -46,8 +48,7 @@ impl<'a> ProjectBuilder<'a> {
 
     #[instrument(level = "debug", skip_all)]
     pub fn build(&mut self) -> Result<Project, Error> {
-        let mut entry_types = Vec::new();
-
+        let mut entry_types = Vec::with_capacity(INITIAL_VECTOR_CAPACITY);
         let mut builder = WalkBuilder::new(&self.base_path);
         builder.hidden(false);
         let walkdir = builder.build();
@@ -56,7 +57,9 @@ impl<'a> ProjectBuilder<'a> {
             let entry = entry.change_context(Error::Io)?;
             entry_types.push(self.build_entry_type(entry)?);
         }
-        self.build_project_from_entry_types(entry_types)
+        let project = self.build_project_from_entry_types(entry_types)?;
+
+        Ok(project)
     }
 
     fn build_entry_type(&mut self, entry: ignore::DirEntry) -> Result<EntryType, Error> {
@@ -101,7 +104,7 @@ impl<'a> ProjectBuilder<'a> {
             .fold(
                 || {
                     (
-                        Vec::<ProjectFile>::new(),
+                        Vec::<ProjectFile>::with_capacity(INITIAL_VECTOR_CAPACITY),
                         Vec::<Package>::new(),
                         Vec::<VendoredGem>::new(),
                         Vec::<DirectoryCodeownersFile>::new(),
@@ -223,7 +226,6 @@ fn build_project_file(path: PathBuf) -> ProjectFile {
     };
 
     let first_line = content.lines().next();
-
     let Some(first_line) = first_line else {
         return ProjectFile { path, owner: None };
     };