This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [c/c++/fortran] PR35058: -Werror= works only with some warnin
- From: FX <fxcoudert at gmail dot com>
- To: "Manuel López-Ibáñez" <lopezibanez at gmail dot com>
- Cc: "Gcc Patch List" <gcc-patches at gcc dot gnu dot org>, "fortran at gcc dot gnu dot org List" <fortran at gcc dot gnu dot org>, "Joseph S. Myers" <joseph at codesourcery dot com>
- Date: Fri, 13 Jun 2008 17:45:54 +0100
- Subject: Re: [c/c++/fortran] PR35058: -Werror= works only with some warnin
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:in-reply-to:references :mime-version:content-type:message-id:cc:content-transfer-encoding :from:subject:date:to:x-mailer; bh=n8oYNaDnPrJPx3G2FIGm4D6Sal+HAhKkDzWwTMtLIZQ=; b=K2c3MRt/w87Tv7vDTmyLIIAZihltUnTV0D3k3OFokA0bm/KGziQkPtjMyGRuj6Mzz1 AUG3jGzSfpfN2ZqNQrrpfOUxAXFsnfdU+iCljtnCjkNbtMbTapKPcWsZYUPiGM1Rr0NW MtSIAutbDapqxBManvPbJis7dyyR8E4hyh3IY=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=in-reply-to:references:mime-version:content-type:message-id:cc :content-transfer-encoding:from:subject:date:to:x-mailer; b=coC90duGIgXJXhIU+EM2KcBm0FkkipZTowvj6n5cgzRhwMZBiLNLxgzvZhDknPehur PsdjYpsjn6Qg2D0Vl+c49QU782/RldE51R981W/VZ01KkNNt+1JwD4FLpF5/2jq9Zd+7 Wo/X0dO9y5G5ADQ0aS9/r6wubHn3cwVSTdAy4=
- References: <6c33472e0806130933u28a4a3acu701309d47b1e1c3@mail.gmail.com>
(Writing this Changelog entry was a horrible horrible experience! And
I find it totally uninformative. I think there is something wrong with
our approach.)
I do agree with you that the current requirements for ChangeLog
entries aren't at all reasonable (or even useful) for large patches
of this type.
One thing that you should not do, however, is to use "Likewise." for
the first entry of a given ChangeLog file, because it looks like it
refers to the previous entry in the same ChangeLog:
fortran/
* f95-lang.c (gfc_mark_addressable): Likewise.
cp/
* typeck.c (composite_pointer_type_r): Likewise.
Otherwise, the fortran/ part of it is OK (and thanks for CC'ing the
fortran list!).
FX
--
François-Xavier Coudert
http://www.homepages.ucl.ac.uk/~uccafco/