[patch/rfc[ Opaque bcache

Andrew Cagney ac131313@ges.redhat.com
Fri Jul 12 08:48:00 GMT 2002


> On Wed, Jul 10, 2002 at 06:15:47PM -0400, Andrew Cagney wrote:
> 
> Hello,
> 
> This makes the bcache object opaque.  Testing so far hasn't shown problems.  I'll look to commit it in a few days.
> 
> comments?  Hmm, did I remember to update the makefile ....
> 
> 
> Nope to updating the Makefile, and nope to comments:
> 
> 
>  :-)
> 
> 
> +/* The type used to hold a single bcache string.  The user data is
> +   stored in d.data.  Since it can be any type, it needs to have the
> +   same alignment as the most strict alignment of any type on the host
> +   machine.  I don't know of any really correct way to do this in
> +   stock ANSI C, so just do it the same way obstack.h does.
> +
> +   It would be nicer to have this stuff hidden away in bcache.c, but
> +   struct objstack contains a struct bcache directly --- not a pointer
> +   to one --- and then the memory-mapped stuff makes this a real pain.
> +   We don't strictly need to expose struct bstring, but it's better to
> +   have it all in one place.  */
> 
> 
> That's not accurate any more.  Other than that, this looks great; and
> it removes a lot of &'s we used to need when accessing a bcache, which
> I quite like.
> 
> Hmm, yes, I should clean up the comments I simply cut/paste.

I've checked in the attached.  The differences are very minor.

Andrew

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: diffs
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20020712/1984c94b/attachment.ksh>


More information about the Gdb-patches mailing list