This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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: Patch to remove the use of the c++ keyword class in binutil sources.

From: Dave Korn <dave dot korn dot cygwin at googlemail dot com>
To: Martin Thuresson <martin at mtme dot org>
Cc: binutils <binutils at sourceware dot org>
Date: Fri, 28 Aug 2009 21:23:23 +0100
Subject: Re: Patch to remove the use of the c++ keyword class in binutil sources.
References: <624836f10906242120t5eb61d92l538a44eb05c696b0@mail.gmail.com> <624836f10908271700m5cbff0dk3006acf94c506b5b@mail.gmail.com> <624836f10908281147p277bc533he7ef6a7cba10d725@mail.gmail.com>

Martin Thuresson wrote:
> Found a small issue with new.patch. Updated the patch again.
> 
> I also included a patch that renames all uses of the word template to
> avoid clashes with the C++ keyword.

  Sorry for the delay in getting to this.  I can only approve the COFF parts,
but I'll try to at least review the whole thing for you sometime tomorrow;
thanks for doing this work, and be assured you haven't gone un-noticed!

  Do you have a GNU copyright assignment on record?  I'm not 100% certain, but
I think the scale of the patch might require one, even though it's basically a
mechanical job.

    cheers,
      DaveK

Follow-Ups:
- Re: Patch to remove the use of the c++ keyword class in binutil sources.
  - From: Martin Thuresson

References:
- Re: Patch to remove the use of the c++ keyword class in binutil sources.
  - From: Martin Thuresson
- Re: Patch to remove the use of the c++ keyword class in binutil sources.
  - From: Martin Thuresson

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