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: remove dependency on cp/parser.h from cp/lang.c

From: Aaron Gray <aaronngray dot lists at gmail dot com>
To: gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Wed, 29 Aug 2012 20:54:56 +0100
Subject: Re: remove dependency on cp/parser.h from cp/lang.c
References: <CANkmNDcUxTK4aqaihfc8x7nkEpJeF_SYOdO2rQ6zjgvDrWL_gQ@mail.gmail.com> <alpine.DEB.2.02.1208292042060.3307@laptop-mg.saclay.inria.fr>

On 29 August 2012 19:47, Marc Glisse <marc.glisse@inria.fr> wrote:
>
> On Wed, 29 Aug 2012, Aaron Gray wrote:
>
>> Just got my copyright assignment through, so here's my first GCC patch,
>
>
> Welcome!

Thanks Marc !

>
>
>
>> This is a one liner removing the unneeded dependency of cp-lang.c on
>> cp/parser.h. This has been tested on Linux.
>
>
> I think you need to attach a ChangeLog entry with every patch.

Okay

>
>
> Also, dependencies are often repeated in makefiles. Is there anything to update with your patch? (maybe not, just asking)

Yes there is a dependency in cp/Make-lang.in, I will resubmit the patch soon.

--
Aaron

References:
- remove dependency on cp/parser.h from cp/lang.c
  - From: Aaron Gray
- Re: remove dependency on cp/parser.h from cp/lang.c
  - From: Marc Glisse

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