[patch] some mindless additions of BLOCK_ macros
Andrew Cagney
ac131313@redhat.com
Wed Oct 23 16:04:00 GMT 2002
> I recently noticed that the BLOCK_ macros weren't used everywhere they
> could be. I know Andrew doesn't like macros, but given that these
> ones are used almost everywhere, they might as well be used
> everywhere.
Yep.
It's more that I like opaque types - it is all about `control' - with an
opaque type it simply isn't possible to sneak in [old] code that grubs
around in the internals. You could consider block.[hc]?
(I need to come clean and admit that I'm quietly moving all the frame
specific stuff out of blockframe.c and into frame.c so it can be opaque.
I suspect everone guessed this long ago, though :-)
> I've tested the parse.c part of this by doing a make check, the
> x86-64-tdep.c part by doing an all targets build and noticing that it
> still compiles, and the objc-lang.c part hasn't been tested at all.
> They're all trivial changes, so that amount of testing seems to me to
> be sufficient.
>
> This patch seems obvious to me; if nobody complains, I'll commit it in
> a day or two.
I think its safe for today.
Andrew
More information about the Gdb-patches
mailing list