Hi,

On 27.04.19 14:07, Enrico Olivelli wrote:
Il sab 27 apr 2019, 13:41 Hervé BOUTEMY <herve.bout...@free.fr> ha scritto:

Hi,

There are 2 fixes:
1. MNG-6636 an issue in Maven model inheritance of input location that
leads to no input location tracking where it should be inherited
2. MNG-6644 and MNG-6647 where some external extensions don't inject input
location tracking, then reporting has to be tolerant to missing data

See for each the Jira issue, the associated fix and the CI Job:

1. MNG-6636:
https://issues.apache.org/jira/browse/MNG-6636

https://gitbox.apache.org/repos/asf?p=maven.git;a=commitdiff;h=dc043c978b48cfe05cd2d5ebd86f8f6c82702f95

+1 for both of them...



Nit: Why are you using Integer.valueOf ?

The usual constructor like new Integer(5); has been marked deprecated
with Java9. This is also the case for new Long(..). Those should be
replaced with Integer.valueOf() or Long.valueOf()...

The valueOf() variants do some caching on JDK level which can speed up
things...

Kind regards
Karl Heinz Marbaise



Would it be useful to add a test?

Maybe for review it is easier to comment on a PR on github for the future



https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven/job/MNG-6636/

2. MNG-6644 and MNG-6647
https://issues.apache.org/jira/browse/MNG-6644
https://issues.apache.org/jira/browse/MNG-6647

https://gitbox.apache.org/repos/asf?p=maven.git;a=commitdiff;h=fb8edc076da0a310995866bfa4567e9b15c47bad

https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven/job/MNG-6644/


+1

Enrico




Seconders, please?

Regards,

Hervé

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Reply via email to