java/1390: this in an inner class object refers to the outer class object

java/1390: this in an inner class object refers to the outer class object

[mark]

>Number:         1390
>Category:       java
>Synopsis:       this in an inner class object refers to the outer class object
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    apbianco
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Dec 20 12:19:32 PST 2000
>Closed-Date:    Sun Oct 29 18:33:20 PST 2000
>Last-Modified:  Sun Oct 29 18:40:01 PST 2000
>Originator:     Mark Wielaard
>Release:        gcc version 2.97 20001024 (experimental) + patch from 283
>Organization:
>Environment:

>Description:
When using this in an inner class it seems to refer to the outer class object.
The following should print "class Test@Inner" but prints "class Test".

public class Test {
    public static void main(String[] args) {
        new Test().getInner().m();
    }

    public Inner getInner() {
        return new Inner();
    }

    public class Inner {
        public void m() {
            System.out.println(this.getClass().toString());
        }
    }
}

This could be related to gcj/300
>How-To-Repeat:

>Fix:

>Release-Note:

>Audit-Trail:

Formerly PR gcj/361

State-Changed-From-To: open->feedback
State-Changed-By: apbianco
State-Changed-When: Wed Oct 25 16:53:57 2000
State-Changed-Why:
    I have a patch.

From: apbianco@cygnus.com
To: apbianco@cygnus.com, java-gnats@sourceware.cygnus.com, mark@klomp.org
Cc:  
Subject: Re: gcj/361
Date: 25 Oct 2000 23:53:57 -0000

 Synopsis: this in an inner class object refers to the outer class object
 
 State-Changed-From-To: open->feedback
 State-Changed-By: apbianco
 State-Changed-When: Wed Oct 25 16:53:57 2000
 State-Changed-Why:
     I have a patch.
 
 http://sources.redhat.com/cgi-bin/gnatsweb.pl?cmd=view&pr=361&database=java

From: Mark Wielaard <mark@klomp.org>
To: apbianco@cygnus.com
Cc: java-prs@sourceware.cygnus.com
Subject: Re: gcj/361
Date: Fri, 27 Oct 2000 01:06:10 +0200

 Hi,
 
 Verified with a current compiler gcc version 2.97 20001026 (experimental)
 with the patches for gcj/283 and gcj/361 applied and now "this" refers
 correctly to the inner class object.
 
 I also rechecked gcj/362 and gcj/363 with these patches applied but
 those bugs are still present.
 
 Thanks,
 
 Mark

From: Alexandre Petit-Bianco <apbianco@cygnus.com>
To: java-gnats@sourceware.cygnus.com
Cc:  
Subject: Re: gcj/361
Date: Thu, 26 Oct 2000 16:09:40 -0700 (PDT)

 Mark Wielaard writes:
 
 > Verified with a current compiler gcc version 2.97 20001026
 > (experimental) with the patches for gcj/283 and gcj/361 applied and
 > now "this" refers correctly to the inner class object.
 
 Thanks.
 
 > I also rechecked gcj/362 and gcj/363 with these patches applied but
 > those bugs are still present.
 
 Yes, they're different problems.
 
 ./A
State-Changed-From-To: feedback->closed
State-Changed-By: apbianco
State-Changed-When: Sun Oct 29 18:33:20 2000
State-Changed-Why:
    I checked in a patch:
      http://gcc.gnu.org/ml/gcc-patches/2000-10/msg00993.html

From: apbianco@cygnus.com
To: apbianco@cygnus.com, java-gnats@sourceware.cygnus.com, mark@klomp.org
Cc:  
Subject: Re: gcj/361
Date: 30 Oct 2000 02:33:20 -0000

 Synopsis: this in an inner class object refers to the outer class object
 
 State-Changed-From-To: feedback->closed
 State-Changed-By: apbianco
 State-Changed-When: Sun Oct 29 18:33:20 2000
 State-Changed-Why:
     I checked in a patch:
       http://gcc.gnu.org/ml/gcc-patches/2000-10/msg00993.html
 
 http://sources.redhat.com/cgi-bin/gnatsweb.pl?cmd=view&pr=361&database=java
>Unformatted:
----gnatsweb-attachment----
Content-Type: application/octet-stream; name="patch"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="patch"

