[ https://issues.apache.org/jira/browse/MNG-8179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17864507#comment-17864507 ]
ASF GitHub Bot commented on MNG-8179: ------------------------------------- slawekjaranowski commented on code in PR #1605: URL: https://github.com/apache/maven/pull/1605#discussion_r1671730082 ########## maven-api-impl/src/main/java/org/apache/maven/internal/impl/DefaultToolchainsXmlFactory.java: ########## @@ -53,10 +52,6 @@ public PersistedToolchains read(@Nonnull XmlReaderRequest request) throws XmlRea throw new IllegalArgumentException("reader or inputStream must be non null"); } try { - InputSource source = null; - if (request.getModelId() != null || request.getLocation() != null) { - source = new InputSource(request.getModelId(), request.getLocation()); - } Review Comment: Yes new rule was detected it. @gnodet - why is not used here ....? > Upgrade Parent to 43 > -------------------- > > Key: MNG-8179 > URL: https://issues.apache.org/jira/browse/MNG-8179 > Project: Maven > Issue Type: Dependency upgrade > Reporter: Slawomir Jaranowski > Assignee: Slawomir Jaranowski > Priority: Major > Fix For: 4.0.0 > > -- This message was sent by Atlassian Jira (v8.20.10#820010)