Refactor LocatedItems to use the function addItem() instead of

initItemsIfNull().

1 files changed, 8 insertions, 8 deletions

diff --git a/dexlib2/src/main/java/org/jf/dexlib2/builder/LocatedItems.java b/dexlib2/src/main/java/org/jf/dexlib2/builder/LocatedItems.java
index 261742f9..89f0ac11 100644
--- a/dexlib2/src/main/java/org/jf/dexlib2/builder/LocatedItems.java
+++ b/dexlib2/src/main/java/org/jf/dexlib2/builder/LocatedItems.java
@@ -13,12 +13,6 @@ public abstract class LocatedItems<T extends ItemWithLocation> {
     @Nullable
     private List<T> items = null;
 
-    private void initItemsIfNull() {
-        if (items == null) {
-            items = new ArrayList<>(1);
-        }
-    }
-
     @Nonnull
     private List<T> getItems() {
         if (items == null) {
@@ -70,13 +64,19 @@ public abstract class LocatedItems<T extends ItemWithLocation> {
                     throw new IllegalArgumentException(getAddLocatedItemError());
                 }
                 item.setLocation(newItemsLocation);
-                initItemsIfNull();
-                items.add(item);
+                addItem(item);
                 return true;
             }
         };
     }
 
+    private void addItem(@Nonnull T item) {
+        if (items == null) {
+            items = new ArrayList<>(1);
+        }
+        items.add(item);
+    }
+
     protected abstract String getAddLocatedItemError();
 
     public void mergeItemsIntoNext(@Nonnull MethodLocation nextLocation, LocatedItems<T> otherLocatedItems) {