[Live-devel] Re: building using CC

Ross Finlayson finlayson at live.com
Sat Mar 19 09:27:13 PST 2005


>Sun's documentation implies that they do not consider this behaviour a bug;
>see http://www.sun.com/software/sundev/suncc/faqs/cpp.xml#q17 .
>Their suggested solution (patch below) appears to work (compile-time test
>only) both with their compiler and with g++ (I tried 2.95 and 3.3.5).
>
>I am not enough of a C++ language lawyer for my opinion to carry much weight,
>but I suspect that they are right and the unpatched BasicHashTable.hh is not
>standard-conforming.
>
>--- old/BasicUsageEnvironment/include/BasicHashTable.hh 2005-03-11 
>10:59:39.000000000 +0100
>+++ new/BasicUsageEnvironment/include/BasicHashTable.hh 2005-03-19 
>13:30:00.425148000 +0100
>@@ -38,6 +38,8 @@
>    virtual ~BasicHashTable();
>
>    // Used to iterate through the members of the table:
>+  class Iterator;
>+  friend class Iterator;
>    class Iterator: public HashTable::Iterator {
>    public:
>      Iterator(BasicHashTable& table);

There was already a
         "friend class Iterator;"
further down in the defintion.  I believe that that should have been 
sufficient.  However, to make your compiler happy, I have made the change 
that you suggested.

Is there anywhere else in the code where you need a similar change?


	Ross Finlayson
	LIVE.COM
	<http://www.live.com/>



More information about the live-devel mailing list