Final Local Variable Check, fixed false-positive, issue #258 #601

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@alexkravin
Contributor

alexkravin commented Jan 26, 2015

According to #258

Fixed false-positive, added corresponding UT and input.

So now this Check doesn't put violation on variables in for init (except for first one).

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 26, 2015

Coverage Status

Coverage decreased (-0.01%) to 90.61% when pulling f565eb2 on alexkravin:FinalLocalVariableCheck#258 into 204c073 on checkstyle:master.

Coverage Status

Coverage decreased (-0.01%) to 90.61% when pulling f565eb2 on alexkravin:FinalLocalVariableCheck#258 into 204c073 on checkstyle:master.

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Jan 27, 2015

Member

merged as FF.

Member

romani commented Jan 27, 2015

merged as FF.

@romani romani closed this Jan 27, 2015

@romani

This comment has been minimized.

Show comment
Hide comment
@romani

romani Jan 27, 2015

Member

@alexkravin , recheck why coverage is decreased and fix that.

Member

romani commented Jan 27, 2015

@alexkravin , recheck why coverage is decreased and fix that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment