Harmony
  1. Harmony
  2. HARMONY-6533

[classlib] [luni] Proxy.getProxyClass's interfaces paramter should be declared as varargs

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 6.0M1
    • Fix Version/s: 6.0M3, 5.0M15
    • Component/s: Classlib
    • Labels:
      None

      Description

      Currently, java.lang.reflect.Proxy.getProxyClass accepts an array of Class<?> as its second argument: interfaces. According to the Java SE 6 API, however, interfaces should be a varargs parameter. This might not matter in a lot of cases, but it confuses the Scala compiler (scalac) big-time, making it impossible to compile the Scala library against the Apache Harmony classlib (note the array of arrays scalac mistakenly requires):

      found : Array[java.lang.Class[_]]
      required: Array[_ <: Array[java.lang.Class[_]]]
      java.lang.reflect.Proxy.getProxyClass(cl, ifaces: _*)

        Activity

        Hide
        Mark Hindess added a comment -

        Thanks for the bug report.

        Fixed in r950187. I will merge it to the java6 branch in the next day or two. Then it will be available in the stable builds on Hudson at:

        http://hudson.zones.apache.org/hudson/view/Harmony/job/Harmony-1.6-head-linux-x86_64/

        Please confirm it has been fixed by closing this JIRA issue. Thanks.

        Show
        Mark Hindess added a comment - Thanks for the bug report. Fixed in r950187. I will merge it to the java6 branch in the next day or two. Then it will be available in the stable builds on Hudson at: http://hudson.zones.apache.org/hudson/view/Harmony/job/Harmony-1.6-head-linux-x86_64/ Please confirm it has been fixed by closing this JIRA issue. Thanks.
        Hide
        Hudson added a comment -

        Integrated in Harmony-1.5-head-linux-x86_64 #818 (See http://hudson.zones.apache.org/hudson/job/Harmony-1.5-head-linux-x86_64/818/)
        Apply fix for "HARMONY-6533 [classlib] [luni] Proxy.getProxyClass's
        interfaces paramter should be declared as varargs". This changed from
        java 1.4 to java 1.5. I wonder if there are more of these that we have
        missed.

        Show
        Hudson added a comment - Integrated in Harmony-1.5-head-linux-x86_64 #818 (See http://hudson.zones.apache.org/hudson/job/Harmony-1.5-head-linux-x86_64/818/ ) Apply fix for " HARMONY-6533 [classlib] [luni] Proxy.getProxyClass's interfaces paramter should be declared as varargs". This changed from java 1.4 to java 1.5. I wonder if there are more of these that we have missed.
        Hide
        Mark Hindess added a comment -

        I've merge the change to the java6 branch at r950206, so this build should have the fix in it:

        http://hudson.zones.apache.org/hudson/job/Harmony-1.6-head-linux-x86_64/238/

        Show
        Mark Hindess added a comment - I've merge the change to the java6 branch at r950206, so this build should have the fix in it: http://hudson.zones.apache.org/hudson/job/Harmony-1.6-head-linux-x86_64/238/
        Hide
        Hudson added a comment -

        Integrated in Harmony-select-1.5-head-linux-x86_64 #10 (See http://hudson.zones.apache.org/hudson/job/Harmony-select-1.5-head-linux-x86_64/10/)
        Apply fix for "HARMONY-6533 [classlib] [luni] Proxy.getProxyClass's
        interfaces paramter should be declared as varargs". This changed from
        java 1.4 to java 1.5. I wonder if there are more of these that we have
        missed.

        Show
        Hudson added a comment - Integrated in Harmony-select-1.5-head-linux-x86_64 #10 (See http://hudson.zones.apache.org/hudson/job/Harmony-select-1.5-head-linux-x86_64/10/ ) Apply fix for " HARMONY-6533 [classlib] [luni] Proxy.getProxyClass's interfaces paramter should be declared as varargs". This changed from java 1.4 to java 1.5. I wonder if there are more of these that we have missed.
        Hide
        Andreas Sewe added a comment -

        Scalac is happy now.

        Show
        Andreas Sewe added a comment - Scalac is happy now.

          People

          • Assignee:
            Mark Hindess
            Reporter:
            Andreas Sewe
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development