[C++] Unreviewed patches
Jason Merrill
jason_merrill@redhat.com
Sat Oct 13 21:28:00 GMT 2001
>>>>> "Kriang" == Kriang Lerdsuwanakij <lerdsuwa@users.sourceforge.net> writes:
> Jason Merrill wrote:
>>
>> > PR3391
>> > http://gcc.gnu.org/ml/gcc-patches/2001-09/msg00898.html
>>
>> OK. Would it make sense to do this for TYPENAME_TYPEs as well?
> We can define a class like
> template <class T> class C : public T::N { };
> According to the standard, "T::N" should not be prefixed by
> "typename" here. But a TYPENAME_TYPE is created for the base
> class (in uniform with other typename cases). So TYPE_BINFO
> of TYPENAME_TYPE may also be used.
Yes. So will you make the same change for TYPENAME_TYPE?
>> > Template parm dependent template template arg
>> > http://gcc.gnu.org/ml/gcc-patches/2001-09/msg00664.html
> There is a difference between the case in this patch and the one in your
> example. The patch is intended to deal with this construct WITHOUT its
> template argument. Like "C<T::template N>". This can only appear as
> template argument.
Good point.
>> Also, the construct isn't just an identifier, it names a class template, so
>> the name should suggest such. Perhaps UNBOUND_CLASS_TEMPLATE?
> OK I'll adjust its name. A revised version will be submitted
> later.
If you're just changing the name, no need to submit it again.
Thanks,
Jason
More information about the Gcc-patches
mailing list