Name: rmT116609 Date: 05/16/2002


FULL PRODUCT VERSION :
java version "1.4.0"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.4.0-b92)
Java HotSpot(TM) 64-Bit Server VM (build 1.4.0-b92, mixed mode)


FULL OPERATING SYSTEM VERSION :
SunOS mars 5.8 Generic_108528-14 sun4u sparc SUNW,Ultra-Enterprise

DESCRIPTION OF THE PROBLEM :
When performing independent package compilations, the java compiler allows a non-public implementation of an interface method.

An interface and abstract class must exist and be compiled in one package. In another package, a concrete class extends the abstract class, but implements a non-public version of the method declared in the interface. The compiler does not report an error.


STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :

(refer to attached classes)
-----
If each package is compiled independently, the compiler does NOT report any errors and produces classfiles for all java files. Attempting to call ConcreteTest_object.perform() results in a runtime error.

  From the top directory:
> javac -d . Test.java AbstractTest.java
> javac -d . ConcreteTest.java

No error is reported

On Windows 2000 using JDK1.4.0:
--------------------------------

C:\>javac -d . Test.java AbstractTest.java
C:\>javac -d . ConcreteTest.java
ConcreteTest.java:7: perform() in two.ConcreteTest cannot implement perform() in one.AbstractTe
st; attempting to assign weaker access privileges; was public
    private void perform()

-----
If both packages are compiled in a single command, the compiler correctly indicates that the method must be public.

> javac -d . Test.java AbstractTest.java ConcreteTest.java

produces the correct error:

two/ConcreteTest.java:8: perform() in two.ConcreteTest
cannot implement perform() in one.AbstractTest; attempting
to assign weaker access privileges; was public
    private void perform()
                 ^
1 error

-----

If an output directory is specified for the compiler, and
compilation is performed in multiple steps, the result
appears to be either of the above depending on the location
of files and prior existence of classfiles.

> javac -d out one/*.java
> javac -d out two/*.java

For the attached classes this produces the error message,
but some similar test cases did not.


This bug can be reproduced always.

---------- BEGIN SOURCE ----------
//------------------ file one/Test.java
package one;

public interface Test
{
    void perform();
}


//------------------ file one/AbstractTest.java
package one;

public abstract class AbstractTest implements Test
{
}


//------------------ file two/ConcreteTest.java
package two;
import one.*;

public class ConcreteTest extends AbstractTest
{
    // ### should be public
    private void perform()
    {
    }
}

---------- END SOURCE ----------
(Review ID: 146658)
======================================================================