From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 12942 invoked by alias); 25 May 2009 20:56:10 -0000 Received: (qmail 12932 invoked by uid 22791); 25 May 2009 20:56:09 -0000 X-SWARE-Spam-Status: No, hits=-1.4 required=5.0 tests=AWL,BAYES_00,J_CHICKENPOX_84,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout2.012.net.il (HELO mtaout2.012.net.il) (84.95.2.4) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 25 May 2009 20:56:04 +0000 Received: from conversion-daemon.i_mtaout2.012.net.il by i_mtaout2.012.net.il (HyperSendmail v2004.12) id <0KK700200WQSBV00@i_mtaout2.012.net.il> for gdb-patches@sourceware.org; Mon, 25 May 2009 23:56:01 +0300 (IDT) Received: from HOME-C4E4A596F7 ([84.228.115.215]) by i_mtaout2.012.net.il (HyperSendmail v2004.12) with ESMTPA id <0KK7006AWWTCY920@i_mtaout2.012.net.il>; Mon, 25 May 2009 23:56:01 +0300 (IDT) Date: Mon, 25 May 2009 20:56:00 -0000 From: Eli Zaretskii Subject: Re: [patch 6/8] Types GC [the GC itself] In-reply-to: <20090525080323.GF13323@host0.dyn.jankratochvil.net> To: Jan Kratochvil Cc: gdb-patches@sourceware.org Reply-to: Eli Zaretskii Message-id: <83tz38sxzx.fsf@gnu.org> References: <20090525080323.GF13323@host0.dyn.jankratochvil.net> 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: 2009-05/txt/msg00567.txt.bz2 > Date: Mon, 25 May 2009 10:03:23 +0200 > From: Jan Kratochvil > > gdb/doc/ > 2009-05-25 Jan Kratochvil > > * gdbint.texinfo (Symbol Handling): New menu. > (Partial Symbol Tables, Object File Formats, Debugging File Formats) > (Adding a New Symbol Reader to GDB): New nodes from existing sections. > (Types): New node from existing section. New anchor `Builtin Types'. > (Memory Management for Symbol Files): New node from existing section. > Move types to ... > (Memory Management for Types): ... here, in a new node. > * observer.texi (mark_used): New. Thanks. I have a few comments about this part. > +@code{TYPE_OBJFILE} macro indicates the current memory owner of the type. "The @code{TYPE_OBJFILE} macro ..." > +Non-@code{NULL} value indicates it is owned by an objfile (specifically by its "A non-@code{NULL} value indicates ..." > +User convenience variables, et cetera, have associated types. Normally these Can you spell out what is behind the "et cetera" here? If the full list is too long, naming just the most frequently used ones would be good enough. > +Types with null @code{TYPE_OBJFILE} can be either permanent types > +(@pxref{Builtin Types}) or reclaimable types which will be deallocated at the > +first idle @value{GDBN} moment if the last object referencing them is removed . I think "the first idle GDB moment after the last object ... is removed" is more accurate. "After" instead of "if". > +Permanent types are allocated by the function @code{alloc_type} (and its > +derivations like @code{init_type}) specifying objfile as @code{NULL}. The > +reclaimable types are created the same way but moreover they need to have ^^^^^^^^ I think "in addition" is better here. > +register handler touching all the still used types by @code{type_mark_used}. "to register a handler that will touch all the types still in use with a call to @code{type_mark_used}." OK with those changes.