refactor: clarify naming

PLC-lang · Jan 31, 2025 · 3386fe4 · 3386fe4
1 parent 50a25f7
commit 3386fe4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/compiler/plc_driver/src/pipelines.rs b/compiler/plc_driver/src/pipelines.rs
@@ -795,8 +795,9 @@ impl AnnotatedProject {
     }
 
     // TODO: Find better way, this is hella expensive
-    pub fn redo(self, mut id_provider: IdProvider) -> AnnotatedProject {
-        //Create and call the annotator
+    pub fn re_annotate(self, mut id_provider: IdProvider) -> AnnotatedProject {
+        // TODO: this almost duplicates IndexedProject::annotate(). Can we do this differently?
+        // Create and call the annotator again after lowering property references to calls
         let mut annotated_units = Vec::new();
         let mut all_annotations = AnnotationMapImpl::default();
         let result = self

diff --git a/compiler/plc_driver/src/pipelines/property.rs b/compiler/plc_driver/src/pipelines/property.rs
@@ -24,6 +24,6 @@ impl PipelineParticipantMut for PropertyLowerer {
         }
 
         let project = AnnotatedProject { units, index, annotations: self.annotations.take().unwrap() };
-        project.redo(self.id_provider.clone())
+        project.re_annotate(self.id_provider.clone())
     }
 }