From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 82214 invoked by alias); 30 Nov 2016 20:25:45 -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 82201 invoked by uid 89); 30 Nov 2016 20:25:45 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_PASS,URIBL_RED autolearn=ham version=3.3.2 spammy=technology X-HELO: relay1.mentorg.com Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 30 Nov 2016 20:25:35 +0000 Received: from svr-orw-mbx-03.mgc.mentorg.com ([147.34.90.203]) by relay1.mentorg.com with esmtp id 1cCBRc-0003pI-8Z from Luis_Gustavo@mentor.com ; Wed, 30 Nov 2016 12:25:32 -0800 Received: from [172.30.5.15] (147.34.91.1) by svr-orw-mbx-03.mgc.mentorg.com (147.34.90.203) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Wed, 30 Nov 2016 12:25:28 -0800 Reply-To: Luis Machado Subject: Re: [PATCH v2 5/8] Fix test names starting with uppercase using multi-line gdb_test/mi_gdb_test References: <1480107244-1484-1-git-send-email-lgustavo@codesourcery.com> <1480107244-1484-6-git-send-email-lgustavo@codesourcery.com> <17bcdc1f-8f2c-92af-1b76-c0558c22b2c2@redhat.com> To: Pedro Alves , From: Luis Machado Message-ID: Date: Wed, 30 Nov 2016 20:25:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <17bcdc1f-8f2c-92af-1b76-c0558c22b2c2@redhat.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: svr-orw-mbx-03.mgc.mentorg.com (147.34.90.203) To svr-orw-mbx-03.mgc.mentorg.com (147.34.90.203) X-IsSubscribed: yes X-SW-Source: 2016-11/txt/msg01017.txt.bz2 On 11/30/2016 02:11 PM, Pedro Alves wrote: > On 11/25/2016 08:54 PM, Luis Machado wrote: >> This fixes offender testcases that have test names starting with uppercase >> when using gdb_test/mi_gdb_test in a multi-line construct. >> >> gdb/testsuite/ChangeLog >> 2016-11-25 Luis Machado >> >> Fix test names starting with uppercase throughout the files. >> >> * gdb/testsuite/gdb.ada/array_return.exp > > Drop gdb/testsuite/. Otherwise LGTM. > Thanks. This is fixed locally throughout. >> diff --git a/gdb/testsuite/gdb.ada/array_return.exp b/gdb/testsuite/gdb.ada/array_return.exp >> index f4dae48..37f2845 100644 >> --- a/gdb/testsuite/gdb.ada/array_return.exp >> +++ b/gdb/testsuite/gdb.ada/array_return.exp >> @@ -52,7 +52,7 @@ gdb_test "break create_small_float_vector" \ >> >> gdb_test "cont" \ >> "Breakpoint \[0-9\]+, pck.create_small \\(\\).*" \ >> - "Continuing to Create_Small" >> + "continuing to Create_Small" > > This looks like a good example showing why enforcing uppercase > in the testsuite's internal routines would be going too far. > I.e., it's quite reasonable to want to print the name of some random > symbol from the test's source code which might want to be uppercase. > Yeah, given the current test names we have, it doesn't make much sense. It would only make sense if had patterns for test names that did not include variables, technology names and the like. Not sure if that's sensible or not at this point. But, with the series in place, i think we are more coherent with regards to test names.