From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 125906 invoked by alias); 27 Dec 2018 02:36:14 -0000 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 Received: (qmail 125895 invoked by uid 89); 27 Dec 2018 02:36:13 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-6.9 required=5.0 tests=BAYES_00,GIT_PATCH_2,RCVD_IN_DNSWL_NONE,SPF_PASS,TIME_LIMIT_EXCEEDED autolearn=unavailable version=3.3.2 spammy=HContent-Transfer-Encoding:8bit X-HELO: rock.gnat.com Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 27 Dec 2018 02:36:03 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id C56A211731A; Wed, 26 Dec 2018 21:36:01 -0500 (EST) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id 3oN144RR5UJR; Wed, 26 Dec 2018 21:36:01 -0500 (EST) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 1F1A31172AB; Wed, 26 Dec 2018 21:35:59 -0500 (EST) Received: by joel.gnat.com (Postfix, from userid 1000) id 8EEDF86753; Thu, 27 Dec 2018 06:35:52 +0400 (+04) Date: Thu, 27 Dec 2018 02:36:00 -0000 From: Joel Brobecker To: Philippe Waroquiers Cc: gdb-patches@sourceware.org Subject: Re: [RFAv2 2/4] Fix gdb.ada/rename_subscript_param.exp by using more unique names. Message-ID: <20181227023552.GF21851@adacore.com> References: <20181226140304.19594-1-philippe.waroquiers@skynet.be> <20181226140304.19594-3-philippe.waroquiers@skynet.be> <20181226152743.GD21851@adacore.com> <1545839429.31031.15.camel@skynet.be> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1545839429.31031.15.camel@skynet.be> User-Agent: Mutt/1.9.4 (2018-02-28) X-SW-Source: 2018-12/txt/msg00349.txt.bz2 > What about this new commit message ? Loks very good. Thanks Philippe! > > -------------------------------------------------- > Improve gdb.ada/rename_subscript_param.exp by using more unique names. > > With old compilers, the test fails because no debug info is generated > for 'B' and GDB finds some 'b' in atnat.h: > > (gdb) print b > Multiple matches for b > [0] cancel > [1] b at ../sysdeps/ieee754/dbl-64/atnat.h:106 > [2] b at ../sysdeps/ieee754/dbl-64/atnat.h:106 > [3] b at ../sysdeps/ieee754/dbl-64/atnat.h:106 > > FAIL: gdb.ada/rename_subscript_param.exp: print b before changing its value (timeout) > > Avoid the timeout by renaming 'b' to rename_subscript_param_b. > > Also, change 'before' to 'after' in the gdb_test message that prints > the value after changing it. > > The test still fails with old compilers that do not properly > generate debug info for this renaming: > (gdb) print rename_subscript_param_b > No definition of "rename_subscript_param_b" in current context. > (gdb) FAIL: gdb.ada/rename_subscript_param.exp: print rename_subscript_param_b before changing its value > > Note: if the compiler would generate the correct debug info, the test should > succeed with the name B.  However, waiting for this fix, changing the name > ensures that the test fails directly, instead of causing a timeout. > > 2018-12-26  Philippe Waroquiers   > > PR ada/23381 > * gdb.ada/rename_subscript_param/pkg.adb (B): Rename to > Rename_Subscript_Param_B.  All users updated. > gdb.ada/rename_subscript_param.exp: Test names made unique. > Note that PR ada/23381 is only fully fixed when using a recent > compiler. > > > > > > Also, change 'before' to 'after' in the gdb_test message that prints > > > the value after changing it. > > > The test however still fails for me, probably because the Debian stable > > > Ada compiler is too old, and does not properly generate debug info for > > > this renamining: > > > (gdb) print rename_subscript_param_b > > > No definition of "rename_subscript_param_b" in current context. > > > (gdb) FAIL: gdb.ada/rename_subscript_param.exp: print rename_subscript_param_b before changing its value > > > > > > Note: if the compiler would generate the correct debug info, the test should > > > succeed with the name B. However, waiting for this fix, changing the name > > > ensures that the test fails directly, instead of causing a timeout. > > > > > > 2018-12-26 Philippe Waroquiers > > > > > > * gdb.ada/rename_subscript_param/pkg.adb (B): Rename to > > > Rename_Subscript_Param_B. All users updated. > > > > The patch itself looks good, though. > > -- Joel