opendev/gerrit
Code Issues Proposed changes

Merge "Fix Eclipse warnings introduced by I51f3ac5a"

Browse Source
This commit is contained in:
Dave Borowitz
2017-04-04 18:29:52 +00:00
committed by Gerrit Code Review
parent 1de8791ba5 4d03dd2021
commit 01db2e524f
3 changed files with 21 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
gerrit-gwtui/src/main/java/com/google/gerrit/client/api/ExtensionPanel.java
View File

@@ -38,14 +38,13 @@ public class ExtensionPanel extends FlowPanel {
this(extensionPoint, new ArrayList<String>());
}
public ExtensionPanel(GerritUiExtensionPoint extensionPoint,
List<String> panelNames) {
public ExtensionPanel(GerritUiExtensionPoint extensionPoint, List<String> panelNames) {
this.extensionPoint = extensionPoint;
this.contexts = create(panelNames);
}
private List<Context> create(List<String> panelNames) {
List<Context> contexts = new ArrayList<Context>();
List<Context> contexts = new ArrayList<>();
for (Definition def : getOrderedDefs(panelNames)) {
SimplePanel p = new SimplePanel();
add(p);
@@ -58,31 +57,28 @@ public class ExtensionPanel extends FlowPanel {
if (panelNames == null) {
panelNames = Collections.emptyList();
}
Map<String, List<Definition>> defsOrderedByName =
new LinkedHashMap<String, List<Definition>>();
Map<String, List<Definition>> defsOrderedByName = new LinkedHashMap<>();
for (String name : panelNames) {
defsOrderedByName.put(name, new ArrayList<Definition>());
}
for (Definition def : Natives.asList(
Definition.get(extensionPoint.name()))) {
for (Definition def : Natives.asList(Definition.get(extensionPoint.name()))) {
addDef(def, defsOrderedByName);
}
List<Definition> orderedDefs = new ArrayList<Definition>();
List<Definition> orderedDefs = new ArrayList<>();
for (List<Definition> defList : defsOrderedByName.values()) {
orderedDefs.addAll(defList);
}
return orderedDefs;
}
private static void addDef(Definition def,
Map<String, List<Definition>> defsOrderedByName) {
private static void addDef(Definition def, Map<String, List<Definition>> defsOrderedByName) {
String panelName = def.getPanelName();
if (panelName.equals(def.getPluginName() + ".undefined")) {
/* Handle a partially undefined panel name from the
javascript layer by generating a random panel name.
This maintains support for panels that do not provide a name. */
panelName = def.getPluginName() + "." + Long.toHexString(
Double.doubleToLongBits(Math.random()));
/* Handle a partially undefined panel name from the
javascript layer by generating a random panel name.
This maintains support for panels that do not provide a name. */
panelName =
def.getPluginName() + "." + Long.toHexString(Double.doubleToLongBits(Math.random()));
}
if (defsOrderedByName.containsKey(panelName)) {
defsOrderedByName.get(panelName).add(def);
@@ -163,6 +159,7 @@ public class ExtensionPanel extends FlowPanel {
protected Definition() {}
public final native String getPanelName() /*-{ return this.pluginName + "." + this.name; }-*/;
public final native String getPluginName() /*-{ return this.pluginName; }-*/;
}

2
gerrit-gwtui/src/main/java/com/google/gerrit/client/projects/ConfigInfoCache.java
View File

@@ -22,8 +22,6 @@ import com.google.gerrit.reviewdb.client.Change;
import com.google.gerrit.reviewdb.client.Project;
import com.google.gwt.core.client.GWT;
import com.google.gwt.user.client.rpc.AsyncCallback;
import java.util.Collections;
import java.util.LinkedHashMap;
import java.util.List;
import java.util.Map;