{"action":"created","issue":{"url":"https://api.github.com/repos/mockito/mockito/issues/426","repository_url":"https://api.github.com/repos/mockito/mockito","labels_url":"https://api.github.com/repos/mockito/mockito/issues/426/labels{/name}","comments_url":"https://api.github.com/repos/mockito/mockito/issues/426/comments","events_url":"https://api.github.com/repos/mockito/mockito/issues/426/events","html_url":"https://github.com/mockito/mockito/issues/426","id":159773328,"number":426,"title":"Refactor instance based utility classes to static utility classes","user":{"login":"ChristianSchwarz","id":1437411,"avatar_url":"https://avatars.githubusercontent.com/u/1437411?v=3","gravatar_id":"","url":"https://api.github.com/users/ChristianSchwarz","html_url":"https://github.com/ChristianSchwarz","followers_url":"https://api.github.com/users/ChristianSchwarz/followers","following_url":"https://api.github.com/users/ChristianSchwarz/following{/other_user}","gists_url":"https://api.github.com/users/ChristianSchwarz/gists{/gist_id}","starred_url":"https://api.github.com/users/ChristianSchwarz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ChristianSchwarz/subscriptions","organizations_url":"https://api.github.com/users/ChristianSchwarz/orgs","repos_url":"https://api.github.com/users/ChristianSchwarz/repos","events_url":"https://api.github.com/users/ChristianSchwarz/events{/privacy}","received_events_url":"https://api.github.com/users/ChristianSchwarz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-06-11T13:23:18Z","updated_at":"2016-06-12T18:51:41Z","closed_at":null,"body":"The internal implementation of Mockito contains numerous helper classed that serve as a container for useful methods. By its nature these classes don't have a state. Currently some them are instance based in other word you need to create the utility class to call a helper method. This not only pollutes the heap but also the code cause instance methods can't be imported statically. \r\n\r\nHere is an example:\r\n```\r\nHelper helper = new Helper();\r\nif (helper.isInputValid(input)){\r\n  [...] \r\n}\r\n```\r\nvs. static import of Helper.isInputValid\r\n```\r\nif (isInputValid(input)){\r\n  [...] \r\n}\r\n```\r\n\r\n\r\nThe aim of this ticket is to identify canidates that can be refactored to static utility classes. If you like to refactoring and mockito feel free to send a PR and reference this issue.\r\n\r\nRefactoring canidates:\r\n - [ ] `ArgumentsComparator`\r\n - [x] `FieldSetter` fixed in PR #427\r\n - [x] `Reporter` fixed in PR #427\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/mockito/mockito/issues/comments/225453716","html_url":"https://github.com/mockito/mockito/issues/426#issuecomment-225453716","issue_url":"https://api.github.com/repos/mockito/mockito/issues/426","id":225453716,"user":{"login":"ChristianSchwarz","id":1437411,"avatar_url":"https://avatars.githubusercontent.com/u/1437411?v=3","gravatar_id":"","url":"https://api.github.com/users/ChristianSchwarz","html_url":"https://github.com/ChristianSchwarz","followers_url":"https://api.github.com/users/ChristianSchwarz/followers","following_url":"https://api.github.com/users/ChristianSchwarz/following{/other_user}","gists_url":"https://api.github.com/users/ChristianSchwarz/gists{/gist_id}","starred_url":"https://api.github.com/users/ChristianSchwarz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ChristianSchwarz/subscriptions","organizations_url":"https://api.github.com/users/ChristianSchwarz/orgs","repos_url":"https://api.github.com/users/ChristianSchwarz/repos","events_url":"https://api.github.com/users/ChristianSchwarz/events{/privacy}","received_events_url":"https://api.github.com/users/ChristianSchwarz/received_events","type":"User","site_admin":false},"created_at":"2016-06-12T18:51:41Z","updated_at":"2016-06-12T18:51:41Z","body":"@TimvdLippeI know what you mean! The next PR's include only one refactored class at a time."}}
{"actor":{"display_login":"ChristianSchwarz"}}