bug: Diary was getting time of first collection set properly from the…

… diarizer
microsoft · Apr 21, 2022 · 6cea993 · 6cea993
1 parent c478a62
commit 6cea993
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/api/src/main/java/com/microsoft/gctoolkit/jvm/Diary.java b/api/src/main/java/com/microsoft/gctoolkit/jvm/Diary.java
@@ -373,6 +373,7 @@ public void setTimeOfFirstEvent(DateTimeStamp startTime) {
         if ( this.timeOfFirstEvent == null)
             this.timeOfFirstEvent = startTime;
     }
+
     public DateTimeStamp getTimeOfFirstEvent() {
         return this.timeOfFirstEvent;
     }

diff --git a/parser/src/main/java/com/microsoft/gctoolkit/parser/jvm/PreUnifiedDiarizer.java b/parser/src/main/java/com/microsoft/gctoolkit/parser/jvm/PreUnifiedDiarizer.java
@@ -240,8 +240,10 @@ private void timeOfFirstEvent(String line) {
                 return;      // G1Ergonomics doesn't respect PrintDateStamp which confuses downstream calculations
             }
             GCLogTrace trace = PreUnifiedTokens.DATE_TIMESTAMP_RECORD.parse(line);
-            if (trace != null)
+            if (trace != null) {
                 timeOfFirstEvent = trace.getDateTimeStamp();
+                diary.setTimeOfFirstEvent(timeOfFirstEvent);
+            }
         }
     }
 

diff --git a/pom.xml b/pom.xml
@@ -28,7 +28,7 @@
         <connection>scm:git:${repository.url}</connection>
         <developerConnection>scm:git:${repository.url}</developerConnection>
         <url>${repository.url}</url>
-      <tag>gctoolkit-2.0.5</tag>
+      <tag>gctoolkit-2.0.10</tag>
     </scm>
 
     <developers>