From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31070 invoked by alias); 6 Dec 2006 20:33:31 -0000 Received: (qmail 31058 invoked by uid 22791); 6 Dec 2006 20:33:29 -0000 X-Spam-Check-By: sourceware.org Received: from main.gmane.org (HELO ciao.gmane.org) (80.91.229.2) by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 06 Dec 2006 20:33:21 +0000 Received: from list by ciao.gmane.org with local (Exim 4.43) id 1Gs3SA-0002vk-7S for gdb-patches@sources.redhat.com; Wed, 06 Dec 2006 21:33:18 +0100 Received: from 73-198.umostel.ru ([82.179.73.198]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Wed, 06 Dec 2006 21:33:18 +0100 Received: from ghost by 73-198.umostel.ru with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Wed, 06 Dec 2006 21:33:18 +0100 To: gdb-patches@sources.redhat.com From: Vladimir Prus Subject: Re: MI: fix base members in references Date: Wed, 06 Dec 2006 20:33:00 -0000 Message-ID: References: <200611291555.42209.ghost@cs.msu.su> <20061205204447.GC25572@nevyn.them.org> <20061206203016.GA30520@adacore.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit User-Agent: KNode/0.10.2 X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-12/txt/msg00077.txt.bz2 Joel Brobecker wrote: > Volodya, > >> >> If this patch is fine, can I also commit it to 6.6 branch? The bug in >> >> question is quite problematic for C++ code. >> > >> > Sounds fine to me, but check with Joel. >> >> I'll check. > > This patch: > >> 2006-12-06 Vladimir Prus >> >> * varobj.c (cplus_value_of_child): When accessing >> base suboject, don't specially process references. > > is OK for the branch. Thanks! > Do we have a testcase for that, btw? We don't. As I've mentioned, we've short of MI/C++ testcases. My other references patch adds MI/C++/varobj testcase, after committing that patch I'll also add tests for base/references problem. - Volodya