Re: [C++0x] implementing forward declarations for enums

2010-09-25 Thread Jason Merrill
On 09/20/2010 09:58 AM, Rodrigo Rivas wrote: This patch tries to implement the C++0x featue "Forward declarations for enums" aka "opaque enum declarations": Great! BTW, please send C++ patches to gcc-patches and CC me so that I see them right away; I tend to fall behind on the mailing lists.

Re: [C++0x] implementing forward declarations for enums

2010-09-22 Thread Rodrigo Rivas
I had to initialize the variable nested_being_defined to get it to compile (possible uninitialized warning).  I initialized it to false. >>> Ok, actually it is never used uninitialized, but let's get rid of the >>> warning. >>> >> I saw that it was never used uninitialized and was s

RE: [C++0x] implementing forward declarations for enums

2010-09-22 Thread Hargett, Matt
On 09/21/2010 07:05 PM, Rodrigo Rivas wrote: >>> I had to initialize the variable nested_being_defined to get it to compile >>> (possible uninitialized warning). I initialized it to false. >>> >> Ok, actually it is never used uninitialized, but let's get rid of the >> warning. >> > I sa

Re: [C++0x] implementing forward declarations for enums

2010-09-22 Thread Rodrigo Rivas
> I saw the flag situation after I sent my message. There must be a "Keeper of > the Sacred Tree Flag Set" There are a few comments en tree.h about that, I if remember correctly... > I get the idea the real estate is at a premium and consuming flags could be > a problem. > Perhaps there is a con

Re: [C++0x] implementing forward declarations for enums

2010-09-21 Thread Ed Smith-Rowland
On 09/21/2010 07:05 PM, Rodrigo Rivas wrote: I had to initialize the variable nested_being_defined to get it to compile (possible uninitialized warning). I initialized it to false. Ok, actually it is never used uninitialized, but let's get rid of the warning. I saw that it was never u

Re: [C++0x] implementing forward declarations for enums

2010-09-21 Thread Rodrigo Rivas
> I had to initialize the variable nested_being_defined to get it to compile > (possible uninitialized warning).  I initialized it to false. Ok, actually it is never used uninitialized, but let's get rid of the warning. > It looks like the first two are related.  What does an enum look like in > t

Re: [C++0x] implementing forward declarations for enums

2010-09-21 Thread Ed Smith-Rowland
On 09/20/2010 09:58 AM, Rodrigo Rivas wrote: Hello all. This patch tries to implement the C++0x featue "Forward declarations for enums" aka "opaque enum declarations": http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2008/n2764.pdf Please note that this is a WIP, and as such lacks formatting,

RE: [C++0x] implementing forward declarations for enums

2010-09-20 Thread Hargett, Matt
Rivas Sent: Monday, September 20, 2010 6:58 AM To: gcc@gcc.gnu.org Subject: [C++0x] implementing forward declarations for enums Hello all. This patch tries to implement the C++0x featue "Forward declarations for enums" aka "opaque enum declarations": http://www.open-std.org

[C++0x] implementing forward declarations for enums

2010-09-20 Thread Rodrigo Rivas
Hello all. This patch tries to implement the C++0x featue "Forward declarations for enums" aka "opaque enum declarations": http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2008/n2764.pdf Please note that this is a WIP, and as such lacks formatting, comments, testcases, etc. Except for the thin