opendev/gerrit
Code Issues Proposed changes

CommentSender#setComments: Remove unused computation of paths

Browse Source 
Change Ie57b89cea removed the usage of the paths, now we do not need to
compute them anymore.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: Ib868e7fc29bac3b51531cb675c2b5b4c208f2914
This commit is contained in:
Edwin Kempin
2019-07-22 11:38:57 +02:00
parent 784eb83def
commit 911b4a2722
1 changed files with 0 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
java/com/google/gerrit/server/mail/send/CommentSender.java
View File

@@ -54,7 +54,6 @@ import java.util.HashSet;
import java.util.List; import java.util.List;
import java.util.Map; import java.util.Map;
import java.util.Optional; import java.util.Optional;
import java.util.Set;
import org.apache.james.mime4j.dom.field.FieldName; import org.apache.james.mime4j.dom.field.FieldName;
import org.eclipse.jgit.lib.Config; import org.eclipse.jgit.lib.Config;
import org.eclipse.jgit.lib.Repository; import org.eclipse.jgit.lib.Repository;
@@ -127,13 +126,6 @@ public class CommentSender extends ReplyToChangeSender {
public void setComments(List<Comment> comments) { public void setComments(List<Comment> comments) {
inlineComments = comments; inlineComments = comments;
Set<String> paths = new HashSet<>();
for (Comment c : comments) {
if (!Patch.isMagic(c.key.filename)) {
paths.add(c.key.filename);
}
}
} }
public void setPatchSetComment(String comment) { public void setPatchSetComment(String comment) {