Skip to content

Commit 154da4b

Browse files
committed
Use ConfigContainer.copyOf instead of creating a new method.
1 parent 567ac9e commit 154da4b

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

firebase-config/src/main/java/com/google/firebase/remoteconfig/internal/ConfigContainer.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -139,10 +139,6 @@ public long getTemplateVersionNumber() {
139139
return templateVersionNumber;
140140
}
141141

142-
public ConfigContainer getCopy() throws JSONException {
143-
return ConfigContainer.copyOf(containerJson);
144-
}
145-
146142
@Override
147143
public String toString() {
148144
return containerJson.toString();
@@ -168,7 +164,7 @@ public boolean equals(Object o) {
168164
*/
169165
public Set<String> getChangedParams(ConfigContainer other) throws JSONException {
170166
// Make a copy of the other config before modifying it
171-
JSONObject otherConfig = other.getCopy().getConfigs();
167+
JSONObject otherConfig = ConfigContainer.copyOf(other.containerJson).getConfigs();
172168

173169
// Experiments aren't associated with params, so we can just compare arrays once
174170
Boolean experimentsChanged = !this.getAbtExperiments().equals(other.getAbtExperiments());

0 commit comments

Comments
 (0)