From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25838 invoked by alias); 11 Apr 2007 01:16:41 -0000 Received: (qmail 25830 invoked by uid 22791); 11 Apr 2007 01:16:40 -0000 X-Spam-Check-By: sourceware.org Received: from viper.snap.net.nz (HELO viper.snap.net.nz) (202.37.101.8) by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 11 Apr 2007 02:16:35 +0100 Received: from farnswood.snap.net.nz (105.63.255.123.dynamic.snap.net.nz [123.255.63.105]) by viper.snap.net.nz (Postfix) with ESMTP id 8A6DF3D9182; Wed, 11 Apr 2007 13:16:32 +1200 (NZST) Received: by farnswood.snap.net.nz (Postfix, from userid 500) id 784D5627ED; Wed, 11 Apr 2007 02:13:58 +0100 (BST) From: Nick Roberts MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <17948.13909.221294.650367@farnswood.snap.net.nz> Date: Wed, 11 Apr 2007 01:16:00 -0000 To: Daniel Jacobowitz Cc: Denis PILAT , gdb-patches Subject: Re: [RFC] -thread-info new command In-Reply-To: <20070410220434.GA8126@caradoc.them.org> References: <45FE9E6A.3030906@st.com> <17919.15500.439138.600411@kahikatea.snap.net.nz> <17919.20575.286260.761826@kahikatea.snap.net.nz> <20070320031409.GA7336@caradoc.them.org> <17919.21588.93918.661753@kahikatea.snap.net.nz> <17921.40699.742164.983281@kahikatea.snap.net.nz> <20070327195414.GN28164@caradoc.them.org> <17929.36288.344474.485310@farnswood.snap.net.nz> <20070410145346.GC30086@caradoc.them.org> <17948.1808.87473.108774@farnswood.snap.net.nz> <20070410220434.GA8126@caradoc.them.org> X-Mailer: VM 7.19 under Emacs 22.0.97.3 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: 2007-04/txt/msg00124.txt.bz2 > This is the bit I was talking about, since catch_excepions_with_msg > requires it be "int" and not "enum gdb_rc". > > > do_captured_list_thread_ids (struct ui_out *uiout, void *arg) > > { > > ... > > do_cleanups (cleanup_chain); > > ui_out_field_int (uiout, "number-of-threads", num); > > return GDB_RC_OK; > > } OK I see now. There still seems to be some kind of mismatch. Perhaps (sometime) enum return_reason and enum gdb_rc can be harmonised. >... > > It would also help stop bugs in front ends from being reported on the gdb > > mailing list. > > Maybe - not if the front end didn't handle it... Sure. If poeple elect not to use a feature, it won't provide any benefit. -- Nick http://www.inet.net.nz/~nickrob