From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [205.139.110.61]) by sourceware.org (Postfix) with ESMTP id F00E6395446E for ; Tue, 23 Jun 2020 11:26:47 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org F00E6395446E Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-486-iCWM8ORYNr-vvbMeODNbZg-1; Tue, 23 Jun 2020 07:26:45 -0400 X-MC-Unique: iCWM8ORYNr-vvbMeODNbZg-1 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 28555107ACF6 for ; Tue, 23 Jun 2020 11:26:45 +0000 (UTC) Received: from blade.nx (ovpn-112-118.ams2.redhat.com [10.36.112.118]) by smtp.corp.redhat.com (Postfix) with ESMTP id C5C295C290 for ; Tue, 23 Jun 2020 11:26:44 +0000 (UTC) Received: from blade.com (localhost [127.0.0.1]) by blade.nx (Postfix) with ESMTP id DDADF816CCA9 for ; Tue, 23 Jun 2020 12:26:43 +0100 (BST) From: Gary Benson To: gdb-patches@sourceware.org Subject: [OB PATCH] Avoid testcase build failures with -Wunused-value Date: Tue, 23 Jun 2020 12:26:42 +0100 Message-Id: <1592911602-4914-1-git-send-email-gbenson@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=US-ASCII X-Spam-Status: No, score=-15.5 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 23 Jun 2020 11:26:50 -0000 A number of testcases fail to build with -Wunused-value enabled. This commit adds dummy values to avoid this. gdb/testsuite/ChangeLog: * gdb.cp/namespace.cc: Avoid build failure with -Wunused-value. * gdb.cp/nsimport.cc: Likewise. * gdb.cp/nsnested.cc: Likewise. * gdb.cp/nsnoimports.cc: Likewise. * gdb.cp/nsusing.cc: Likewise. * gdb.cp/smartp.cc: Likewise. * gdb.python/py-pp-integral.c: Likewise. * gdb.python/py-pp-re-notag.c: Likewise. --- gdb/testsuite/ChangeLog | 11 +++++++++++ gdb/testsuite/gdb.cp/namespace.cc | 22 +++++++++++----------- gdb/testsuite/gdb.cp/nsimport.cc | 6 +++--- gdb/testsuite/gdb.cp/nsnested.cc | 2 +- gdb/testsuite/gdb.cp/nsnoimports.cc | 12 ++++++------ gdb/testsuite/gdb.cp/nsusing.cc | 12 ++++++------ gdb/testsuite/gdb.cp/smartp.cc | 8 ++++---- gdb/testsuite/gdb.python/py-pp-integral.c | 4 ++-- gdb/testsuite/gdb.python/py-pp-re-notag.c | 4 ++-- 9 files changed, 46 insertions(+), 35 deletions(-) diff --git a/gdb/testsuite/gdb.cp/namespace.cc b/gdb/testsuite/gdb.cp/namespace.cc index f918b63..1ff3426 100644 --- a/gdb/testsuite/gdb.cp/namespace.cc +++ b/gdb/testsuite/gdb.cp/namespace.cc @@ -150,22 +150,22 @@ namespace C // plan to have GDB try to print out, just to make sure that the // compiler and I agree which ones should be legal! It's easy // to screw up when testing the boundaries of namespace stuff. - c; + int unused1 = c; //cc; - C::cc; - cd; + int unused2 = C::cc; + int unused3 = cd; //C::D::cd; - E::cde; - shadow; + int unused4 = E::cde; + int unused5 = shadow; //E::ce; - cX; - F::cXf; - F::cXfX; - X; - G::Xg; + int unused6 = cX; + int unused7 = F::cXf; + int unused8 = F::cXfX; + int unused9 = X; + int unusedA = G::Xg; //cXOtherFile; //XOtherFile; - G::XgX; + int unusedB = G::XgX; return; } diff --git a/gdb/testsuite/gdb.cp/nsimport.cc b/gdb/testsuite/gdb.cp/nsimport.cc index 6b180d6..5fc57b0 100644 --- a/gdb/testsuite/gdb.cp/nsimport.cc +++ b/gdb/testsuite/gdb.cp/nsimport.cc @@ -13,8 +13,8 @@ namespace{ int main() { - x; - xx; - xxx; + int unused1 = x; + int unused2 = xx; + int unused3 = xxx; return 0; } diff --git a/gdb/testsuite/gdb.cp/nsnested.cc b/gdb/testsuite/gdb.cp/nsnested.cc index 9723f87..fc3e11f 100644 --- a/gdb/testsuite/gdb.cp/nsnested.cc +++ b/gdb/testsuite/gdb.cp/nsnested.cc @@ -15,7 +15,7 @@ namespace C int second() { - ab; + int unused = ab; return 0; } } diff --git a/gdb/testsuite/gdb.cp/nsnoimports.cc b/gdb/testsuite/gdb.cp/nsnoimports.cc index d1c68ab..9968c35 100644 --- a/gdb/testsuite/gdb.cp/nsnoimports.cc +++ b/gdb/testsuite/gdb.cp/nsnoimports.cc @@ -18,9 +18,9 @@ namespace A } int first(){ - _a; - ab; - C::abc; + int unused1 = _a; + int unused2 = ab; + int unused3 = C::abc; return C::second(); } } @@ -30,8 +30,8 @@ namespace A int main() { - A::_a; - A::B::ab; - A::B::C::abc; + int unused1 = A::_a; + int unused2 = A::B::ab; + int unused3 = A::B::C::abc; return A::B::first(); } diff --git a/gdb/testsuite/gdb.cp/nsusing.cc b/gdb/testsuite/gdb.cp/nsusing.cc index 72ff941..980a91a 100644 --- a/gdb/testsuite/gdb.cp/nsusing.cc +++ b/gdb/testsuite/gdb.cp/nsusing.cc @@ -35,7 +35,7 @@ namespace L using namespace J; int marker8 () { - jx; + int unused = jx; return K::marker9 (); } } @@ -53,7 +53,7 @@ namespace I int marker7 () { using namespace G::H; - ghx; + int unused = ghx; return L::marker8 (); } } @@ -69,7 +69,7 @@ namespace E using namespace E::F; int marker6 () { - efx; + int unused = efx; return I::marker7 (); } @@ -92,7 +92,7 @@ namespace D using namespace C; int marker5 () { - cc; + int unused = cc; return marker6 (); } @@ -110,7 +110,7 @@ int marker3 () int marker2 () { namespace B = A; - B::_a; + int unused = B::_a; return marker3 (); } @@ -134,6 +134,6 @@ int marker1 () int main () { using namespace A; - _a; + int unused = _a; return marker1 (); } diff --git a/gdb/testsuite/gdb.cp/smartp.cc b/gdb/testsuite/gdb.cp/smartp.cc index 2e71d1a..ed52102 100644 --- a/gdb/testsuite/gdb.cp/smartp.cc +++ b/gdb/testsuite/gdb.cp/smartp.cc @@ -131,12 +131,12 @@ int main(){ sp3->foo(1); sp3->foo('a'); - sp4->a; - sp4->b; + int unused1 = sp4->a; + int unused2 = sp4->b; Type4 *mt4p = &mt4; - mt4p->a; - mt4p->b; + int unused3 = mt4p->a; + int unused4 = mt4p->b; A a; B b; diff --git a/gdb/testsuite/gdb.python/py-pp-integral.c b/gdb/testsuite/gdb.python/py-pp-integral.c index 5cefc00..eadb466 100644 --- a/gdb/testsuite/gdb.python/py-pp-integral.c +++ b/gdb/testsuite/gdb.python/py-pp-integral.c @@ -17,10 +17,10 @@ typedef long time_t; -static void +static time_t tick_tock (time_t *t) { - *t++; + return *t++; } int diff --git a/gdb/testsuite/gdb.python/py-pp-re-notag.c b/gdb/testsuite/gdb.python/py-pp-re-notag.c index 5cefc00..eadb466 100644 --- a/gdb/testsuite/gdb.python/py-pp-re-notag.c +++ b/gdb/testsuite/gdb.python/py-pp-re-notag.c @@ -17,10 +17,10 @@ typedef long time_t; -static void +static time_t tick_tock (time_t *t) { - *t++; + return *t++; } int -- 1.8.3.1