MjAwMC0xMC0yNSAgQWxleGFuZHJlIFBldGl0LUJpYW5jbyAgPGFwYmlhbmNvQGN5Z251cy5jb20+
CgoJKiBwYXJzZS55IChwYXRjaF9tZXRob2RfaW52b2NhdGlvbik6IE5VTExpZnkgdGhpc19hcmcg
d2hlbiBhbHJlYWR5CglpbnNlcnRlZC4KCShtYXliZV91c2VfYWNjZXNzX21ldGhvZCk6IEhhbmRs
ZSBjYWxsIHRvIG1ldGhvZHMgdW5yZWxhdGVkIHRvIHRoZQoJY3VycmVudCBjbGFzcy4KCkluZGV4
OiBwYXJzZS55Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT0KUkNTIGZpbGU6IC9jdnMvZ2NjL2VnY3MvZ2NjL2phdmEvcGFy
c2UueSx2CnJldHJpZXZpbmcgcmV2aXNpb24gMS4yMjEKZGlmZiAtdSAtcCAtcjEuMjIxIHBhcnNl
LnkKLS0tIHBhcnNlLnkJMjAwMC8xMC8yNSAwNTo0NzoyOAkxLjIyMQorKysgcGFyc2UueQkyMDAw
LzEwLzI1IDIzOjIwOjI4CkBAIC05OTMwLDcgKzk5MzAsMTAgQEAgcGF0Y2hfbWV0aG9kX2ludm9j
YXRpb24gKHBhdGNoLCBwcmltYXJ5LAogCSAgICAgYXJndW1lbnQgbGlzdC4gSW4gdGhlIG1lYW50
aW1lLCB0aGUgc2VsZWN0ZWQgZnVuY3Rpb24KIAkgICAgIG1pZ2h0IGhhdmUgYmUgcmVwbGFjZWQg
YnkgYSBnZW5lcmF0ZWQgc3R1Yi4gKi8KIAkgIGlmIChtYXliZV91c2VfYWNjZXNzX21ldGhvZCAo
aXNfc3VwZXJfaW5pdCwgJmxpc3QsICZ0aGlzX2FyZykpCi0JICAgIGFyZ3MgPSB0cmVlX2NvbnMg
KE5VTExfVFJFRSwgdGhpc19hcmcsIGFyZ3MpOworCSAgICB7CisJICAgICAgYXJncyA9IHRyZWVf
Y29ucyAoTlVMTF9UUkVFLCB0aGlzX2FyZywgYXJncyk7CisJICAgICAgdGhpc19hcmcgPSBOVUxM
X1RSRUU7IC8qIFNvIGl0IGRvZXNuJ3QgZ2V0IGNoYWluZWQgdHdpY2UgKi8KKwkgICAgfQogCX0K
ICAgICB9CiAKQEAgLTEwMDg2LDkgKzEwMDg5LDEwIEBAIG1heWJlX3VzZV9hY2Nlc3NfbWV0aG9k
IChpc19zdXBlcl9pbml0LCAKICAgCiAgIC8qIElmIHdlJ3JlIGNhbGxpbmcgYSBtZXRob2QgZm91
bmQgaW4gYW4gZW5jbG9zaW5nIGNsYXNzLCBnZW5lcmF0ZQogICAgICB3aGF0IGl0IHRha2VzIHRv
IHJldHJpZXZlIHRoZSByaWdodCB0aGlzLiBEb24ndCBkbyB0aGF0IGlmIHdlJ3JlCi0gICAgIGlu
dm9raW5nIGEgc3RhdGljIG1ldGhvZC4gKi8KKyAgICAgaW52b2tpbmcgYSBzdGF0aWMgbWV0aG9k
LiBOb3RlIHRoYXQgaWYgTUQncyB0eXBlIGlzIHVucmVsYXRlZCB0bworICAgICBDVVJSRU5UX0NM
QVNTLCB0aGVuIHRoZSBjdXJyZW50IHRoaXMgY2FuIGJlIHVzZWQuICovCiAKLSAgaWYgKG5vbl9z
dGF0aWNfY29udGV4dCkKKyAgaWYgKG5vbl9zdGF0aWNfY29udGV4dCAmJiBERUNMX0NPTlRFWFQg
KG1kKSAhPSBvYmplY3RfdHlwZV9ub2RlKQogICAgIHsKICAgICAgIGN0eCA9IFRSRUVfVFlQRSAo
REVDTF9DT05URVhUIChUWVBFX05BTUUgKGN1cnJlbnRfY2xhc3MpKSk7CiAgICAgICBpZiAoaW5o
ZXJpdHNfZnJvbV9wIChjdHgsIERFQ0xfQ09OVEVYVCAobWQpKSkK