From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5556 invoked by alias); 11 Mar 2009 17:38:13 -0000 Received: (qmail 5545 invoked by uid 22791); 11 Mar 2009 17:38:12 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL,BAYES_00,J_CHICKENPOX_42,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mx2.redhat.com (HELO mx2.redhat.com) (66.187.237.31) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 11 Mar 2009 17:38:02 +0000 Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id n2BHbxD2017404 for ; Wed, 11 Mar 2009 13:37:59 -0400 Received: from ns3.rdu.redhat.com (ns3.rdu.redhat.com [10.11.255.199]) by int-mx2.corp.redhat.com (8.13.1/8.13.1) with ESMTP id n2BHbxwB032085; Wed, 11 Mar 2009 13:37:59 -0400 Received: from zebedee.pink (vpn-12-253.rdu.redhat.com [10.11.12.253]) by ns3.rdu.redhat.com (8.13.8/8.13.8) with ESMTP id n2BHbvN1032451; Wed, 11 Mar 2009 13:37:58 -0400 Message-ID: <49B7F6F4.1060106@redhat.com> Date: Wed, 11 Mar 2009 17:38:00 -0000 From: Andrew Haley User-Agent: Thunderbird 2.0.0.17 (X11/20081009) MIME-Version: 1.0 To: Tom Tromey CC: GCJ Subject: Patch for _Jv_CheckOrCreateLoadingConstraint Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact java-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: java-owner@gcc.gnu.org X-SW-Source: 2009-03/txt/msg00004.txt.bz2 Maven2 doesn't build at the moment because our code to check the loading constraints (Section 5.3.4, JVM Spec Second Edition) is wrong. Tom, is this the correct fix? Andrew. 2009-03-11 Tom Tromey * link.cc (_Jv_Linker::find_field): Pass the field_type to _Jv_CheckOrCreateLoadingConstraint, not the class that is requesting the field. --- link.cc~ 2009-01-14 11:10:39.000000000 +0000 +++ link.cc 2009-03-11 17:33:31.000000000 +0000 @@ -248,7 +248,7 @@ _Jv_Linker::find_field (jclass klass, jc // Note that the field returned by find_field_helper is always // resolved. However, we still use the constraint mechanism // because this may affect other lookups. - _Jv_CheckOrCreateLoadingConstraint (klass, (*found_class)->loader); + _Jv_CheckOrCreateLoadingConstraint (field_type, (*found_class)->loader); } else {