public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug c++/14361] New: Accepting invalid code with weird default parameter trailer. @ 2004-03-01 9:38 mattyt-bugzilla at tpg dot com dot au 2004-03-01 9:55 ` [Bug c++/14361] " pinskia at gcc dot gnu dot org ` (5 more replies) 0 siblings, 6 replies; 7+ messages in thread From: mattyt-bugzilla at tpg dot com dot au @ 2004-03-01 9:38 UTC (permalink / raw) To: gcc-bugs The code: class A { A ( int n=0 int n ); }; compiles in G++-3.4, but shouldn't, as I understand it. -- Summary: Accepting invalid code with weird default parameter trailer. Product: gcc Version: 3.4.0 Status: UNCONFIRMED Severity: normal Priority: P2 Component: c++ AssignedTo: unassigned at gcc dot gnu dot org ReportedBy: mattyt-bugzilla at tpg dot com dot au CC: gcc-bugs at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14361 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug c++/14361] Accepting invalid code with weird default parameter trailer. 2004-03-01 9:38 [Bug c++/14361] New: Accepting invalid code with weird default parameter trailer mattyt-bugzilla at tpg dot com dot au @ 2004-03-01 9:55 ` pinskia at gcc dot gnu dot org 2004-03-01 21:10 ` [Bug c++/14361] [3.4/3.5 Regression] " mmitchel at gcc dot gnu dot org ` (4 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: pinskia at gcc dot gnu dot org @ 2004-03-01 9:55 UTC (permalink / raw) To: gcc-bugs ------- Additional Comments From pinskia at gcc dot gnu dot org 2004-03-01 09:55 ------- Confirmed a regression from 3.3.1, also note it is even worse than that. this is accepted: struct A { A ( int n=0 ,int n1 ); }; And this is too: struct A { A ( int n=0 i ); }; Looks the parser is ignorning anything after the default argument until the ')' token. -- What |Removed |Added ---------------------------------------------------------------------------- CC| |mmitchel at gcc dot gnu dot | |org Status|UNCONFIRMED |NEW Ever Confirmed| |1 Keywords| |accepts-invalid Known to fail| |3.4.0 3.5.0 Known to work| |3.3.1 Last reconfirmed|0000-00-00 00:00:00 |2004-03-01 09:55:36 date| | Target Milestone|--- |3.4.0 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14361 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug c++/14361] [3.4/3.5 Regression] Accepting invalid code with weird default parameter trailer. 2004-03-01 9:38 [Bug c++/14361] New: Accepting invalid code with weird default parameter trailer mattyt-bugzilla at tpg dot com dot au 2004-03-01 9:55 ` [Bug c++/14361] " pinskia at gcc dot gnu dot org @ 2004-03-01 21:10 ` mmitchel at gcc dot gnu dot org 2004-03-02 2:00 ` cvs-commit at gcc dot gnu dot org ` (3 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: mmitchel at gcc dot gnu dot org @ 2004-03-01 21:10 UTC (permalink / raw) To: gcc-bugs -- What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|unassigned at gcc dot gnu |mark at codesourcery dot com |dot org | Status|NEW |ASSIGNED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14361 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug c++/14361] [3.4/3.5 Regression] Accepting invalid code with weird default parameter trailer. 2004-03-01 9:38 [Bug c++/14361] New: Accepting invalid code with weird default parameter trailer mattyt-bugzilla at tpg dot com dot au 2004-03-01 9:55 ` [Bug c++/14361] " pinskia at gcc dot gnu dot org 2004-03-01 21:10 ` [Bug c++/14361] [3.4/3.5 Regression] " mmitchel at gcc dot gnu dot org @ 2004-03-02 2:00 ` cvs-commit at gcc dot gnu dot org 2004-03-02 2:06 ` cvs-commit at gcc dot gnu dot org ` (2 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: cvs-commit at gcc dot gnu dot org @ 2004-03-02 2:00 UTC (permalink / raw) To: gcc-bugs ------- Additional Comments From cvs-commit at gcc dot gnu dot org 2004-03-02 02:00 ------- Subject: Bug 14361 CVSROOT: /cvs/gcc Module name: gcc Branch: gcc-3_4-branch Changes by: mmitchel@gcc.gnu.org 2004-03-02 02:00:06 Modified files: gcc/cp : ChangeLog decl.c parser.c pt.c gcc/testsuite : ChangeLog gcc/testsuite/g++.old-deja/g++.ns: koenig5.C Added files: gcc/testsuite/g++.dg/parse: defarg7.C gcc/testsuite/g++.dg/template: friend26.C Log message: PR c++/14360 * parser.c (cp_parser_postfix_expression): Do not perform Koenig lookup if ordinary name-lookup finds a non-function. * pt.c (tsubst_copy_and_build): Likewise. PR c++/14361 * parser.c (cp_parser_late_parsing_default_args): Check that there are no extra tokens after the end of the default-argument expression. Backport 2004-02-12 Mark Mitchell <mark@codesourcery.com> * decl.c (redeclaration_error_message): Correct handling of templates. * pt.c (tsubst_friend_declaration): Adjust code to determine whether or not a friend template is a definition. (tsubst_decl): Clear DECL_INITIAL for new FUNCTION_DECLs. Patches: http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.3892.2.65&r2=1.3892.2.66 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/decl.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.1174.2.10&r2=1.1174.2.11 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/parser.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.157.2.17&r2=1.157.2.18 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/pt.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.816.2.15&r2=1.816.2.16 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.3389.2.115&r2=1.3389.2.116 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/parse/defarg7.C.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=NONE&r2=1.1.2.1 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/template/friend26.C.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=NONE&r2=1.1.2.1 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.old-deja/g++.ns/koenig5.C.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.3&r2=1.3.16.1 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14361 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug c++/14361] [3.4/3.5 Regression] Accepting invalid code with weird default parameter trailer. 2004-03-01 9:38 [Bug c++/14361] New: Accepting invalid code with weird default parameter trailer mattyt-bugzilla at tpg dot com dot au ` (2 preceding siblings ...) 2004-03-02 2:00 ` cvs-commit at gcc dot gnu dot org @ 2004-03-02 2:06 ` cvs-commit at gcc dot gnu dot org 2004-03-02 2:06 ` mmitchel at gcc dot gnu dot org 2004-03-02 2:07 ` mmitchel at gcc dot gnu dot org 5 siblings, 0 replies; 7+ messages in thread From: cvs-commit at gcc dot gnu dot org @ 2004-03-02 2:06 UTC (permalink / raw) To: gcc-bugs ------- Additional Comments From cvs-commit at gcc dot gnu dot org 2004-03-02 02:06 ------- Subject: Bug 14361 CVSROOT: /cvs/gcc Module name: gcc Changes by: mmitchel@gcc.gnu.org 2004-03-02 02:06:18 Modified files: gcc/cp : ChangeLog parser.c pt.c gcc/testsuite : ChangeLog gcc/testsuite/g++.old-deja/g++.ns: koenig5.C Added files: gcc/testsuite/g++.dg/parse: defarg7.C gcc/testsuite/g++.dg/template: friend26.C Log message: PR c++/14360 * parser.c (cp_parser_postfix_expression): Do not perform Koenig lookup if ordinary name-lookup finds a non-function. * pt.c (tsubst_copy_and_build): Likewise. PR c++/14361 * parser.c (cp_parser_late_parsing_default_args): Check that there are no extra tokens after the end of the default-argument expression. PR c++/14360 * g++.old-deja/g++.ns/koenig5.C: Remove some error markers. PR c++/14361 * g++.dg/parse/defarg7.C: New test. PR c++/14359 * g++.dg/template/friend26.C: New test. Patches: http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/ChangeLog.diff?cvsroot=gcc&r1=1.3981&r2=1.3982 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/parser.c.diff?cvsroot=gcc&r1=1.178&r2=1.179 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/pt.c.diff?cvsroot=gcc&r1=1.834&r2=1.835 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.3560&r2=1.3561 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/parse/defarg7.C.diff?cvsroot=gcc&r1=1.1&r2=1.2 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/template/friend26.C.diff?cvsroot=gcc&r1=1.1&r2=1.2 http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.old-deja/g++.ns/koenig5.C.diff?cvsroot=gcc&r1=1.3&r2=1.4 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14361 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug c++/14361] [3.4/3.5 Regression] Accepting invalid code with weird default parameter trailer. 2004-03-01 9:38 [Bug c++/14361] New: Accepting invalid code with weird default parameter trailer mattyt-bugzilla at tpg dot com dot au ` (3 preceding siblings ...) 2004-03-02 2:06 ` cvs-commit at gcc dot gnu dot org @ 2004-03-02 2:06 ` mmitchel at gcc dot gnu dot org 2004-03-02 2:07 ` mmitchel at gcc dot gnu dot org 5 siblings, 0 replies; 7+ messages in thread From: mmitchel at gcc dot gnu dot org @ 2004-03-02 2:06 UTC (permalink / raw) To: gcc-bugs ------- Additional Comments From mmitchel at gcc dot gnu dot org 2004-03-02 02:06 ------- Fixed in GCC 3.4 and GCC 3.5. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14361 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug c++/14361] [3.4/3.5 Regression] Accepting invalid code with weird default parameter trailer. 2004-03-01 9:38 [Bug c++/14361] New: Accepting invalid code with weird default parameter trailer mattyt-bugzilla at tpg dot com dot au ` (4 preceding siblings ...) 2004-03-02 2:06 ` mmitchel at gcc dot gnu dot org @ 2004-03-02 2:07 ` mmitchel at gcc dot gnu dot org 5 siblings, 0 replies; 7+ messages in thread From: mmitchel at gcc dot gnu dot org @ 2004-03-02 2:07 UTC (permalink / raw) To: gcc-bugs ------- Additional Comments From mmitchel at gcc dot gnu dot org 2004-03-02 02:07 ------- Fixed in GCC 3.4 and GCC 3.5. -- What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=14361 ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2004-03-02 2:07 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2004-03-01 9:38 [Bug c++/14361] New: Accepting invalid code with weird default parameter trailer mattyt-bugzilla at tpg dot com dot au 2004-03-01 9:55 ` [Bug c++/14361] " pinskia at gcc dot gnu dot org 2004-03-01 21:10 ` [Bug c++/14361] [3.4/3.5 Regression] " mmitchel at gcc dot gnu dot org 2004-03-02 2:00 ` cvs-commit at gcc dot gnu dot org 2004-03-02 2:06 ` cvs-commit at gcc dot gnu dot org 2004-03-02 2:06 ` mmitchel at gcc dot gnu dot org 2004-03-02 2:07 ` mmitchel at gcc dot gnu dot org
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).