[Bug java/16474] New: gcjh: Illegal C++ produced for some non-gcj bytecode

[Bug java/16474] New: gcjh: Illegal C++ produced for some non-gcj bytecode

[mckinlay at redhat dot com]

When producing header files from bytecode produced by ecj, gcjh can emit invalid
C++ headers. For example, given the following code in libgcj,

public class SimpleSHSStream extends java.io.DataOutputStream
{
  int counter;
 
  final int SHS_BLOCKSIZE = 64;
  final int SHS_DIGESTSIZE = 20;

gcjh will emit:

public:
  const jint SHS_BLOCKSIZE = 64L;
  const jint SHS_DIGESTSIZE = 20L;

But this is illegal ISO C++:

./gnu/gcj/io/SimpleSHSStream.h:40: error: ISO C++ forbids initialization of
member `SHS_BLOCKSIZE'
./gnu/gcj/io/SimpleSHSStream.h:40: error: making `SHS_BLOCKSIZE' static
./gnu/gcj/io/SimpleSHSStream.h:41: error: ISO C++ forbids initialization of
member `SHS_DIGESTSIZE'
./gnu/gcj/io/SimpleSHSStream.h:41: error: making `SHS_DIGESTSIZE' static

-- 
           Summary: gcjh: Illegal C++ produced for some non-gcj bytecode
           Product: gcc
           Version: 3.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: java
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: mckinlay at redhat dot com
                CC: gcc-bugs at gcc dot gnu dot org,java-prs at gcc dot gnu
                    dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16474

[Bug java/16474] gcjh: Illegal C++ produced for some non-gcj bytecode

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-07-12 05:51 -------
Confirmed and patch here: <http://gcc.gnu.org/ml/gcc-patches/2004-07/msg01164.html>.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
           Keywords|                            |patch
   Last reconfirmed|0000-00-00 00:00:00         |2004-07-12 05:51:45
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16474

[Bug java/16474] gcjh: Illegal C++ produced for some non-gcj bytecode

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-07-12 18:06 -------
Subject: Bug 16474

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	bryce@gcc.gnu.org	2004-07-12 18:06:04

Modified files:
	gcc/java       : ChangeLog gjavah.c 

Log message:
	2004-07-12  Bryce McKinlay  <mckinlay@redhat.com>
	
	PR java/16474
	gjavah.c (print_field_info): Emit constant only if field is static.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/java/ChangeLog.diff?cvsroot=gcc&r1=1.1412&r2=1.1413
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/java/gjavah.c.diff?cvsroot=gcc&r1=1.116&r2=1.117



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16474

[Bug java/16474] gcjh: Illegal C++ produced for some non-gcj bytecode

[mckinlay at redhat dot com]

------- Additional Comments From mckinlay at redhat dot com  2004-07-12 18:13 -------
Fixed

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |3.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16474