[jira] [Commented] (MCHECKSTYLE-344) StringIndexOutOfBoundsException in RuleUtil

["Jonathan Haber (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MCHECKSTYLE-344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16371903#comment-16371903 ] 

Jonathan Haber commented on MCHECKSTYLE-344:
--------------------------------------------

I just wanted to report that we're also experiencing this issue in our codebase and that the PR James sent looks good to me.

> StringIndexOutOfBoundsException in RuleUtil
> -------------------------------------------
>
>                 Key: MCHECKSTYLE-344
>                 URL: https://issues.apache.org/jira/browse/MCHECKSTYLE-344
>             Project: Maven Checkstyle Plugin
>          Issue Type: Bug
>          Components: checkstyle:check
>    Affects Versions: 2.17
>            Reporter: fdvxxii
>            Priority: Major
>
> {{RuleUtil}} has a bug at line 95:
> {code}
> 		final int end = eventSrcName.lastIndexOf('.');
> 		eventSrcName = eventSrcName.substring(0, end);		
> {code}
> This code leads to a StringIndexOutOfBoundsException if the variable does not contain a period. I don't know if its a convention to have a period in that string, but either way the error message should me more expressive.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)