Changeset 8375 in josm for trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java

Timestamp:

2015-05-17T03:06:35+02:00 (9 years ago)

Author:

Don-vip

Message:

code style - Method stores return result in local before immediately returning it

File:

• trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java

@@ -171 +171 @@
 
         boolean __add_content(T o) {
-            boolean ret = false;
             // The split_lock will keep two concurrent calls from overwriting content
             if (content == null) {
                 content = new ArrayList<>();
             }
-            ret = content.add(o);
-            return ret;
+            return content.add(o);
         }
 
@@ -229 +227 @@
             QBLevel<T> next = this;
             QBLevel<T> sibling = next.next_sibling();
-            // Walk back up the tree to find the
-            // next sibling node.  It may be either
-            // a leaf or branch.
+            // Walk back up the tree to find the next sibling node.
+            // It may be either a leaf or branch.
             while (sibling == null) {
                 next = next.parent;
@@ -239 +236 @@
                 sibling = next.next_sibling();
             }
-            next = sibling;
-            return next;
+            return sibling;
         }