From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31795 invoked by alias); 12 Jun 2003 01:30:04 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 31776 invoked from network); 12 Jun 2003 01:30:03 -0000 Received: from unknown (HELO crack.them.org) (146.82.138.56) by sources.redhat.com with SMTP; 12 Jun 2003 01:30:03 -0000 Received: from dsl093-172-017.pit1.dsl.speakeasy.net ([66.93.172.17] helo=nevyn.them.org ident=mail) by crack.them.org with asmtp (Exim 3.12 #1 (Debian)) id 19QGvQ-0006ZG-00 for ; Wed, 11 Jun 2003 20:30:48 -0500 Received: from drow by nevyn.them.org with local (Exim 3.36 #1 (Debian)) id 19QGud-0005dn-00 for ; Wed, 11 Jun 2003 21:29:59 -0400 Date: Thu, 12 Jun 2003 01:30:00 -0000 From: Daniel Jacobowitz To: gdb-patches@sources.redhat.com Subject: Re: [rfa] move make_symbol_overload_list to cp-support.c Message-ID: <20030612012958.GB21583@nevyn.them.org> Mail-Followup-To: gdb-patches@sources.redhat.com References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.1i X-SW-Source: 2003-06/txt/msg00407.txt.bz2 On Wed, Jun 11, 2003 at 05:24:42PM -0700, David Carlton wrote: > As previously discussed, here's a patch to move > make_symbol_overload_list and everything associated with it to > cp-support.c. It should be pretty obvious, and Daniel and Elena have > both agreed in principle, so I'll commit this once one of them gives a > thumbs-up. (I'm pretty sure I'm not missing anything - I just moved > code, so it's hard to imagine how anything could have gone wrong.) Looks good. I'm not thrilled with externing lookup_partial_symbol but it'll do for a while. > David Carlton > carlton@bactrian.org > > 2003-06-11 David Carlton > > * symtab.h: Delete declaration of make_symbol_overload_list. > Add declaration of lookup_partial_symbol. > * symtab.c (remove_params): Move to cp-support.c. > (overload_list_add_symbol, make_symbol_overload_list) > (sym_return_val_size, sym_return_val_index): Ditto. > symtab.c (lookup_partial_symbol): Make extern. > * cp-support.h: Add declaration of make_symbol_overload_list. > * cp-support.c: Include dictionary.h, objfiles.h, frame.h, > symtab.h, and block.h. > (remove_params): Move here from symtab.c. > (overload_list_add_symbol, make_symbol_overload_list) > (sym_return_val_size, sym_return_val_index): Ditto. > * valops.c: Include cp-support.h. > * Makefile.in (cp-support.o): Depend on dictionary_h, objfiles_h, > frame_h, and block_h. > (valops.o): Depend on cp_support_h. -- Daniel Jacobowitz MontaVista Software Debian GNU/Linux Developer