From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13859 invoked by alias); 26 Mar 2008 15:41:34 -0000 Received: (qmail 13844 invoked by uid 22791); 26 Mar 2008 15:41:33 -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, 26 Mar 2008 15:40:59 +0000 Received: from list by ciao.gmane.org with local (Exim 4.43) id 1JeXkC-0005bT-FI for gdb-patches@sources.redhat.com; Wed, 26 Mar 2008 15:40:52 +0000 Received: from 78.158.192.230 ([78.158.192.230]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Wed, 26 Mar 2008 15:40:52 +0000 Received: from ghost by 78.158.192.230 with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Wed, 26 Mar 2008 15:40:52 +0000 To: gdb-patches@sources.redhat.com From: Vladimir Prus Subject: Re: Cleanup MI testsuite Date: Wed, 26 Mar 2008 15:41:00 -0000 Message-ID: References: <200803261629.53454.vladimir@codesourcery.com> <200803261511.57740.pedro@codesourcery.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit User-Agent: KNode/0.10.5 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: 2008-03/txt/msg00401.txt.bz2 Pedro Alves wrote: > A Wednesday 26 March 2008 13:29:52, Vladimir Prus wrote: >> When I've checked in the patch to implement per-thread varobjs, I've >> implicitly assumed that MI output does not change in single-threaded >> programs, so no testsuite update is necessary. However, that's not >> necessary true, and with Pedro's recent patch, is not true on x86. > > My patch affected every linux native target, independently > of arch. Were you testing on Cygwin instead for example, when you > get to main, there's already 2 threads listed in info threads. The > main thread and another one internal to cygwin1.dll. Yes, that why I said "not necessary true" -- your patch just exposed the problem that was already there. - Volodya