From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 125873 invoked by alias); 6 Aug 2018 11:55:57 -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 125860 invoked by uid 89); 6 Aug 2018 11:55:54 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-19.8 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_2,GIT_PATCH_3,KAM_LAZY_DOMAIN_SECURITY autolearn=ham version=3.3.2 spammy=Tested, dist X-HELO: smtp.CeBiTec.Uni-Bielefeld.DE Received: from smtp.CeBiTec.Uni-Bielefeld.DE (HELO smtp.CeBiTec.Uni-Bielefeld.DE) (129.70.160.84) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 06 Aug 2018 11:55:52 +0000 Received: from localhost (localhost.CeBiTec.Uni-Bielefeld.DE [127.0.0.1]) by smtp.CeBiTec.Uni-Bielefeld.DE (Postfix) with ESMTP id F24A362C for ; Mon, 6 Aug 2018 13:55:49 +0200 (CEST) Received: from smtp.CeBiTec.Uni-Bielefeld.DE ([127.0.0.1]) by localhost (malfoy.CeBiTec.Uni-Bielefeld.DE [127.0.0.1]) (amavisd-new, port 10024) with LMTP id j0cSTROplFzy for ; Mon, 6 Aug 2018 13:55:47 +0200 (CEST) Received: from lokon.CeBiTec.Uni-Bielefeld.DE (lokon.CeBiTec.Uni-Bielefeld.DE [129.70.161.152]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.CeBiTec.Uni-Bielefeld.DE (Postfix) with ESMTPS id 87AC562B for ; Mon, 6 Aug 2018 13:55:47 +0200 (CEST) Received: (from ro@localhost) by lokon.CeBiTec.Uni-Bielefeld.DE (8.15.2+Sun/8.15.2/Submit) id w76Btkm6018487; Mon, 6 Aug 2018 13:55:47 +0200 (MEST) From: Rainer Orth To: gdb-patches@sourceware.org Subject: Remove unused variables in procfs.c etc. Date: Mon, 06 Aug 2018 11:55:00 -0000 Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (usg-unix-v) MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-SW-Source: 2018-08/txt/msg00081.txt.bz2 --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Content-length: 1704 The recent patch to enable -Wunused-variable uncovered a couple of instances in Solaris-specific files: /vol/src/gnu/gdb/hg/master/dist/gdb/proc-api.c: In function =E2=80=98void _= initialize_proc_api()=E2=80=99: /vol/src/gnu/gdb/hg/master/dist/gdb/proc-api.c:422:28: error: unused variab= le =E2=80=98c=E2=80=99 [-Werror=3Dunused-variable] struct cmd_list_element *c; ^ /vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c: In function =E2=80=98void pro= cfs_init_inferior(target_ops*, int)=E2=80=99: /vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c:2895:12: error: unused variabl= e =E2=80=98signals=E2=80=99 [-Werror=3Dunused-variable] sigset_t signals; ^~~~~~~ /vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c: In member function =E2=80=98v= irtual char* procfs_target::make_corefile_notes(bfd*, int*)=E2=80=99: /vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c:3845:19: error: unused variabl= e =E2=80=98old_chain=E2=80=99 [-Werror=3Dunused-variable] struct cleanup *old_chain; ^~~~~~~~~ /vol/src/gnu/gdb/hg/master/dist/gdb/procfs.c:3847:18: error: unused variabl= e =E2=80=98fpregs=E2=80=99 [-Werror=3Dunused-variable] gdb_fpregset_t fpregs; ^~~~~~ Fixed as follows. Tested on amd64-pc-solaris2.11, installed as obvious. Rainer --=20 ---------------------------------------------------------------------------= -- Rainer Orth, Center for Biotechnology, Bielefeld University 2018-08-03 Rainer Orth * proc-api.c (_initialize_proc_api): Remove c, unused. * procfs.c (procfs_init_inferior): Remove signals, unused. (procfs_target::make_corefile_notes): Remove old_chain, fpregs, unused. --=-=-= Content-Type: text/x-patch Content-Disposition: inline; filename=sol2-unused-vars.patch Content-length: 1056 # HG changeset patch # Parent 84125362cdb6fb98ee6e31d4d3bbc1e8214e21bc Remove unused variables in procfs.c etc. diff --git a/gdb/proc-api.c b/gdb/proc-api.c --- a/gdb/proc-api.c +++ b/gdb/proc-api.c @@ -419,8 +419,6 @@ proc_prettyfprint_status (long flags, in void _initialize_proc_api (void) { - struct cmd_list_element *c; - add_setshow_boolean_cmd ("procfs-trace", no_class, &procfs_trace, _("\ Set tracing for /proc api calls."), _("\ Show tracing for /proc api calls."), NULL, diff --git a/gdb/procfs.c b/gdb/procfs.c --- a/gdb/procfs.c +++ b/gdb/procfs.c @@ -2892,7 +2892,6 @@ static void procfs_init_inferior (struct target_ops *ops, int pid) { procinfo *pi; - sigset_t signals; int fail; int lwpid; @@ -3842,9 +3841,7 @@ find_stop_signal (void) char * procfs_target::make_corefile_notes (bfd *obfd, int *note_size) { - struct cleanup *old_chain; gdb_gregset_t gregs; - gdb_fpregset_t fpregs; char fname[16] = {'\0'}; char psargs[80] = {'\0'}; procinfo *pi = find_procinfo_or_die (inferior_ptid.pid (), 0); --=-=-=--