{"comment":{"url":"https://api.github.com/repos/xen0n/android-maven-plugin/comments/10158078","html_url":"https://github.com/xen0n/android-maven-plugin/commit/24ce08241e927bb13b1f978a83cab9c0973966d6#commitcomment-10158078","id":10158078,"user":{"login":"xen0n","id":1175567,"avatar_url":"https://avatars.githubusercontent.com/u/1175567?v=3","gravatar_id":"","url":"https://api.github.com/users/xen0n","html_url":"https://github.com/xen0n","followers_url":"https://api.github.com/users/xen0n/followers","following_url":"https://api.github.com/users/xen0n/following{/other_user}","gists_url":"https://api.github.com/users/xen0n/gists{/gist_id}","starred_url":"https://api.github.com/users/xen0n/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xen0n/subscriptions","organizations_url":"https://api.github.com/users/xen0n/orgs","repos_url":"https://api.github.com/users/xen0n/repos","events_url":"https://api.github.com/users/xen0n/events{/privacy}","received_events_url":"https://api.github.com/users/xen0n/received_events","type":"User","site_admin":false},"position":47,"line":239,"path":"src/main/java/com/jayway/maven/plugins/android/standalonemojos/ManifestMergerMojo.java","commit_id":"24ce08241e927bb13b1f978a83cab9c0973966d6","created_at":"2015-03-12T06:50:24Z","updated_at":"2015-03-12T06:50:24Z","body":"OK, the old impl has been moved to the class `SimpleVersionElementParser`. I didn't touch any of the parsing code.\r\n\r\nNow, maybe `versionName.replaceAll( \"[^0-9.]\", \"\" ).split( \"\\\\.\" )` is the code creating your confusion. At best we still have to dynamically generate a regex that has the correct number of capturing groups (no, repeating groups like `(?:\\.([^.]+))+` won't work, [regexes don't work like this](http://www.regular-expressions.info/captureall.html), at least in Java). Then to make `Integer.toString()` happy we would then have to do a `replaceAll` on each captured part, which seems not that elegant; certainly worse than the original impl.\r\n\r\nHope this clears things up."}}