You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@groovy.apache.org by Linary <gi...@git.apache.org> on 2018/03/02 03:46:44 UTC

[GitHub] groovy pull request #669: Fix ArrayIndexOutOfBoundsException in CallSiteWrit...

GitHub user Linary opened a pull request:

    https://github.com/apache/groovy/pull/669

    Fix ArrayIndexOutOfBoundsException in CallSiteWriter.getCreateArraySignature()

    It would throw `ArrayIndexOutOfBoundsException` if the `numberOfArguments` exceed 255,
    but there is no check for array index and it's hard to get the root cause to users.
    
    We should throw a more readable exception here.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Linary/groovy array-index-check2

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/groovy/pull/669.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #669
    
----
commit 022fd903b2ed69e01613bd3ede4a22c3deeb7662
Author: liningrui <li...@...>
Date:   2018-03-02T03:44:23Z

    Fix ArrayIndexOutOfBoundsException in CallSiteWriter.getCreateArraySignature()
    
    It would throw ArrayIndexOutOfBoundsException if the numberOfArguments exceed 255,
    but there is no check for array index and it's hard to get the root cause to users.
    
    We should throw a more readable exception here.

----


---

[GitHub] groovy pull request #669: Fix ArrayIndexOutOfBoundsException in CallSiteWrit...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/groovy/pull/669


---