opendev/gerrit
Code Issues Proposed changes

Fix equals method of DynamicMap.NamePair

Browse Source 
Equals was wrongly implemented so that the lookup of functions from
the map failed.

Change-Id: Id996f4eb52e33defb93579eeecb7ef3825e95b3f
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
This commit is contained in:
Edwin Kempin
2012-05-11 08:09:08 +02:00
parent 5de34d9713
commit 5289ca399a
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-extension-api/src/main/java/com/google/gerrit/extensions/registration/DynamicMap.java
View File

@@ -147,7 +147,7 @@ public abstract class DynamicMap<T> {
public boolean equals(Object other) { public boolean equals(Object other) {
if (other instanceof NamePair) { if (other instanceof NamePair) {
NamePair np = (NamePair) other; NamePair np = (NamePair) other;
return pluginName.equals(np) && exportName.equals(np); return pluginName.equals(np.pluginName) && exportName.equals(np.exportName);
} }
return false; return false;
} }