[GitHub] ant pull request #59: Fix NPE in ChainedMapper.

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] ant pull request #59: Fix NPE in ChainedMapper.

twogee
GitHub user jpountz opened a pull request:

    https://github.com/apache/ant/pull/59

    Fix NPE in ChainedMapper.

    This NPE happens whenever any of the sub mappers returns `null`, which may
    happen eg. with `GlobPatternMapper`.

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

    $ git pull https://github.com/jpountz/ant fix/chained_mapper_npe

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

    https://github.com/apache/ant/pull/59.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 #59
   
----
commit e777ee85a5c077949c81e23c359cf857dd13ca83
Author: Adrien Grand <jpountz@...>
Date:   2018-02-08T14:22:02Z

    Fix NPE in ChainedMapper.
   
    This NPE happens whenever any of the sub mappers returns `null`, which may
    happen eg. with `GlobPatternMapper`.

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant issue #59: Fix NPE in ChainedMapper.

twogee
Github user asfgit commented on the issue:

    https://github.com/apache/ant/pull/59
 
   
    Refer to this link for build results (access rights to CI server needed):
    https://builds.apache.org/job/Ant%20Github-PR-Windows/38/



---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant issue #59: Fix NPE in ChainedMapper.

twogee
In reply to this post by twogee
Github user asfgit commented on the issue:

    https://github.com/apache/ant/pull/59
 
   
    Refer to this link for build results (access rights to CI server needed):
    https://builds.apache.org/job/Ant%20Github-PR-Linux/32/



---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant issue #59: Fix NPE in ChainedMapper.

twogee
In reply to this post by twogee
Github user jaikiran commented on the issue:

    https://github.com/apache/ant/pull/59
 
    Thanks for the PR @jpountz. Can we add you to our contributors file, crediting you for this fix? If yes, what would you like your name to be mentioned as?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] ant pull request #59: Fix NPE in ChainedMapper.

twogee
In reply to this post by twogee
Github user asfgit closed the pull request at:

    https://github.com/apache/ant/pull/59


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]