From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 50950 invoked by alias); 1 Dec 2018 16:26:58 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 50936 invoked by uid 89); 1 Dec 2018 16:26:58 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_HELO_PASS autolearn=ham version=3.3.2 spammy=learn, day X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sat, 01 Dec 2018 16:26:57 +0000 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id F179236809 for ; Sat, 1 Dec 2018 16:26:55 +0000 (UTC) Received: from redhat.com (ovpn-125-91.rdu2.redhat.com [10.10.125.91]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 22827600C5; Sat, 1 Dec 2018 16:26:54 +0000 (UTC) Date: Sat, 01 Dec 2018 16:26:00 -0000 From: Marek Polacek To: Jason Merrill Cc: GCC Patches Subject: Re: C++ PATCH to implement C++20 P0634R3, Down with typename! Message-ID: <20181201162702.GB21364@redhat.com> References: <20181112152731.GU9403@redhat.com> <7aa971d8-b832-07ff-f58e-4c01b1d60db8@redhat.com> <20181130230808.GA21364@redhat.com> <47555d33-c4b5-fd08-62a0-1c1aebceba8d@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <47555d33-c4b5-fd08-62a0-1c1aebceba8d@redhat.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-SW-Source: 2018-12/txt/msg00008.txt.bz2 On Sat, Dec 01, 2018 at 10:57:12AM -0500, Jason Merrill wrote: > On 11/30/18 6:08 PM, Marek Polacek wrote: > > + TYPENAME_OPTIONAL_P is true if the typename keyword is optional in this > > + context. */ > ... > > + TYPENAME_OPTIONAL_P is true if the typename keyword is optional in > > + this context. > > You have a couple of these left over from the previous patch. OK with that > fixed. One day I'll learn how to use grep. Thanks for the reviews! Marek