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;
 

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Warzone-dev mailing list
Warzone-dev@gna.org
https://mail.gna.org/listinfo/warzone-dev

Reply via email to