Changeset 11879 in josm for trunk/src/org/openstreetmap/josm/gui/conflict

Timestamp:

2017-04-10T03:28:15+02:00 (7 years ago)

Author:

Don-vip

Message:

findbugs - EI_EXPOSE_REP

File:

• trunk/src/org/openstreetmap/josm/gui/conflict/pair/ComparePairType.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/conflict/pair/ComparePairType.java

@@ -10 +10 @@
 /**
  * Enumeration of the possible comparison pairs
- *
+ * @since 1650
  */
 public enum ComparePairType {
@@ -48 +48 @@
 
     /**
-     * replies true, if <code>role</code> is participating in this comparison
-     * pair
+     * replies true, if <code>role</code> is participating in this comparison pair
      *
      * @param role  the list role
-     * @return true, if <code>role</code> is participating in this comparison
-     * pair; false, otherwise
+     * @return true, if <code>role</code> is participating in this comparison pair; false, otherwise
      */
     public boolean isParticipatingIn(ListRole role) {
@@ -63 +61 @@
 
     /**
-     * replies the pair of {@link ListRole}s participating in this comparison
-     * pair
+     * replies the pair of {@link ListRole}s participating in this comparison pair
      *
      * @return  the pair of list roles
      */
     public ListRole[] getParticipatingRoles() {
-        return participatingRoles;
+        return Utils.copyArray(participatingRoles);
     }
 
     /**
-     * replies the opposite role of <code>role</code> participating in this comparison
-     * pair
+     * replies the opposite role of <code>role</code> participating in this comparison pair
      *
      * @param role one of the two roles in this pair