JDK-8153041 : Remove unused redundant parameter in CLDRConverter
  • Type: Bug
  • Component: core-libs
  • Sub-Component: java.util:i18n
  • Affected Version: 9
  • Priority: P3
  • Status: Closed
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2016-03-29
  • Updated: 2016-06-13
  • Resolved: 2016-04-04
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 9
9 b114Fixed
Related Reports
Relates :  
Description
The suspect code:

private String getTarget(String qName, String path, String calType, String context, String width) {
     // qName
     int lastSlash = path.lastIndexOf('/');
     qName = path.substring(lastSlash+1);
     int bracket = qName.indexOf('[');
     if (bracket != -1) {
         qName = qName.substring(0, bracket);
     }

So there's a 'qName' parameter to this method, but it is never used -- instead, it is immediately overwritten by a substring of 'path'.

Here's the only call site:

Entry<?> entry = (Entry<?>) currentContainer;
String containerqName = entry.getParent().getqName();
...
String target = getTarget(containerqName, entry.getKey(), ...

So there's no obvious relationship between 'containerqName' and the eventual value of 'qName' in 'getTarget'.