[Bug c++/63985] New: Accepts invalid range-based for declaration

[Bug c++/63985] New: Accepts invalid range-based for declaration

[R.HL at gmx dot net]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63985

            Bug ID: 63985
           Summary: Accepts invalid range-based for declaration
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: R.HL at gmx dot net

int arr[] {1, 2, 3};
    for (int i = 5: arr)
        std::cout << i << ", ";

Compiles, while it shouldn't. A warning message suggests that GCC ignores the
loop completely.

    warning: variable 'arr' set but not used

Perhaps GCC can't decide what kind of loop this is and decides to ignore it
silently?

[Bug c++/63985] Accepts invalid range-based for declaration

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63985

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-11-20
     Ever confirmed|0                           |1

--- Comment #1 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Confirmed

int main()
{
  int arr;
  for (int i = 5: arr)
    return 1;
  return 0;
}

[Bug c++/63985] Accepts invalid range-based for declaration

[paolo.carlini at oracle dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63985

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |paolo.carlini at oracle dot com
              Build|g++ -std=c++11 -Wall        |
                   |-Wextra -pedantic           |

--- Comment #2 from Paolo Carlini <paolo.carlini at oracle dot com> ---
Mine.

[Bug c++/63985] Accepts invalid range-based for declaration

[paolo at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63985

--- Comment #3 from paolo at gcc dot gnu.org <paolo at gcc dot gnu.org> ---
Author: paolo
Date: Wed Dec 24 09:07:23 2014
New Revision: 219054

URL: https://gcc.gnu.org/viewcvs?rev=219054&root=gcc&view=rev
Log:
/cp
2014-12-04  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/63985
    * parser.c (cp_parser_for_init_statement): Reject invalid declarations
    in range-based for loops.

/testsuite
2014-12-04  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/63985
    * g++.dg/cpp0x/range-for29.C: New.

Added:
    trunk/gcc/testsuite/g++.dg/cpp0x/range-for29.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/parser.c
    trunk/gcc/testsuite/ChangeLog

[Bug c++/63985] Accepts invalid range-based for declaration

[paolo.carlini at oracle dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63985

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
           Assignee|paolo.carlini at oracle dot com    |unassigned at gcc dot gnu.org
   Target Milestone|---                         |5.0

--- Comment #4 from Paolo Carlini <paolo.carlini at oracle dot com> ---
Fixed for 5.0.