Skip to content

Commit b6996e6

Browse files
unhappychoiceclaude
andcommitted
refactor: remove unused LoadingProgress struct
- Remove LoadingProgress struct which was defined but never used - Remove unnecessary Arc/Mutex imports - Keep only actually used progress reporters: ProgressReporter trait, ConsoleProgressReporter, NoOpProgressReporter - Clean up module exports 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent c56ceef commit b6996e6

File tree

2 files changed

+1
-62
lines changed

2 files changed

+1
-62
lines changed

src/extractor/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,5 +11,5 @@ pub use challenge_converter::ChallengeConverter;
1111
pub use chunk::{CodeChunk, ChunkType};
1212
pub use language::Language;
1313
pub use parser::{CodeExtractor, ExtractionOptions};
14-
pub use progress::{ConsoleProgressReporter, LoadingProgress, NoOpProgressReporter, ProgressReporter};
14+
pub use progress::{ConsoleProgressReporter, NoOpProgressReporter, ProgressReporter};
1515
pub use repository_loader::RepositoryLoader;

src/extractor/progress.rs

Lines changed: 0 additions & 61 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
use std::sync::{Arc, Mutex};
21

32
pub trait ProgressReporter {
43
fn set_phase(&self, phase: String);
@@ -11,66 +10,6 @@ pub trait ProgressReporter {
1110
}
1211
}
1312

14-
pub struct LoadingProgress {
15-
phase: Arc<Mutex<String>>,
16-
progress: Arc<Mutex<f64>>,
17-
current_file: Arc<Mutex<Option<String>>>,
18-
files_processed: Arc<Mutex<usize>>,
19-
total_files: Arc<Mutex<usize>>,
20-
}
21-
22-
impl LoadingProgress {
23-
pub fn new() -> Self {
24-
Self {
25-
phase: Arc::new(Mutex::new(String::new())),
26-
progress: Arc::new(Mutex::new(0.0)),
27-
current_file: Arc::new(Mutex::new(None)),
28-
files_processed: Arc::new(Mutex::new(0)),
29-
total_files: Arc::new(Mutex::new(0)),
30-
}
31-
}
32-
33-
pub fn get_phase(&self) -> String {
34-
self.phase.lock().unwrap().clone()
35-
}
36-
37-
pub fn get_progress(&self) -> f64 {
38-
*self.progress.lock().unwrap()
39-
}
40-
41-
pub fn get_current_file(&self) -> Option<String> {
42-
self.current_file.lock().unwrap().clone()
43-
}
44-
45-
pub fn get_file_counts(&self) -> (usize, usize) {
46-
let processed = *self.files_processed.lock().unwrap();
47-
let total = *self.total_files.lock().unwrap();
48-
(processed, total)
49-
}
50-
}
51-
52-
impl ProgressReporter for LoadingProgress {
53-
fn set_phase(&self, phase: String) {
54-
*self.phase.lock().unwrap() = phase;
55-
}
56-
57-
fn set_progress(&self, progress: f64) {
58-
*self.progress.lock().unwrap() = progress;
59-
}
60-
61-
fn set_current_file(&self, file: Option<String>) {
62-
*self.current_file.lock().unwrap() = file;
63-
}
64-
65-
fn set_file_counts(&self, processed: usize, total: usize) {
66-
*self.files_processed.lock().unwrap() = processed;
67-
*self.total_files.lock().unwrap() = total;
68-
}
69-
70-
fn update_spinner(&self) {
71-
// No-op for this implementation
72-
}
73-
}
7413

7514
pub struct NoOpProgressReporter;
7615

0 commit comments

Comments
 (0)