From 14dbb245bd1ec8bf3765a27f0b9b508152c39457 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Mon, 7 Sep 2015 17:27:44 +0900 Subject: [PATCH 1/2] VersionedAccountDestinations: Access createLoggerSink statically Change-Id: I5bec2ed4b60bfbbadbff16c02a318c2ed736ce3d --- .../gerrit/server/account/VersionedAccountDestinations.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/account/VersionedAccountDestinations.java b/gerrit-server/src/main/java/com/google/gerrit/server/account/VersionedAccountDestinations.java index d928becb7b..5e65acd5dc 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/account/VersionedAccountDestinations.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/account/VersionedAccountDestinations.java @@ -17,6 +17,7 @@ package com.google.gerrit.server.account; import com.google.gerrit.reviewdb.client.Account; import com.google.gerrit.reviewdb.client.RefNames; import com.google.gerrit.server.git.DestinationList; +import com.google.gerrit.server.git.TabFile; import com.google.gerrit.server.git.ValidationError; import com.google.gerrit.server.git.VersionedMetaData; @@ -61,7 +62,7 @@ public class VersionedAccountDestinations extends VersionedMetaData { String path = p.path; if (path.startsWith(prefix)) { String label = path.substring(prefix.length()); - ValidationError.Sink errors = destinations.createLoggerSink(path, log); + ValidationError.Sink errors = TabFile.createLoggerSink(path, log); destinations.parseLabel(label, readUTF8(path), errors); } } From d3b795a4cb328869ea86bcb8b4b7dcd6bea62747 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Mon, 7 Sep 2015 17:28:19 +0900 Subject: [PATCH 2/2] TabFile: Fix Eclipse warnings and code style - Redundant specification of type arguments - Missing @Override annotation - Incorrect indentation Change-Id: Ifafc51bd0ef7f8fa8a3e7ad9cd6e42d70a913cbc --- .../java/com/google/gerrit/server/git/TabFile.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/git/TabFile.java b/gerrit-server/src/main/java/com/google/gerrit/server/git/TabFile.java index 87f9a23879..3cbac3ba26 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/git/TabFile.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/git/TabFile.java @@ -32,11 +32,11 @@ public class TabFile { } public static Parser TRIM = new Parser() { - public String parse(String str) { - return str.trim(); - } - }; - + @Override + public String parse(String str) { + return str.trim(); + } + }; protected static class Row { public String left; @@ -50,7 +50,7 @@ public class TabFile { protected static List parse(String text, String filename, Parser left, Parser right, ValidationError.Sink errors) throws IOException { - List rows = new ArrayList(); + List rows = new ArrayList<>(); BufferedReader br = new BufferedReader(new StringReader(text)); String s; for (int lineNumber = 1; (s = br.readLine()) != null; lineNumber++) {