{"action":"created","issue":{"url":"https://api.github.com/repos/checkstyle/checkstyle/issues/758","labels_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/758/labels{/name}","comments_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/758/comments","events_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/758/events","html_url":"https://github.com/checkstyle/checkstyle/pull/758","id":60801992,"number":758,"title":"Issue #286","user":{"login":"liscju","id":6627878,"avatar_url":"https://avatars.githubusercontent.com/u/6627878?v=3","gravatar_id":"","url":"https://api.github.com/users/liscju","html_url":"https://github.com/liscju","followers_url":"https://api.github.com/users/liscju/followers","following_url":"https://api.github.com/users/liscju/following{/other_user}","gists_url":"https://api.github.com/users/liscju/gists{/gist_id}","starred_url":"https://api.github.com/users/liscju/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/liscju/subscriptions","organizations_url":"https://api.github.com/users/liscju/orgs","repos_url":"https://api.github.com/users/liscju/repos","events_url":"https://api.github.com/users/liscju/events{/privacy}","received_events_url":"https://api.github.com/users/liscju/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-03-12T11:01:14Z","updated_at":"2015-03-20T10:46:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/checkstyle/checkstyle/pulls/758","html_url":"https://github.com/checkstyle/checkstyle/pull/758","diff_url":"https://github.com/checkstyle/checkstyle/pull/758.diff","patch_url":"https://github.com/checkstyle/checkstyle/pull/758.patch"},"body":"Introduce new parameter in ModifiedControlVariable check ignoreForControlVariablesNotIterated to ignore changes in variable inside for loop which are not change in iterator part of for loop. To do it before adding all variables declared in for first part to current variable stack, we calculate all variables which are changed in iterator part of for loop. The cases in this iterator part which can change defined variables are: \r\npostincrement,preincrement,postdecrement,predecrement with IDENT operator in it(simple variable name),\r\nassignment with IDENT as first child(assignment to simple variable name). We only add variable to variable stack when it is not changed in for iterator part or ignoreForControlVariablesNotIterated is false. When we are leaving for part we pop it according to how many variables was put in it."},"comment":{"url":"https://api.github.com/repos/checkstyle/checkstyle/issues/comments/83984409","html_url":"https://github.com/checkstyle/checkstyle/pull/758#issuecomment-83984409","issue_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/758","id":83984409,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars.githubusercontent.com/u/2354108?v=3","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2015-03-20T10:46:09Z","updated_at":"2015-03-20T10:46:09Z","body":"\n[![Coverage Status](https://coveralls.io/builds/2157100/badge)](https://coveralls.io/builds/2157100)\n\nCoverage increased (+0.03%) to 90.17% when pulling **e653c66459df877981fac6fb874ed9dd00687fe3 on liscju:issue286** into **59d1b6ad1ff2efe155bc1006a4f7b7b5c88421f8 on checkstyle:master**.\n"}}