[Bug c++/41185] New: size of array ... has non-integral type ...

[Bug c++/41185] New: size of array ... has non-integral type ...

[pluto at agmk dot net]

hi,

during compiling bmpx package (http://bmpx.backtrace.info/site)
i get an error with gcc-4.4.1:

ui-part-library.cc:512: error:
size of array 'i_toplevel' has non-integral type
'Gtk::TreeModelColumn<Bmp::AlbumArtist>'

here's the line that confuses compiler:

UID uid(
  Bmp::AlbumArtist(
    (*i_toplevel)[mStoreArtistCR.artist]
  ).bmpx_album_artist_id);

the minor reorganization of code:

Bmp::AlbumArtist aa((*i_toplevel)[mStoreArtistCR.artist]);
UID uid(aa.bmpx_album_artist_id);

and it compiles w/o errors.
mentioned error didn't occur in gcc-4.3.x. new 4.4 regression?


-- 
           Summary: size of array ... has non-integral type ...
           Product: gcc
           Version: 4.4.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: pluto at agmk dot net
 GCC build triplet: x86_64-gnu-linux
  GCC host triplet: x86_64-gnu-linux
GCC target triplet: x86_64-gnu-linux


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] size of array ... has non-integral type ...

[pluto at agmk dot net]

------- Comment #1 from pluto at agmk dot net  2009-08-29 08:15 -------
Created an attachment (id=18450)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=18450&action=view)
testcase

g++ ui-part-library.ii -c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] size of array ... has non-integral type ...

[pluto at agmk dot net]

------- Comment #2 from pluto at agmk dot net  2010-03-09 19:35 -------
ping^0


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] size of array ... has non-integral type ...

[paolo dot carlini at oracle dot com]

------- Comment #3 from paolo dot carlini at oracle dot com  2010-03-09 19:42 -------
If you could provide a small self contained snippet exhibiting the problem, it
would be great.

Jason, can you have a look at this? Just in case it's actually a regression...
Thanks in advance.


-- 

paolo dot carlini at oracle dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jason at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] size of array ... has non-integral type ...

[pluto at agmk dot net]

------- Comment #4 from pluto at agmk dot net  2010-03-09 22:14 -------
Created an attachment (id=20065)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20065&action=view)
reduced testcase.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] size of array ... has non-integral type ...

[pluto at agmk dot net]

------- Comment #5 from pluto at agmk dot net  2010-03-09 22:15 -------
$ g++ t.cpp -c
t.cpp: In member function 'void Bmp::Artist_V::put_artist(const
Bmp::AlbumArtist&)':
t.cpp:33: error: size of array 'i_toplevel' has non-integral type
'Gtk::TreeModelColumn<Bmp::AlbumArtist>'

t.cpp:33: error: no match for 'operator[]' in
'i_toplevel[((Bmp::Artist_V*)this)->Bmp::Artist_V::mStoreArtistCR.Bmp::Artist_V::ArtistCR::artist]'


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] size of array ... has non-integral type ...

[pinskia at gcc dot gnu dot org]

------- Comment #6 from pinskia at gcc dot gnu dot org  2010-03-09 22:21 -------
This looks like the normal issue of confusing between function declaration vs a
variable initialization.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] size of array ... has non-integral type ...

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2010-03-09 22:22 -------
very much related to PR 41786


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] [4.4/4.5 regression] size of array ... has non-integral type ...

[jason at gcc dot gnu dot org]

------- Comment #8 from jason at gcc dot gnu dot org  2010-03-09 22:42 -------
This and 41786 are both bugs due to my change to push parameter declarations
early, so we get some semantic errors even though we're parsing tentatively.


-- 

jason at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |jason at gcc dot gnu dot org
                   |dot org                     |
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2010-03-09 22:42:44
               date|                            |
            Summary|size of array ... has non-  |[4.4/4.5 regression] size of
                   |integral type ...           |array ... has non-integral
                   |                            |type ...


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] [4.4/4.5 regression] size of array ... has non-integral type ...

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |rejects-valid
           Priority|P3                          |P2
   Target Milestone|---                         |4.4.4


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] [4.4/4.5 regression] size of array ... has non-integral type ...

[jason at gcc dot gnu dot org]

------- Comment #9 from jason at gcc dot gnu dot org  2010-03-30 21:19 -------
Subject: Bug 41185

Author: jason
Date: Tue Mar 30 21:19:23 2010
New Revision: 157838

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157838
Log:
        PR c++/41185
        PR c++/41786
        * parser.c (cp_parser_direct_declarator): Don't allow VLAs in
        function parameter context.  Don't print an error if parsing
        tentatively.

Added:
    trunk/gcc/testsuite/g++.dg/parse/ambig5.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/parser.c
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/parse/varmod1.C


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] [4.4/4.5 regression] size of array ... has non-integral type ...

[jason at gcc dot gnu dot org]

------- Comment #10 from jason at gcc dot gnu dot org  2010-03-30 21:21 -------
Subject: Bug 41185

Author: jason
Date: Tue Mar 30 21:20:58 2010
New Revision: 157839

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157839
Log:
        PR c++/41185
        PR c++/41786
        * parser.c (cp_parser_direct_declarator): Don't allow VLAs in
        function parameter context.  Don't print an error if parsing
        tentatively.

Added:
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/parse/ambig5.C
Modified:
    branches/gcc-4_4-branch/gcc/cp/ChangeLog
    branches/gcc-4_4-branch/gcc/cp/parser.c
    branches/gcc-4_4-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/parse/varmod1.C


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185

[Bug c++/41185] [4.4/4.5 regression] size of array ... has non-integral type ...

[jason at gcc dot gnu dot org]

------- Comment #11 from jason at gcc dot gnu dot org  2010-03-30 21:23 -------
Fixed for 4.4.4 and 4.5.0.


-- 

jason at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41185