Dennis Schridde schreef: > Author: devurandom > Date: Sat Dec 2 15:38:09 2006 > New Revision: 528 > > URL: http://svn.gna.org/viewcvs/warzone?rev=528&view=rev > Log: > Fix last commits and remove another bunch of warnings. > > Modified: > ... > trunk/lib/widget/widgbase.h > ... > Resending since previous mail didn't arrive:
The modifications to widgbase.h move `enum _widget_type' lower into the file and forward declare it instead. The C++ standard however does not support forward declarations of enums. So as a result compilability cannot be guaranteed for this file, in fact it breaks compileability on the current MinGW version (I'm not sure about newer versions of GCC). See these links for info about forward declarations of enum in C/C++: http://www.thescripts.com/forum/thread62285.html http://www.cygwin.com/ml/cygwin/2000-04/msg00738.html Therefore I suggest the attached patch. This removes the forward declaration of enum (since the header-file itself doesn't refer to this enum a forward declaration isn't even required) back up top to comply with C++ standard again, and in a result make it be compilable again for MinGW (which is what I'm using btw). All this patch does is remove the forward declaring line, and with that change renders the file compileable again (or to be precise, the files that include it). -- Giel
Index: lib/widget/widgbase.h =================================================================== --- lib/widget/widgbase.h (revision 535) +++ lib/widget/widgbase.h (working copy) @@ -17,7 +17,6 @@ /* Forward definitions */ -enum _widget_type; struct _widget; struct _w_context;
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev