This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [C++ PATCH]: Fix 16174


Mark Mitchell wrote:
I think this patch is OK, but I want to think about it a little bit.

I think we should leave complete_type (with its current parameters) around as a macro or forwarder function; that way the vast majority of callers will not have to be changed, and new uses will not have to add the extra parameter. That would also make it easier for me to see the meat of the patch here.
Yeah, I pondered this too, but we already have 4 variants,
extern tree require_complete_type		(tree);
extern tree complete_type			(tree, tsubst_flags_t);
extern tree complete_type_or_diagnostic         (tree, tree, int);
#define complete_type_or_else(T,V) (complete_type_or_diagnostic ((T), (V), 0))

I didn't want to make it more confusing!

nathan
--
Nathan Sidwell    ::   http://www.codesourcery.com   ::     CodeSourcery LLC
nathan@codesourcery.com    ::     http://www.planetfall.pwp.blueyonder.co.uk



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]