Index: applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/actions/SplitRoundaboutAction.java
===================================================================
--- applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/actions/SplitRoundaboutAction.java	(revision 33725)
+++ applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/actions/SplitRoundaboutAction.java	(revision 33726)
@@ -49,5 +49,5 @@
  * point to the exit point of the roundabout.
  *
- * @author giacomo
+ * @author giacomo, polyglot
  */
 public class SplitRoundaboutAction extends JosmAction {
@@ -108,5 +108,5 @@
         MainApplication.undoRedo.add(getRemoveRoundaboutFromRelationsCommand(roundabout));
 
-        //split the roundabout on the designed nodes
+        //split the roundabout on the designated nodes
         List<Node> splitNodes = getSplitNodes(roundabout);
         SplitWayCommand result = SplitWayCommand.split(
@@ -254,5 +254,5 @@
     private Pair<Way, Way> getEntryExitWays(Relation r, Integer position) {
 
-        //the ways returned are the one exactly before and after the roundabout
+        //the ways returned are the ones exactly before and after the roundabout
         Pair<Way, Way> ret = new Pair<>(null, null);
 
@@ -268,6 +268,6 @@
     }
 
-    //split only on the nodes which might be the
-    //entry or exit point for some public transport route
+    //split on all nodes which are the
+    //entry or exit point for route relations
     public List<Node> getSplitNodes(Way roundabout) {
         Set<Node> noDuplicateSplitNodes = new HashSet<>(roundabout.getNodes());
@@ -277,14 +277,13 @@
             List<Way> parents = n.getParentWays();
             if (parents.size() == 1)
-                return true;
+                return true; // return value for removeIf, not of the method
             parents.remove(roundabout);
             for (Way parent: parents) {
                 if (!getRouteParents(parent).isEmpty()) {
-                        return false;
+                        return false; // return value for removeIf
                 }
             }
-
-            return true;
-        });
+            return true; // return value for removeIf
+            });
         return splitNodes;
     }
