{"comment":{"url":"https://api.github.com/repos/taconaut/ums-mlx/comments/13579972","html_url":"https://github.com/taconaut/ums-mlx/commit/02016d53438b289eb3816efbf9af0704478145b9#commitcomment-13579972","id":13579972,"user":{"login":"skeptical","id":1594203,"avatar_url":"https://avatars.githubusercontent.com/u/1594203?v=3","gravatar_id":"","url":"https://api.github.com/users/skeptical","html_url":"https://github.com/skeptical","followers_url":"https://api.github.com/users/skeptical/followers","following_url":"https://api.github.com/users/skeptical/following{/other_user}","gists_url":"https://api.github.com/users/skeptical/gists{/gist_id}","starred_url":"https://api.github.com/users/skeptical/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/skeptical/subscriptions","organizations_url":"https://api.github.com/users/skeptical/orgs","repos_url":"https://api.github.com/users/skeptical/repos","events_url":"https://api.github.com/users/skeptical/events{/privacy}","received_events_url":"https://api.github.com/users/skeptical/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"02016d53438b289eb3816efbf9af0704478145b9","created_at":"2015-10-04T12:10:20Z","updated_at":"2015-10-04T12:10:20Z","body":"@taconaut Maybe we're not on the same page somehow :).  I'm talking about the while in [RootFolder.addAdditionalFoldersAtRoot()](https://github.com/taconaut/ums-mlx/blob/master/core/src/main/java/net/pms/medialibrary/dlna/RootFolder.java#L114):\r\n```java\r\nwhile ((child = l.getChildren().next()) != null) {\r\n```\r\nin every cycle of this while a fresh iterator is created by calling ```l.getChildren()```, which then fetches the first child via ```next()```, then in the next cycle ```l.getChildren()``` creates another fresh iterator, (discarding the previous iterator), so its ```next()``` again fetches the first child.  We never complete the iterator because we keep replacing it before every ```next()``` call."}}