This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH]: All unused functions and variables could be removed...
- To: cih at xlinux dot com
- Subject: Re: [PATCH]: All unused functions and variables could be removed...
- From: "Martin v. Loewis" <martin at loewis dot home dot cs dot tu-berlin dot de>
- Date: Wed, 5 Jan 2000 23:56:56 +0100
- CC: gcc-patches at gcc dot gnu dot org, gcc at gcc dot gnu dot org
- References: <3872D49E.F1DE4861@xlinux.com>
> Let's testing...
Thanks for your patch. I didn't review it in detail, but I see that you
made a lot of formatting changes of the style
- ; /* cpp handles this one. */
+ ; /* cpp handles this one. */
Please don't make these changes. It is the GNU coding convention to
have two spaces after a full stop, before the end of the
comment. Among other things, that makes Emacs auto-wrap work for
comments. Another set of changes is of the form
-
+
or
-^L (actually, this should be a form feed)
+
Also, because of all these extra changes, it is very hard to see what
the patch actually does. If you remove these, you probably wouldn't
even need to compress the patch.
Regards,
Martin