Addressed a code review comment.

facebookincubator · Feb 12, 2024 · 3f7fb3d · 3f7fb3d
1 parent bb4ede4
commit 3f7fb3d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/main/java/com/meta/cp4m/message/Message.java b/src/main/java/com/meta/cp4m/message/Message.java
@@ -9,8 +9,6 @@
 package com.meta.cp4m.message;
 
 import com.meta.cp4m.Identifier;
-import org.checkerframework.checker.lock.qual.NewObject;
-import org.checkerframework.checker.nullness.qual.Nullable;
 
 import java.time.Instant;
 
@@ -49,7 +47,7 @@ enum Role {
       this.priority = priority;
     }
 
-    public int getPriority(){
+    public int priority(){
       return this.priority;
     }
 

diff --git a/src/main/java/com/meta/cp4m/message/ThreadState.java b/src/main/java/com/meta/cp4m/message/ThreadState.java
@@ -43,7 +43,7 @@ private ThreadState(ThreadState<T> current, ThreadState<T> old, T newMessage) {
     MessageNode<T> mWithParentMessage = new MessageNode<>(newMessage,old.tail());
     this.messageNodes =
             Stream.concat(messageNodes.stream(), Stream.of(mWithParentMessage))
-                    .sorted((m1,m2) -> m1.getParentMessage() == m2.getParentMessage() ? compare(m1.getMessage().role().getPriority(),m2.getMessage().role().getPriority()) : (m1.getMessage().timestamp().compareTo(m2.getMessage().timestamp())))
+                    .sorted((m1,m2) -> m1.getParentMessage() == m2.getParentMessage() ? compare(m1.getMessage().role().priority(),m2.getMessage().role().priority()) : (m1.getMessage().timestamp().compareTo(m2.getMessage().timestamp())))
                     .collect(Collectors.toUnmodifiableList());
 
     Preconditions.checkArgument(