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: [PATCH] Changing "Class" in testsuite


A couple more picky things about ChangeLog entries: there must be two
spaces between the date and your name, and two spaces between your name
and email address. Each line in the list of changes starts with a tab,
not spaces.


Your mailer wrapped some of the lines in the patch, so whoever applies
this will need to watch out for that.

A patch submission should say where you tested it. Assuming you tested
this with a C++ compiler, this is OK with fixes to the ChangeLog.


This is good to know. It's my first patch submission to GCC. Here's the revised patch. I've included the patch file as an attachment. It has the correct spacing issues in emacs, at least. I tested this on an iMac here at Apple. The g++ tests that are patched pass just fine. :-)

-bw

2006-10-12 Bill Wendling <wendling@apple.com>

       * g++.old-deja/g++.brendan/scope5.C,
       g++.old-deja/g++.jason/hmc1.C,
       g++.old-deja/g++.jason/cleanup2.C,
       g++.old-deja/g++.law/shadow1.C,
       g++.old-deja/g++.law/ctors10.C,
       g++.dg/debug/namespace1.C,
       g++.dg/inherit/namespace-as-base.C,
       g++.dg/opt/delay-slot-1.C,
       g++.dg/parse/defarg10.C,
       g++.dg/template/ptrmem11.C,
       g++.dg/template/mem_func_ptr.C,
       g++.dg/template/spec19.C:
       Changed "Class" to "Klasse".


Attachment: patch.txt
Description: Text document



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