This is the mail archive of the mauve-patches@sources.redhat.com mailing list for the Mauve project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Package statement error


Hi,

On Sat, 2004-08-14 at 21:55, David Gilbert wrote:
> There is a package statement error in
> gnu.testlet.java.awt.image.Kernel.check.java.  It says:
> 
> package gnu.testlet.java.awt.image.DataBufferByte;
> 
> ...but presumably should be:
> 
> package gnu.testlet.java.awt.image.Kernel;
> 

Yup. And it doesn't compile with a recent jikes compiler.
This fixes both:

2004-08-14  Mark Wielaard  <mark@klomp.org>

        Reported by David Gilbert <david.gilbert@object-refinery.com>
        * gnu/testlet/java/awt/image/Kernel/check.java:
        Set package to gnu.testlet.java.awt.image.Kernel.
        Make sure data1 is always initialized.

With this all 13 tests pass with kaffe CVS.

Cheers,

Mark

--- gnu/testlet/java/awt/image/Kernel/check.java        6 Aug 2004 23:11:08 -0000       1.2
+++ gnu/testlet/java/awt/image/Kernel/check.java        14 Aug 2004 21:28:33 -0000
@@ -19,7 +19,7 @@
 // the Free Software Foundation, 59 Temple Place - Suite 330,
 // Boston, MA 02111-1307, USA.

-package gnu.testlet.java.awt.image.DataBufferByte;
+package gnu.testlet.java.awt.image.Kernel;

 import gnu.testlet.Testlet;
 import gnu.testlet.TestHarness;
@@ -49,6 +49,7 @@
       }
     catch (IllegalArgumentException e)
       {
+       data1 = new float[0];
        h.fail("Kernel.getKernelData");
       }


Attachment: signature.asc
Description: This is a digitally signed message part


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]