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

Posted to notifications@groovy.apache.org by jameskleeh <gi...@git.apache.org> on 2018/08/02 07:05:10 UTC

[GitHub] groovy pull request #779: GROOVY-8726: Store the method node reference in Pa...

GitHub user jameskleeh opened a pull request:

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

    GROOVY-8726: Store the method node reference in Parameter

    This implementation copies all parameters when they are passed to the method node constructor. While there will be some waste in this process, I find it much easier and maintainable compared to changing all of the places in the codebase where parameters are reused.
    
    Note that the build on my fork is failing with an unrelated compilation error

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

    $ git pull https://github.com/jameskleeh/groovy master

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

    https://github.com/apache/groovy/pull/779.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 #779
    
----
commit 6710d938b557dbf709924d3112d84c8de8e97a99
Author: jameskleeh <ja...@...>
Date:   2018-08-02T04:03:41Z

    GROOVY-8726: Store the method node reference in Parameter

----


---

[GitHub] groovy pull request #779: GROOVY-8726: Store the method node reference in Pa...

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

Github user jameskleeh closed the pull request at:

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


---