]> git.draconx.ca Git - gob-dx.git/blobdiff - src/main.c
Release 0.93.3
[gob-dx.git] / src / main.c
index 751c0642113f52804caa73a1422c5d71b87ed4d3..09f59c254a86bc03f0a357645dbe8cf91a51d8ef 100644 (file)
 
 #include "config.h"
 #include <glib.h>
-#if 0
-#include <popt.h>
-#endif
 #include <time.h>
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>
+#include <stdlib.h>
 #include <sys/stat.h>
 
-#include "tree.h"
+#include "treefuncs.h"
 #include "parse.h"
 #include "out.h"
+#include "util.h"
+#include "checks.h"
+
 #include "main.h"
 
 char *filename = NULL;
@@ -46,6 +47,8 @@ extern GList *nodes;
 
 extern GList *include_files;
 
+extern GHashTable *gtk_doc_hash;
+
 char *filebase;
 static char *funcbase;
 static char *pfuncbase;
@@ -57,227 +60,243 @@ static char *ptypebase;
 
 static int signals = 0; /* number of signals */
 static int arguments = 0; /* number of named arguments */
-static int overrides = 0; /* number of override functions */
+static int overrides = 0; /* number of override methods */
 static int privates = 0; /* number of private data members */
+static int protecteds = 0; /* number of protected methods */
+static int destructors = 0; /* number of variable destructors */
+static int initializers = 0; /* number of variable initializers */
 
 static gboolean made_aliases = FALSE;  /* if we made any shorthand aliases
                                          and need the REALLY UGLY HACK to
                                          avoid warnings */
 
+/* the special variable types we need to define */
+static gboolean special_array[SPECIAL_LAST] = {0};
+static gboolean any_special = FALSE;
+
 FILE *out = NULL;
 FILE *outh = NULL;
 FILE *outph = NULL;
+FILE *devnull = NULL;
 
 gboolean no_touch_headers = FALSE;
 gboolean for_cpp = FALSE;
+gboolean no_gnu = FALSE;
 gboolean exit_on_warn = FALSE;
 gboolean exit_on_error = TRUE;
 gboolean got_error = FALSE;
 gboolean always_private_header = FALSE;
 gboolean no_private_header = FALSE;
-
-void
-print_error(int is_warn, char *error,int line)
-{
-       char *w;
-       if(is_warn)
-               w = "Warning:";
-       else {
-               w = "Error:";
-               got_error = TRUE;
-       }
-       if(line>0)
-               fprintf(stderr,"%s:%d: %s %s\n",filename,line,w,error);
-       else
-               fprintf(stderr,"%s: %s %s\n",filename,w,error);
-       if((!is_warn || exit_on_warn) && exit_on_error)
-               exit(1);
-}
-
-static char *
-remove_sep(char *base)
-{
-       char *p;
-       char *s = g_strdup(base);
-       while((p=strchr(s,':')))
-               strcpy(p,p+1);
-       return s;
-}
-
-static char *
-replace_sep(char *base, char r)
-{
-       char *p;
-       char *s = g_strdup(base);
-       while((p=strchr(s,':')))
-               *p = r;
-       if(*s == r) {
-               p = g_strdup(s+1);
-               g_free(s);
-               return p;
-       }
-       return s;
-}
-
-/*separate the namespace part and then replace rest of
-  separators with r*/
-static void
-separns_replace_sep(char *base, char **ns, char **name, char r)
-{
-       char *p;
-       char *s = g_strdup(base);
-       *ns = NULL;
-       if((p=strchr(s,':')) && p!=s) {
-               *p = '\0';
-               *ns = g_strdup(s);
-               p = g_strdup(p+1);
-               g_free(s);
-               s = p;
-       }
-       while((p=strchr(s,':')))
-               *p = r;
-       if(*s == r) {
-               *name = g_strdup(s+1);
-               g_free(s);
-       } else
-               *name = s;
-}
-
-/* make a macro with some prefix before the name but after
-   namespace */
-static char *
-make_pre_macro(char *base, char *pre)
-{
-       char *s1,*s2;
-       char *s;
-
-       separns_replace_sep(base,&s1,&s2,'_');
-       if(s1)
-               s = g_strconcat(s1,"_",pre,"_",s2,NULL);
-       else
-               s = g_strconcat(pre,"_",s2,NULL);
-
-       g_strup(s);
-       
-       g_free(s1);
-       g_free(s2);
-
-       return s;
-}
+gboolean no_extern_c = FALSE;
+gboolean no_write = FALSE;
+gboolean no_lines = FALSE;
 
 static void
 make_bases(void)
 {
-       filebase = replace_sep(((Class *)class)->otype,'-');
+       filebase = replace_sep(((Class *)class)->otype, '-');
        g_strdown(filebase);
 
-       funcbase = replace_sep(((Class *)class)->otype,'_');
+       funcbase = replace_sep(((Class *)class)->otype, '_');
        g_strdown(funcbase);
 
-       pfuncbase = replace_sep(((Class *)class)->ptype,'_');
+       pfuncbase = replace_sep(((Class *)class)->ptype, '_');
        g_strdown(pfuncbase);
 
-       macrobase = replace_sep(((Class *)class)->otype,'_');
+       macrobase = replace_sep(((Class *)class)->otype, '_');
        g_strup(macrobase);
        
-       macrois = make_pre_macro(((Class *)class)->otype,"IS");
-       macrotype = make_pre_macro(((Class *)class)->otype,"TYPE");
+       macrois = make_pre_macro(((Class *)class)->otype, "IS");
+       macrotype = make_pre_macro(((Class *)class)->otype, "TYPE");
 
        typebase = remove_sep(((Class *)class)->otype);
 
        ptypebase = remove_sep(((Class *)class)->ptype);
 }
 
-static void
-print_type(FILE *fp, Type *t, gboolean postfix_to_stars)
+static char *
+get_type(Type *t, gboolean postfix_to_stars)
 {
        char *s;
        int i;
        int extra;
+       GString *gs;
 
        s = remove_sep(t->name);
-       out_printf(fp,"%s ",s); 
+       gs = g_string_new(s);
        g_free(s);
 
        extra = 0;
        if(postfix_to_stars) {
                char *p;
                /*XXX: this is ugly perhaps we can do this whole postfix thing
-                 in a nicer way, we just count the number of '[' s and from that
-                 we deduce the number of dimensions */
+                 in a nicer way, we just count the number of '[' s and from
+                 that we deduce the number of dimensions, so that we can print
+                 that many stars */
                for(p=t->postfix; p && *p; p++)
                        if(*p == '[') extra++;
        }
+       g_string_append_c(gs, ' ');
        
-       for(i=0;i<(t->stars+extra);i++)
-               out_printf(fp,"*"); 
+       for(i=0; i<(t->stars+extra); i++)
+               g_string_append_c(gs, '*');
+
+       s = gs->str;
+       g_string_free(gs, FALSE);
+       return s;
+}
+
+static char *
+get_gtk_doc(char *id)
+{
+       char *val, *s;
+
+       if(!gtk_doc_hash)
+               return NULL;
+
+       val = g_hash_table_lookup(gtk_doc_hash, id);
+       if(val)
+               return g_strdup_printf("/**\n * %s_%s:\n%s **/\n",
+                                      funcbase, id, val);
+       s = g_strconcat(funcbase, "_", id, NULL);
+       val = g_hash_table_lookup(gtk_doc_hash, s);
+       g_free(s);
+       if(val)
+               return g_strdup_printf("/**\n * %s_%s:\n%s **/\n",
+                                      funcbase, id, val);
+       return NULL;
+}
+
+static void
+print_type(FILE *fp, Type *t, gboolean postfix_to_stars)
+{
+       char *s;
+
+       s = get_type(t, postfix_to_stars);
+       out_printf(fp, "%s", s); 
+       g_free(s);
 }
 
+
 static void
 print_method(FILE *fp, char *typeprefix, char *nameprefix,
-            char *namepostfix,char *postfix, Method *m,
+            char *subnameprefix,
+            char *namepostfix, char *postfix, Method *m,
+            gboolean one_arg_per_line,
             gboolean no_funcbase)
 {
        GList *li;
 
-       out_printf(fp,"%s",typeprefix); 
-       print_type(fp,m->mtype,TRUE);
+       out_printf(fp, "%s", typeprefix); 
+       print_type(fp, m->mtype, TRUE);
        if(no_funcbase)
-               out_printf(fp,"%s%s%s(",
-                          nameprefix,m->id,namepostfix); 
+               out_printf(fp, "%s%s%s%s(",
+                          nameprefix, subnameprefix, m->id, namepostfix); 
        else
-               out_printf(fp,"%s%s_%s%s(",
-                          nameprefix,funcbase,m->id,namepostfix); 
+               out_printf(fp, "%s%s_%s%s%s(",
+                          nameprefix, funcbase, subnameprefix, m->id,
+                          namepostfix); 
        
        if(m->args) {
-               for(li=m->args;li;li=g_list_next(li)) {
+               for(li=m->args; li; li=g_list_next(li)) {
                        FuncArg *arg = li->data;
-                       print_type(fp,arg->atype,FALSE);
+                       print_type(fp, arg->atype, FALSE);
                        if(li->next)
-                               out_printf(fp,"%s%s, ",arg->name,
+                               out_printf(fp, "%s%s,%s", arg->name,
                                           arg->atype->postfix?
-                                          arg->atype->postfix:""); 
+                                          arg->atype->postfix:"",
+                                          one_arg_per_line?"\n\t\t\t\t\t":" ");
                        else
-                               out_printf(fp,"%s%s",arg->name,
+                               out_printf(fp, "%s%s", arg->name,
                                           arg->atype->postfix?
                                           arg->atype->postfix:""); 
                }
                if(m->vararg)
-                       out_printf(fp,", ..."); 
+                       out_printf(fp, ",%s...",
+                                  one_arg_per_line?"\n\t\t\t\t\t":" "); 
        } else {
-               out_printf(fp,"void"); 
+               out_printf(fp, "void"); 
+       }
+       out_printf(fp, ")%s", postfix); 
+}
+
+static gboolean
+any_method_to_alias(Class *c)
+{
+       GList *li;
+       
+       for(li=c->nodes;li;li=g_list_next(li)) {
+               Node *node = li->data;
+               if(node->type == METHOD_NODE) {
+                       Method *m = (Method *)node;
+                       
+                       if(m->method == INIT_METHOD ||
+                          m->method == CLASS_INIT_METHOD ||
+                          m->method == OVERRIDE_METHOD)
+                               continue;
+
+                       return TRUE;
+               }
        }
-       out_printf(fp,")%s",postfix); 
+       return FALSE;
 }
 
 
 static void
-make_method_pointers(Class *c)
+make_method_gnu_aliases(Class *c)
 {
        GList *li;
        
-       out_printf(out,"\n");
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *node = li->data;
                if(node->type == METHOD_NODE) {
                        Method *m = (Method *)node;
                        
-                       if(m->scope == INIT_METHOD ||
-                          m->scope == CLASS_INIT_METHOD ||
-                          m->scope == OVERRIDE_METHOD)
+                       if(m->method == INIT_METHOD ||
+                          m->method == CLASS_INIT_METHOD ||
+                          m->method == OVERRIDE_METHOD)
                                continue;
 
                        /* in C++ mode don't alias new */
-                       if(for_cpp && strcmp(m->id,"new")==0)
+                       if(for_cpp && strcmp(m->id, "new")==0)
+                               continue;
+
+                       out_printf(out, "static const typeof(&%s_%s) %s "
+                                  "__attribute__ ((__unused__)) "
+                                  "= %s_%s;\n", funcbase, m->id, m->id,
+                                  funcbase, m->id);
+                       out_printf(out, "#define %s(args...) "
+                                  "%s_%s(##args)\n", m->id, funcbase, m->id);
+               }
+       }
+}
+
+static void
+make_method_nongnu_aliases(Class *c)
+{
+       GList *li;
+       
+       for(li=c->nodes; li; li=g_list_next(li)) {
+               Node *node = li->data;
+               if(node->type == METHOD_NODE) {
+                       Method *m = (Method *)node;
+                       
+                       if(m->method == INIT_METHOD ||
+                          m->method == CLASS_INIT_METHOD ||
+                          m->method == OVERRIDE_METHOD)
+                               continue;
+
+                       /* in C++ mode don't alias new */
+                       if(for_cpp && strcmp(m->id, "new")==0)
                                continue;
 
-                       print_method(out,"static ","(* ",") ","",m,TRUE);
-                       out_printf(out," = %s_%s;\n",funcbase,m->id);
+                       print_method(out, "static ", "(* ", "", ") ", "",
+                                    m, FALSE, TRUE);
+                       out_printf(out, " = %s_%s;\n", funcbase, m->id);
 
                        made_aliases = TRUE;
                }
        }
-       out_printf(out,"\n");
 }
 
 static void
@@ -289,23 +308,25 @@ add_bad_hack_to_avoid_unused_warnings(Class *c)
        if(!made_aliases)
                return;
        
-       out_printf(out,"\n\n/*REALLY BAD HACK\n"
+       if(!no_gnu)
+               out_printf(out,"\n\n#ifndef __GNUC__\n");
+       out_printf(out,
+                  "/*REALLY BAD HACK\n"
                   "  This is to avoid unused warnings if you don't call\n"
-                  "  some method, it pollutes the namespace but the call\n"
-                  "  is weird enough.  I need to find a better way to do\n"
-                  "  this */\n");
-       out_printf(out,"void __%s_really_bad_hack_to_avoid_warnings(void);\n",
-                  funcbase);
-       out_printf(out,"void\n__%s_really_bad_hack_to_avoid_warnings(void)\n"
+                  "  some method.  I need to find a better way to do\n"
+                  "  this, not needed in GCC since we use some gcc\n"
+                  "  extentions to make saner, faster code */\n"
+                  "static void\n"
+                  "___%s_really_bad_hack_to_avoid_warnings(void)\n"
                   "{\n",funcbase);
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *node = li->data;
                if(node->type == METHOD_NODE) {
                        Method *m = (Method *)node;
                        
-                       if(m->scope == INIT_METHOD ||
-                          m->scope == CLASS_INIT_METHOD ||
-                          m->scope == OVERRIDE_METHOD)
+                       if(m->method == INIT_METHOD ||
+                          m->method == CLASS_INIT_METHOD ||
+                          m->method == OVERRIDE_METHOD)
                                continue;
 
                        /* in C++ mode we don't alias new */
@@ -315,7 +336,12 @@ add_bad_hack_to_avoid_unused_warnings(Class *c)
                        out_printf(out,"\t((void (*)(void))%s)();\n",m->id);
                }
        }
-       out_printf(out,"}\n\n");
+       out_printf(out, "\t___%s_really_bad_hack_to_avoid_warnings();\n",
+                  funcbase);
+       if(!no_gnu)
+               out_printf(out,"}\n#endif /* __GNUC__ */\n\n");
+       else
+               out_printf(out,"}\n\n");
 }
 
 static void
@@ -323,61 +349,109 @@ put_variable(Variable *v, FILE *fp)
 {
        out_printf(fp,"\t");
        print_type(fp,v->vtype,FALSE);
-       out_printf(fp,"%s%s;\n",v->id,
+       out_printf(fp,"%s%s;",v->id,
                   v->vtype->postfix?
                   v->vtype->postfix:""); 
+       if(v->scope == PROTECTED_SCOPE)
+               out_printf(fp," /* protected */");
+       out_printf(fp,"\n");
 }
 
 static void
 put_vs_method(Method *m)
 {
-       if(m->scope != SIGNAL_LAST_METHOD &&
-          m->scope != SIGNAL_FIRST_METHOD &&
-          m->scope != PRIVATE_SIGNAL_LAST_METHOD &&
-          m->scope != PRIVATE_SIGNAL_FIRST_METHOD &&
-          m->scope != VIRTUAL_METHOD &&
-          m->scope != PRIVATE_VIRTUAL_METHOD)
+       if(m->method != SIGNAL_LAST_METHOD &&
+          m->method != SIGNAL_FIRST_METHOD &&
+          m->method != VIRTUAL_METHOD)
                return;
 
-       print_method(outh,"\t","(* ",") ",";\n",m,TRUE);
+       /* if a signal mark it as such */
+       if(m->method != VIRTUAL_METHOD)
+               print_method(outh,"\t/*signal*/","(* ","",") ",";\n",m,
+                            FALSE,TRUE);
+       else
+               print_method(outh,"\t","(* ","",") ",";\n",m,FALSE,TRUE);
 }
 
 static void
 put_pub_method(Method *m)
 {
-       if(m->scope == PRIVATE_SCOPE ||
-          m->scope == OVERRIDE_METHOD ||
-          m->scope == INIT_METHOD ||
-          m->scope == CLASS_INIT_METHOD ||
-          m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-          m->scope == PRIVATE_SIGNAL_FIRST_METHOD ||
-          m->scope == PRIVATE_VIRTUAL_METHOD)
+       if(m->scope != PUBLIC_SCOPE)
                return;
 
-       print_method(outh,"","\t","\t",";\n",m,FALSE);
+       print_method(outh,"","\t","","\t",";\n",m,TRUE,FALSE);
 }
 
+/* I'm starting not to like this idea */
+#if 0
 static void
-put_priv_method_prot(Method *m)
+put_signal_connect(Method *m)
 {
-       if(m->scope == PUBLIC_SCOPE)
+       if(m->method != SIGNAL_LAST_METHOD &&
+          m->method != SIGNAL_FIRST_METHOD)
                return;
 
-       if(m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-          m->scope == PRIVATE_SIGNAL_FIRST_METHOD ||
-          m->scope == PRIVATE_VIRTUAL_METHOD) {
-               if(m->cbuf && *m->cbuf->str)
-                       print_method(out,"static ","_real_"," ",";\n",m,FALSE);
-               print_method(out,"static ",""," ",";\n",m,FALSE);
-       } else if(m->scope == SIGNAL_LAST_METHOD ||
-                 m->scope == SIGNAL_FIRST_METHOD ||
-                 m->scope == VIRTUAL_METHOD) {
-               if(!m->cbuf || !*m->cbuf->str)
-                       return;
-               print_method(out,"static ","_real_"," ",";\n",m,FALSE);
-       } else {
-               print_method(out,"static ",""," ",";\n",m,FALSE);
+       out_printf(outh, "guint \t%s_%s__connect_full\t(%s *object,\n"
+                  "\t\t\t\t\tconst char *name,\n"
+                  "\t\t\t\t\tGtkSignalFunc func,\n"
+                  "\t\t\t\t\tGtkCallbackMarshal marshal,\n"
+                  "\t\t\t\t\tgpointer data,\n"
+                  "\t\t\t\t\tGtkDestroyNotify destroy_func,\n"
+                  "\t\t\t\t\tgboolean object_signal,\n"
+                  "\t\t\t\t\tgboolean after);\n",
+                  funcbase, m->id, typebase);
+
+       out_printf(outh, "#define %s_%s__connect(object,name,func,data) "
+                  "%s_%s__connect_full((object),(name),(func),NULL,"
+                  "(data),NULL,FALSE,FALSE)\n",
+                  funcbase, m->id, funcbase, m->id);
+       out_printf(outh, "#define %s_%s__connect_after(object,name,func,data) "
+                  "%s__connect_%s_full((object),(name),(func),NULL,"
+                  "(data),NULL,FALSE,TRUE)\n",
+                  funcbase, m->id, funcbase, m->id);
+
+       out_printf(outh, "guint \t%s_%s__connect_while_alive\t(%s *object,\n"
+                  "\t\t\t\t\tconst char *name,\n"
+                  "\t\t\t\t\tGtkSignalFunc func,\n"
+                  "\t\t\t\t\tgpointer data,\n"
+                  "\t\t\t\t\tGtkObject *alive_object);\n\n",
+                  funcbase, m->id, typebase);
+}
+#endif
+
+
+static void
+put_prot_method(Method *m)
+{
+       if(m->scope != PROTECTED_SCOPE)
+               return;
+
+       if(outph)
+               print_method(outph,"","\t","","\t",";\n",m,FALSE,FALSE);
+       else
+               print_method(out,"","\t","","\t",";\n",m,FALSE,FALSE);
+}
+
+static void
+put_priv_method_prot(Method *m)
+{
+       if(m->method == SIGNAL_LAST_METHOD ||
+          m->method == SIGNAL_FIRST_METHOD ||
+          m->method == VIRTUAL_METHOD) {
+               if(m->cbuf)
+                       print_method(out,
+                                    "static ", "___real_", "", " ", ";\n",
+                                    m, FALSE, FALSE);
        }
+
+       if(m->scope == PRIVATE_SCOPE ||
+          m->method == INIT_METHOD ||
+          m->method == CLASS_INIT_METHOD ||
+          (m->method == OVERRIDE_METHOD &&
+           m->cbuf))
+               print_method(out, "static ", "", "", " ",
+                            no_gnu?";\n":" G_GNUC_UNUSED;\n",
+                            m, FALSE, FALSE);
 }
 
 static GList *
@@ -408,11 +482,11 @@ make_inits(Class *cl)
                Node *n = li->data;
                if(n->type == METHOD_NODE) {
                        Method *m = (Method *)n;
-                       if(m->scope == INIT_METHOD) {
+                       if(m->method == INIT_METHOD) {
                                if(got_init)
                                        print_error(FALSE,"init defined more then once",m->line_no);
                                got_init = TRUE;
-                       } else if(m->scope == CLASS_INIT_METHOD) {
+                       } else if(m->method == CLASS_INIT_METHOD) {
                                if(got_class_init)
                                        print_error(FALSE,"class_init defined more then once",m->line_no);
                                got_class_init = TRUE;
@@ -420,160 +494,183 @@ make_inits(Class *cl)
                }
        }
        if(!got_class_init) {
-               node = new_method(CLASS_INIT_METHOD,
-                                 (Type *)new_type(0,g_strdup("void"),NULL),
-                                 NULL,NULL,g_strdup("class_init"),
-                                 make_func_arg(cl->otype,TRUE,g_strdup("c")),
-                                 NULL, NULL,0,0,FALSE);
+               node = new_method(NO_SCOPE, CLASS_INIT_METHOD,
+                                 (Type *)new_type(0, g_strdup("void"),NULL),
+                                 NULL, NULL, NULL, g_strdup("class_init"),
+                                 make_func_arg(cl->otype, TRUE, g_strdup("c")),
+                                 NULL, NULL, NULL, 0, 0, FALSE);
                cl->nodes = g_list_prepend(cl->nodes,node);
        }
        if(!got_init) {
-               node = new_method(INIT_METHOD,
-                                 (Type *)new_type(0,g_strdup("void"),NULL),
-                                 NULL,NULL,g_strdup("init"),
-                                 make_func_arg(cl->otype,FALSE,g_strdup("o")),
-                                 NULL, NULL,0,0,FALSE);
+               node = new_method(NO_SCOPE, INIT_METHOD,
+                                 (Type *)new_type(0, g_strdup("void"),NULL),
+                                 NULL, NULL, NULL, g_strdup("init"),
+                                 make_func_arg(cl->otype, FALSE, g_strdup("o")),
+                                 NULL, NULL, NULL, 0, 0, FALSE);
                cl->nodes = g_list_prepend(cl->nodes,node);
        }
 }
 
 static void
-make_destroy(Class *cl)
+make_finalize(Class *cl)
 {
-       int got_destroy = FALSE;
+       int got_finalize = FALSE;
        GList *li;
        Node *node;
        for(li=cl->nodes;li;li=g_list_next(li)) {
                Node *n = li->data;
                if(n->type == METHOD_NODE) {
                        Method *m = (Method *)n;
-                       if(m->scope == OVERRIDE_METHOD &&
-                          strcmp(m->id,"destroy")==0) {
-                               if(strcmp(m->otype,"Gtk:Object")==0) {
-                                       got_destroy = TRUE;
+                       if(m->method == OVERRIDE_METHOD &&
+                          strcmp(m->id, "finalize")==0) {
+                               if(strcmp(m->otype, "Gtk:Object")==0) {
+                                       got_finalize = TRUE;
                                        break;
-                               } else {
-                                       print_error(FALSE,"destroy method override "
-                                                   "of class other then Gtk:Object",
-                                                   m->line_no);
                                }
-
+                               print_error(FALSE,"finalize method override "
+                                           "of class other then Gtk:Object",
+                                           m->line_no);
                        }
                }
        }
-       if(!got_destroy) {
-               node = new_method(OVERRIDE_METHOD,
+       if(!got_finalize) {
+               node = new_method(NO_SCOPE, OVERRIDE_METHOD,
                                  (Type *)new_type(0,g_strdup("void"),NULL),
                                  g_strdup("Gtk:Object"),
-                                 NULL,g_strdup("destroy"),
+                                 NULL, NULL, g_strdup("finalize"),
                                  make_func_arg("Gtk:Object",FALSE,g_strdup("o")),
-                                 NULL,
-                                 g_string_new("PARENT_HANDLER (o);\n"),
+                                 NULL, NULL,
+                                 g_strdup("PARENT_HANDLER (o);\n"),
                                  0,0,FALSE);
                cl->nodes = g_list_append(cl->nodes,node);
                overrides++;
        }
 }
 
+
+/* hash of method -> name of signal prototype */
 static GHashTable *marsh = NULL;
 
+/* list of methods with different signal prototypes,
+   we check this list if we can use a signal prototype of a
+   previous signal method, there are only uniques here */
+static GList *eq_signal_methods = NULL;
+
+/* compare a list of strings */
+static gboolean
+is_list_equal(GList *a, GList *b)
+{
+       for(;a && b; a=a->next, b=b->next) {
+               if(strcmp(a->data,b->data)!=0) {
+                       return FALSE;
+               }
+       }
+       /* the the lists were different length */
+       if(a || b)
+               return FALSE;
+       return TRUE;
+}
+
+static Method *
+find_same_type_signal(Method *m)
+{
+       GList *li;
+       for(li=eq_signal_methods;li;li=li->next) {
+               Method *mm = li->data;
+               if(is_list_equal(mm->gtktypes,m->gtktypes))
+                       return mm;
+       }
+       return NULL;
+}
+
+static void
+print_signal_marsal_args(Method *m)
+{
+       if(strcmp(m->gtktypes->next->data, "NONE")!=0) {
+               GList *li;
+               int i;
+               for(i=0,li=m->gtktypes->next;li;
+                   i++,li=g_list_next(li)) {
+                       if(!for_cpp)
+                               out_printf(out, ",\n\t\tGTK_VALUE_%s(args[%d])",
+                                          (char *)li->data,i);
+                       else {
+                               out_printf(out, ",\n\t\t(%s)"
+                                          "GTK_VALUE_%s(args[%d])",
+                                          get_cast(li->data, FALSE),
+                                          (char *)li->data,i);
+                       }
+               }
+       }
+       out_printf(out, ",\n\t\tfunc_data);\n}\n\n");
+}
+
+
 static void
 add_signal_prots(Method *m)
 {
        GList *li;
        static int sig = 1;
        char *s;
+       Method *mm;
        
-       if(m->scope != SIGNAL_LAST_METHOD &&
-          m->scope != SIGNAL_FIRST_METHOD &&
-          m->scope != PRIVATE_SIGNAL_LAST_METHOD &&
-          m->scope != PRIVATE_SIGNAL_FIRST_METHOD)
+       if(m->method != SIGNAL_LAST_METHOD &&
+          m->method != SIGNAL_FIRST_METHOD)
                return;
 
        if(!marsh)
                marsh = g_hash_table_new(NULL,NULL);
        
-       if(strcmp(m->gtktypes->data,"NONE")==0 &&
-          strcmp(m->gtktypes->next->data,"NONE")==0)
+       if(strcmp(m->gtktypes->data, "NONE")==0 &&
+          strcmp(m->gtktypes->next->data, "NONE")==0)
                return;
+
+       /* if we already did a signal prototype just use that */
+       mm = find_same_type_signal(m);
+       if(mm) {
+               s = g_hash_table_lookup(marsh,mm);
+               g_hash_table_insert(marsh,m,s);
+               return;
+       }
        
-       s = g_strdup_printf("__Sig%d",sig++);
-       
-       g_hash_table_insert(marsh,m,s);
+       s = g_strdup_printf("Sig%d",sig++);
        
-       out_printf(out,"\ntypedef ");
-       print_type(out,m->mtype,TRUE);
+       g_hash_table_insert(marsh, m, s);
+       eq_signal_methods = g_list_prepend(eq_signal_methods, m);
        
-       out_printf(out,"(*%s) (",s);
+       /* we know that we'll know all the gtktypes (so get_cast can't fail) */
+       out_printf(out, "\ntypedef %s (*___%s) (%s *, ",
+                  get_cast(m->gtktypes->data, FALSE), s, typebase);
        
-       for(li=m->args;li;li=g_list_next(li)) {
-               FuncArg *arg = li->data;
-               print_type(out,arg->atype,TRUE);
-               out_printf(out,", ");
+       if(strcmp(m->gtktypes->next->data, "NONE")!=0) {
+               for(li=m->gtktypes->next; li; li=g_list_next(li))
+                       out_printf(out, "%s, ", get_cast(li->data, FALSE));
        }
-       out_printf(out,"gpointer);\n"); 
+       out_printf(out, "gpointer);\n"); 
        
        out_printf(out,"\nstatic void\n"
-               "marshal_%s (GtkObject * object,\n"
+               "___marshal_%s (GtkObject * object,\n"
                "\tGtkSignalFunc func,\n"
                "\tgpointer func_data,\n"
                "\tGtkArg * args)\n"
                "{\n",s);
        
-       if(strcmp(m->gtktypes->data,"NONE")==0) {
-               int i;
-               out_printf(out, "\t%s rfunc;\n\n"
-                       "\trfunc = (%s)func;\n\n"
-                       "\t(*rfunc)((%s *)object",s,s,typebase);
-               if(strcmp(m->gtktypes->next->data,"NONE")!=0) {
-                       GList *ali = m->args->next;
-                       for(i=0,li=m->gtktypes->next;li;
-                           i++,li=g_list_next(li)) {
-                               if(!for_cpp || !ali)
-                                       out_printf(out, ",\n\t\tGTK_VALUE_%s(args[%d])",
-                                                  (char *)li->data,i);
-                               else {
-                                       FuncArg *fa = ali->data;
-                                       out_printf(out, ",\n\t\t(");
-                                       print_type(out,fa->atype,TRUE);
-                                       out_printf(out, ")GTK_VALUE_%s(args[%d])",
-                                                  (char *)li->data,i);
-                               }
-
-                               if(ali) ali = ali->next;
-                       }
-               }
-               out_printf(out, ",\n\t\tfunc_data);\n}\n\n");
+       if(strcmp(m->gtktypes->data, "NONE")==0) {
+               out_printf(out, "\t___%s rfunc;\n\n"
+                       "\trfunc = (___%s)func;\n\n"
+                       "\t(*rfunc)((%s *)object", s, s, typebase);
        } else {
-               int i;
-               out_printf(out, "\t%s rfunc;\n\t",s);
-               print_type(out,m->mtype,TRUE);
-               out_printf(out, " *retval;\n\n"
-                       "\trfunc = (%s)func;\n\n"
-                       "\tretval = GTK_RETLOC_%s(args[%d]);\n\n"
-                       "\t*retval = (*rfunc)((%s *)object",
-                       s,(char *)m->gtktypes->data,
-                       g_list_length(m->gtktypes)-1,typebase);
-               if(strcmp(m->gtktypes->next->data,"NONE")!=0) {
-                       GList *ali = m->args->next;
-                       for(i=0,li=m->gtktypes->next;li;
-                           i++,li=g_list_next(li)) {
-                               if(!for_cpp || !ali)
-                                       out_printf(out, ",\n\t\tGTK_VALUE_%s(args[%d])",
-                                                  (char *)li->data,i);
-                               else {
-                                       FuncArg *fa = ali->data;
-                                       out_printf(out, ",\n\t\t(");
-                                       print_type(out,fa->atype,TRUE);
-                                       out_printf(out, ")GTK_VALUE_%s(args[%d])",
-                                                  (char *)li->data,i);
-                               }
-
-                               if(ali) ali = ali->next;
-                       }
-               }
-               out_printf(out, ",\n\t\tfunc_data);\n}\n\n");
+               const char *retcast = get_cast(m->gtktypes->data, FALSE);
+               out_printf(out,
+                          "\t___%s rfunc;\n\t"
+                          "%s *retval;\n\n"
+                          "\trfunc = (___%s)func;\n\n"
+                          "\tretval = GTK_RETLOC_%s(args[%d]);\n\n"
+                          "\t*retval = (*rfunc)((%s *)object",
+                          s, retcast, s, (char *)m->gtktypes->data,
+                          g_list_length(m->gtktypes)-1, typebase);
        }
+       print_signal_marsal_args(m);
 
 }
 
@@ -588,10 +685,8 @@ add_enums(Class *c)
                        Node *n = li->data;
                        if(n->type == METHOD_NODE) {
                                Method *m = (Method *)n;
-                               if(m->scope == SIGNAL_LAST_METHOD ||
-                                  m->scope == SIGNAL_FIRST_METHOD ||
-                                  m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-                                  m->scope == PRIVATE_SIGNAL_FIRST_METHOD) {
+                               if(m->method == SIGNAL_LAST_METHOD ||
+                                  m->method == SIGNAL_FIRST_METHOD) {
                                        char *s = g_strdup(m->id);
                                        g_strup(s);
                                        out_printf(out,"\t%s_SIGNAL,\n",s);
@@ -602,24 +697,25 @@ add_enums(Class *c)
                out_printf(out,"\tLAST_SIGNAL\n};\n\n");
        }
        if(arguments>0) {
-               out_printf(out,"enum {\n\tARG_0,\n");
+               out_printf(out,"enum {\n\tARG_0");
                for(li=c->nodes;li;li=g_list_next(li)) {
                        Node *n = li->data;
                        if(n->type == ARGUMENT_NODE) {
                                Argument *a = (Argument *)n;
                                char *s = g_strdup(a->name);
                                g_strup(s);
-                               out_printf(out,"\tARG_%s,\n",s);
+                               out_printf(out,",\n\tARG_%s",s);
                                g_free(s);
                        }
                }
-               out_printf(out, "};\n\n");
+               out_printf(out, "\n};\n\n");
        }
 
        if(signals>0)
                out_printf(out,
                           "static guint object_signals[LAST_SIGNAL] = {0};\n\n");
 
+       out_printf(out, "/* pointer to the class of our parent */\n");
        out_printf(out, "static %sClass *parent_class = NULL;\n\n",ptypebase);
 }
 
@@ -656,19 +752,18 @@ add_overrides(Class *c, char *oname, gboolean did_gtk_obj)
        GHashTable *done;
        char *s;
        
-       done = g_hash_table_new(g_str_hash,g_str_equal);
+       done = g_hash_table_new(g_str_hash, g_str_equal);
        if(did_gtk_obj) {
                s = g_strdup("GtkObject"); /* This was already done */
-               g_hash_table_insert(done,s,s);
+               g_hash_table_insert(done, s, s);
        }
-       for(li=c->nodes;li;li=g_list_next(li)) {
+       for(li=c->nodes; li; li=g_list_next(li)) {
                Node *n = li->data;
                char *f;
-               Method *m;
+               Method *m = (Method *)n;
                if(n->type != METHOD_NODE ||
-                  ((Method *)n)->scope != OVERRIDE_METHOD)
+                  m->method != OVERRIDE_METHOD)
                        continue;
-               m = (Method *)n;
                
                s = remove_sep(m->otype);
                
@@ -690,6 +785,58 @@ add_overrides(Class *c, char *oname, gboolean did_gtk_obj)
        g_hash_table_destroy(done);
 }
 
+static char *
+make_run_signal_flags(Method *m, gboolean last)
+{
+       GList *li;
+       GString *gs;
+       char *flags[] = {
+               "FIRST",
+               "LAST",
+               "BOTH",
+               "NO_RECURSE",
+               "ACTION",
+               "NO_HOOKS",
+               NULL
+       };
+
+       gs = g_string_new(NULL);
+
+       if(last)
+               g_string_assign(gs, "GTK_RUN_LAST");
+       else
+               g_string_assign(gs, "GTK_RUN_FIRST");
+
+       if(m->scope == PUBLIC_SCOPE)
+               g_string_append(gs, " | GTK_RUN_ACTION");
+
+       for(li = m->flags; li; li = li->next) {
+               char *flag = li->data;
+               int i;
+               for(i=0;flags[i];i++) {
+                       if(strcmp(flags[i],flag)==0)
+                               break;
+               }
+               /* if we haven't found it in our list */
+               if(!flags[i]) {
+                       char *s;
+                       s = g_strdup_printf("Unknown flag '%s' used, "
+                                           "perhaps it was misspelled",
+                                           flag);
+                       print_error(TRUE, s, m->line_no);
+                       g_free(s);
+               }
+               g_string_sprintfa(gs, " | GTK_RUN_%s",flag);
+       }
+
+       {
+               char *ret = gs->str;
+               g_string_free(gs, FALSE);
+               return ret;
+       }
+}
+               
+
 static void
 add_signals(Class *c)
 {
@@ -698,50 +845,46 @@ add_signals(Class *c)
        out_printf(out,"\n");
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *n = li->data;
-               Method *m;
-               char *mar;
-               char *sig;
-               int is_none;
-               int last = FALSE;
+               char *mar, *sig, *flags;
+               gboolean is_none, last = FALSE;
+               Method *m = (Method *)n;
+
                if(n->type != METHOD_NODE ||
-                  (((Method *)n)->scope != SIGNAL_FIRST_METHOD &&
-                   ((Method *)n)->scope != SIGNAL_LAST_METHOD &&
-                   ((Method *)n)->scope != PRIVATE_SIGNAL_FIRST_METHOD &&
-                   ((Method *)n)->scope != PRIVATE_SIGNAL_LAST_METHOD))
+                  (m->method != SIGNAL_FIRST_METHOD &&
+                   m->method != SIGNAL_LAST_METHOD))
                        continue;
 
-               m = (Method *)n;
-               
-               if(m->scope == SIGNAL_FIRST_METHOD ||
-                  m->scope == PRIVATE_SIGNAL_FIRST_METHOD)
+               if(m->method == SIGNAL_FIRST_METHOD)
                        last = FALSE;
                else
                        last = TRUE;
 
-               if(g_hash_table_lookup(marsh,m))
-                       mar = g_strconcat("marshal_",
+               if(g_hash_table_lookup(marsh, m))
+                       mar = g_strconcat("___marshal_",
                                          (char *)g_hash_table_lookup(marsh,m),
                                          NULL);
                else
                        mar = g_strdup("gtk_signal_default_marshaller");
                
-               is_none = (strcmp(m->gtktypes->next->data,"NONE")==0);
+               is_none = (strcmp(m->gtktypes->next->data, "NONE")==0);
                
                sig = g_strdup(m->id);
                g_strup(sig);
+               flags = make_run_signal_flags(m, last);
                out_printf(out,"\tobject_signals[%s_SIGNAL] =\n"
                        "\t\tgtk_signal_new (\"%s\",\n"
-                       "\t\t\tGTK_RUN_%s,\n"
+                       "\t\t\t(GtkSignalRunType)(%s),\n"
                        "\t\t\tgtk_object_class->type,\n"
                        "\t\t\tGTK_SIGNAL_OFFSET (%sClass, %s),\n"
                        "\t\t\t%s,\n"
                        "\t\t\tGTK_TYPE_%s, %d",
                        sig,m->id,
-                       last?"LAST":"FIRST",
+                       flags,
                        typebase,m->id,mar,(char *)m->gtktypes->data,
                        is_none?0:g_list_length(m->gtktypes->next));
                g_free(mar);
                g_free(sig);
+               g_free(flags);
                
                if(!is_none) {
                        GList *l;
@@ -760,44 +903,61 @@ static void
 set_def_handlers(Class *c, char *oname)
 {
        GList *li;
+       gboolean set_line = FALSE;
 
        out_printf(out,"\n");
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *n = li->data;
-               Method *m;
+               Method *m = (Method *)n;
                if(n->type != METHOD_NODE ||
-                  (((Method *)n)->scope != SIGNAL_FIRST_METHOD &&
-                   ((Method *)n)->scope != SIGNAL_LAST_METHOD &&
-                   ((Method *)n)->scope != PRIVATE_SIGNAL_FIRST_METHOD &&
-                   ((Method *)n)->scope != PRIVATE_SIGNAL_LAST_METHOD &&
-                   ((Method *)n)->scope != VIRTUAL_METHOD &&
-                   ((Method *)n)->scope != PRIVATE_VIRTUAL_METHOD &&
-                   ((Method *)n)->scope != OVERRIDE_METHOD))
+                  (m->method != SIGNAL_FIRST_METHOD &&
+                   m->method != SIGNAL_LAST_METHOD &&
+                   m->method != VIRTUAL_METHOD &&
+                   m->method != OVERRIDE_METHOD))
                        continue;
 
-               m = (Method *)n;
+               if(m->line_no > 0 && m->cbuf) {
+                       out_addline_infile(out, m->line_no);
+                       set_line = TRUE;
+               } else if(set_line) {
+                       out_addline_outfile(out);
+                       set_line = FALSE;
+               }
 
-               if(m->scope == OVERRIDE_METHOD) {
+               if(m->method == OVERRIDE_METHOD) {
                        char *s;
-                       s = replace_sep(m->otype,'_');
+                       s = replace_sep(m->otype, '_');
                        g_strdown(s);
-                       out_printf(out,"\t%s_class->%s = %s_%s;\n",
-                               s,m->id,funcbase,m->id);
+                       if(m->cbuf)
+                               out_printf(out,"\t%s_class->%s = %s_%s;\n",
+                                          s, m->id, funcbase, m->id);
+                       else
+                               out_printf(out,"\t%s_class->%s = NULL;\n",
+                                          s, m->id);
                } else {
-                       if(m->cbuf && *m->cbuf->str)
-                               out_printf(out,"\t%s->%s = _real_%s_%s;\n",
-                                       oname,m->id,funcbase,m->id);
+                       if(m->cbuf)
+                               out_printf(out,"\t%s->%s = ___real_%s_%s;\n",
+                                       oname, m->id, funcbase, m->id);
                        else
                                out_printf(out,"\t%s->%s = NULL;\n",
-                                       oname,m->id);
+                                       oname, m->id);
                }
        }
+       if(set_line)
+               out_addline_outfile(out);
 }
 
 static void
 make_arguments(Class *c)
 {
        GList *li;
+       char *argflags[] = {
+               "CONSTRUCT",
+               "CONSTRUCT_ONLY",
+               "CHILD_ARG",
+               "MASK",
+               NULL
+       };
 
        out_printf(out,"\n");
        for(li=c->nodes;li;li=g_list_next(li)) {
@@ -818,8 +978,32 @@ make_arguments(Class *c)
                else
                        flags = g_string_new("GTK_ARG_WRITABLE");
                
-               for(l=a->flags;l;l=g_list_next(l))
-                       g_string_sprintfa(flags," | GTK_ARG_%s",(char *)l->data);
+               for(l=a->flags;l;l=g_list_next(l)) {
+                       char *flag = l->data;
+                       int i;
+                       if(strcmp(flag,"READWRITE")==0 ||
+                          strcmp(flag,"READABLE")==0 ||
+                          strcmp(flag,"WRITABLE")==0) {
+                               print_error(TRUE,"READWRITE, READABLE and "
+                                           "WRITABLE argument flags are "
+                                           "set automatically",a->line_no);
+                               continue;
+                       }
+                       for(i=0;argflags[i];i++) {
+                               if(strcmp(argflags[i],flag)==0)
+                                       break;
+                       }
+                       /* if we haven't found it in our list */
+                       if(!argflags[i]) {
+                               char *s;
+                               s = g_strdup_printf("Unknown flag '%s' used, "
+                                                   "perhaps it was misspelled",
+                                                   flag);
+                               print_error(TRUE,s,a->line_no);
+                               g_free(s);
+                       }
+                       g_string_sprintfa(flags, " | GTK_ARG_%s",flag);
+               }
 
                s = g_strdup(a->name);
                g_strup(s);
@@ -827,14 +1011,40 @@ make_arguments(Class *c)
                        "\t\tGTK_TYPE_%s,\n"
                        "\t\t%s,\n"
                        "\t\tARG_%s);\n",
-                       typebase,a->name,a->gtktype,flags->str,s);
+                       typebase, a->name, a->gtktype, flags->str, s);
                g_free(s);
-               g_string_free(flags,TRUE);
+               g_string_free(flags, TRUE);
        }
        
        out_printf(out,
-                  "\n\tgtk_object_class->set_arg = __object_set_arg;\n"
-                  "\tgtk_object_class->get_arg = __object_get_arg;\n");
+                  "\n\tgtk_object_class->set_arg = ___object_set_arg;\n"
+                  "\tgtk_object_class->get_arg = ___object_get_arg;\n");
+}
+
+static void
+print_initializer(Method *m, Variable *v)
+{
+       char *root;
+
+       if(v->initializer == NULL)
+               return;
+
+       if(v->scope == PRIVATE_SCOPE)
+               root = g_strconcat(((FuncArg *)m->args->data)->name,
+                                  "->_priv", NULL);
+       else
+               root = g_strdup(((FuncArg *)m->args->data)->name);
+
+       if(v->initializer_line > 0)
+               out_addline_infile(out, v->initializer_line);
+
+       out_printf(out, "\t%s->%s = %s;\n",
+                  root, v->id, v->initializer);
+
+       if(v->initializer_line > 0)
+               out_addline_outfile(out);
+
+       g_free(root);
 }
 
 static void
@@ -847,24 +1057,37 @@ add_inits(Class *c)
                if(n->type != METHOD_NODE)
                        continue;
                m = (Method *)n;
-               if(m->scope == INIT_METHOD) {
-                       if(m->line_no>0)
+               if(m->method == INIT_METHOD) {
+                       if(m->line_no > 0)
                                out_addline_infile(out,m->line_no);
-                       print_method(out,"static ","\n"," ","\n",m,FALSE);
-                       if(m->line_no>0)
+                       print_method(out, "static ", "\n", "", " ", "\n",
+                                    m, FALSE, FALSE);
+                       if(m->line_no > 0)
                                out_addline_outfile(out);
                        out_printf(out,"{\n");
-                       if(privates>0) {
+                       if(privates > 0) {
                                out_printf(out,"\t%s->_priv = "
-                                          "g_new0 (%sPrivate,1);\n",
+                                          "g_new0 (%sPrivate, 1);\n",
                                           ((FuncArg *)m->args->data)->name,
                                           typebase);
                        }
-               } else if(m->scope == CLASS_INIT_METHOD) {
-                       if(m->line_no>0)
-                               out_addline_infile(out,m->line_no);
-                       print_method(out,"static ","\n"," ","\n",m,FALSE);
-                       if(m->line_no>0)
+                       if(initializers > 0) {
+                               GList *li;
+                               for(li = ((Class *)class)->nodes;
+                                   li != NULL;
+                                   li = li->next) {
+                                       Node *n = li->data;
+                                       if(n->type != VARIABLE_NODE)
+                                               continue;
+                                       print_initializer(m, (Variable *)n);
+                               }
+                       }
+               } else if(m->method == CLASS_INIT_METHOD) {
+                       if(m->line_no > 0)
+                               out_addline_infile(out, m->line_no);
+                       print_method(out, "static ", "\n", "", " ", "\n",
+                                    m, FALSE, FALSE);
+                       if(m->line_no > 0)
                                out_addline_outfile(out);
                        out_printf(out,"{\n");
                        if(signals>0 ||
@@ -897,12 +1120,12 @@ add_inits(Class *c)
                } else
                        continue;
 
-               if(m->cbuf && *m->cbuf->str) {
+               if(m->cbuf) {
                        out_printf(out," {\n");
                        out_addline_infile(out,m->ccode_line);
-                       out_printf(out,"%s\n",m->cbuf->str);
+                       out_printf(out,"%s\n",m->cbuf);
                        out_addline_outfile(out);
-                       out_printf(out," }\n",m->cbuf->str);
+                       out_printf(out," }\n");
                } else {
                        out_printf(out,"return;\n");
                }
@@ -911,11 +1134,11 @@ add_inits(Class *c)
 }
 
 static void
-add_getset_arg(Class *c, int is_set)
+add_getset_arg(Class *c, gboolean is_set)
 {
        GList *li;
        out_printf(out,"\nstatic void\n"
-               "__object_%s_arg (GtkObject *object,\n"
+               "___object_%s_arg (GtkObject *object,\n"
                "\tGtkArg *arg,\n"
                "\tguint arg_id)\n"
                "{\n"
@@ -928,7 +1151,7 @@ add_getset_arg(Class *c, int is_set)
                Node *n = li->data;
                Argument *a;
                char *s;
-               GString *cbuf;
+               char *cbuf;
                int line_no;
                if(n->type != ARGUMENT_NODE)
                        continue;
@@ -944,14 +1167,52 @@ add_getset_arg(Class *c, int is_set)
                        continue;
                s = g_strdup(a->name);
                g_strup(s);
-               out_printf(out,"\tcase ARG_%s:\n"
-                       "#define ARG (GTK_VALUE_%s(*arg))\n"
-                       "\t\t{\n",
-                       s,a->gtktype);
+               out_printf(out, "\tcase ARG_%s:\n", s);
+               if(is_set && a->atype) {
+                       char *cast = get_type(a->atype, TRUE);
+                       if(no_gnu || for_cpp) {
+                               out_printf(out, "#define ARG "
+                                          "((%s)GTK_VALUE_%s(*arg))\n",
+                                          cast, a->gtktype);
+                       } else {
+                               out_printf(out, "#ifdef __GNUC__\n");
+                               if(strcmp(a->gtktype, "OBJECT")==0) {
+                                       out_printf(out, "#define ARG "
+                                                  "({%s foo = "
+                                                  "GTK_VALUE_POINTER(*arg); "
+                                                  "foo; })\n",
+                                                  cast);
+                               } else {
+                                       out_printf(out, "#define ARG "
+                                                  "({%s foo = "
+                                                  "GTK_VALUE_%s(*arg); "
+                                                  "foo; })\n",
+                                                  cast, a->gtktype);
+                               }
+                               out_printf(out,"#else /* __GNUC__ */\n");
+                               out_printf(out, "#define ARG "
+                                          "((%s)GTK_VALUE_%s(*arg))\n",
+                                          cast, a->gtktype);
+                               out_printf(out,"#endif /* __GNUC__ */\n\n");
+                       }
+                       out_printf(out, "\t\t{\n");
+                       g_free(cast);
+               } else if(!is_set && strcmp(a->gtktype, "OBJECT")==0) {
+                       out_printf(out,
+                                  "#define ARG (GTK_VALUE_POINTER(*arg))\n"
+                                  "\t\t{\n");
+               } else {
+                       out_printf(out,
+                                  "#define ARG (GTK_VALUE_%s(*arg))\n"
+                                  "\t\t{\n",
+                                  a->gtktype);
+               }
                g_free(s);
-               out_addline_infile(out,line_no);
-               out_printf(out,"%s\n",cbuf->str);
-               out_addline_outfile(out);
+               if(line_no > 0)
+                       out_addline_infile(out, line_no);
+               out_printf(out,"%s\n",cbuf);
+               if(line_no > 0)
+                       out_addline_outfile(out);
                out_printf(out,"\t\t}\n\t\tbreak;\n"
                        "#undef ARG\n");
        }
@@ -962,24 +1223,34 @@ static void
 print_checks(Method *m, FuncArg *fa)
 {
        GList *li;
-       int is_void;
-       is_void = (strcmp(m->mtype->name,"void")==0 &&
+       gboolean is_void;
+       gboolean checked_null = FALSE;
+       is_void = (strcmp(m->mtype->name, "void")==0 &&
                   m->mtype->stars == 0);
        
        for(li=fa->checks;li;li=g_list_next(li)) {
                Check *ch = li->data;
                char *s;
+               /* point to the method prot in .gob for failed checks */
+               if(m->line_no > 0)
+                       out_addline_infile(out,m->line_no);
                if(is_void)
-                       out_printf(out,"\tg_return_if_fail (");
+                       out_printf(out, "\tg_return_if_fail (");
                else
-                       out_printf(out,"\tg_return_val_if_fail (");
+                       out_printf(out, "\tg_return_val_if_fail (");
                switch(ch->chtype) {
                case NULL_CHECK:
-                       out_printf(out,"%s != NULL",fa->name);
+                       out_printf(out, "%s != NULL", fa->name);
+                       checked_null = TRUE;
                        break;
                case TYPE_CHECK:
-                       s = make_pre_macro(fa->atype->name,"IS");
-                       out_printf(out,"%s (%s)",s,fa->name);
+                       s = make_pre_macro(fa->atype->name, "IS");
+                       if(checked_null)
+                               out_printf(out, "%s (%s)", s, fa->name);
+                       else
+                               /* if not check null, null may be valid */
+                               out_printf(out, "!(%s) || %s (%s)", fa->name,
+                                          s, fa->name);
                        g_free(s);
                        break;
                case LT_CHECK:
@@ -1002,11 +1273,11 @@ print_checks(Method *m, FuncArg *fa)
                        break;
                }
                if(is_void)
-                       out_printf(out,");\n");
+                       out_printf(out, ");\n");
                else {
-                       out_printf(out,", (");
-                       print_type(out,m->mtype,TRUE);
-                       out_printf(out,")%s);\n",
+                       out_printf(out, ", (");
+                       print_type(out, m->mtype, TRUE);
+                       out_printf(out, ")%s);\n",
                                m->onerror?m->onerror:"0");
                }
        }
@@ -1022,116 +1293,229 @@ print_preconditions(Method *m)
                if(fa->checks)
                        print_checks(m,fa);
        }
+       if(m->line_no>0)
+               out_addline_outfile(out);
 }
 
-/* put in code if it's needed */
 static void
-put_in_gen_code(Method *m)
+print_destructor(char *self_id, Variable *v)
 {
-       /* now we only have the freeing of the private structure */
-       if(privates>0 &&
-          m->scope == OVERRIDE_METHOD &&
-          strcmp(m->id,"destroy")==0) {
-               out_printf(out,"\tg_free (%s (%s)->_priv);\n"
-                          "\t%s (%s)->_priv = NULL;\n",
-                          macrobase,
-                          ((FuncArg *)m->args->data)->name,
-                          macrobase,
-                          ((FuncArg *)m->args->data)->name);
+       char *root;
 
-       }
-}
+       if(v->destructor == NULL)
+               return;
 
-static void
-print_method_body(Method *m, int pre)
-{
-       out_printf(out,"{\n");
-       if(pre)
+       if(v->scope == PRIVATE_SCOPE)
+               root = g_strconcat(self_id, "->_priv", NULL);
+       else
+               root = g_strdup(self_id);
+
+       if(v->destructor_simple) {
+               if(v->destructor_line > 0)
+                       out_addline_infile(out, v->destructor_line);
+
+               out_printf(out, "\tif(%s->%s) "
+                          "((*(void (*)(void *))%s)) (%s->%s);\n",
+                          root, v->id, v->destructor, root, v->id);
+
+               if(v->destructor_line > 0)
+                       out_addline_outfile(out);
+       } else {
+               out_printf(out, "#define VAR (%s->%s)\n", root, v->id);
+               out_printf(out, "\t{\n\t%s *self G_GNUC_UNUSED = %s;\n",
+                          typebase, self_id);
+               if(v->destructor_line > 0)
+                       out_addline_infile(out, v->destructor_line);
+
+               out_printf(out, "\t%s}\n", v->destructor);
+
+               if(v->destructor_line > 0)
+                       out_addline_outfile(out);
+               out_printf(out, "#undef VAR\n");
+       }
+
+       g_free(root);
+}
+
+/* put in code if it's needed */
+static void
+put_in_gen_code(Method *m)
+{
+       if(m->method == OVERRIDE_METHOD &&
+          strcmp(m->id, "finalize")==0) {
+               if(privates > 0 || destructors > 0) {
+                       out_printf(out,"\t%s *___self = %s (%s);\n",
+                                  typebase, macrobase,
+                                  ((FuncArg *)m->args->data)->name);
+               }
+               if(destructors > 0) {
+                       GList *li;
+                       for(li = ((Class *)class)->nodes;
+                           li != NULL;
+                           li = li->next) {
+                               Node *n = li->data;
+                               if(n->type == VARIABLE_NODE)
+                                       print_destructor("___self",
+                                                        (Variable *)n);
+                       }
+               }
+               if(privates > 0) {
+                       out_printf(out,"\tg_free (___self->_priv);\n"
+                                  "\t___self->_priv = NULL;\n",
+                                  macrobase,
+                                  ((FuncArg *)m->args->data)->name,
+                                  macrobase,
+                                  ((FuncArg *)m->args->data)->name);
+               }
+       }
+}
+
+static void
+print_method_body(Method *m, int pre)
+{
+       out_printf(out,"{\n");
+       if(pre)
                print_preconditions(m);
+
        put_in_gen_code(m);
 
-       out_printf(out,"\t{\n");
+       /* Note: the trailing }'s are on one line, this is so
+          that we get the no return warning correctly and point to
+          the correct line in the .gob file, yes this is slightly
+          ugly in the .c file, but that is not supposed to be
+          human readable anyway. */
+       if(m->cbuf) {
+               out_printf(out, "{\n");
+               if(m->ccode_line>0)
+                       out_addline_infile(out, m->ccode_line);
+               out_printf(out, "\t%s}", m->cbuf);
+       }
 
-       out_addline_infile(out,m->ccode_line);
-       out_printf(out,"\t\t%s\n",m->cbuf->str);
-       out_addline_outfile(out);
+       out_printf(out, "}\n");
 
-       out_printf(out,"\t}\n");
-       out_printf(out,"}\n");
+       if(m->cbuf)
+               out_addline_outfile(out);
+}
+
+static void
+put_signal_args(Method *m)
+{
+       GList *li;
+       GList *ali;
+       for(ali = m->gtktypes->next,li=m->args->next;
+           li && ali;
+           li=li->next, ali=ali->next) {
+               FuncArg *fa = li->data;
+               const char *cast = get_cast(ali->data, FALSE);
+               /* we should have already proved before that
+                  the we know all the types */
+               g_assert(cast);
+
+               out_printf(out,",\n\t\t(%s)%s",cast,
+                          fa->name);
+       }
+}
+
+static char *
+get_arg_names_for_macro(Method *m)
+{
+       char *p;
+       GList *li;
+       GString *gs = g_string_new(NULL);
+       p = "";
+       for(li=m->args;li;li=g_list_next(li)) {
+               FuncArg *arg = li->data;
+               g_string_sprintfa(gs, "%s___%s", p, arg->name);
+               p = ",";
+       }
+       p = gs->str;
+       g_string_free(gs, FALSE);
+       return p;
 }
 
 static void
 put_method(Method *m)
 {
-       char *s;
-       int private = FALSE;
-       int is_void;
+       char *s,*args,*doc;
+       gboolean is_void;
        is_void = (strcmp(m->mtype->name,"void")==0 &&
                   m->mtype->stars == 0);
        out_printf(out,"\n");
-       switch(m->scope) {
-       case PUBLIC_SCOPE:
-               out_addline_infile(out,m->line_no);
-               print_method(out,"","\n"," ","\n",m,FALSE);
-               print_method_body(m,TRUE);
-               break;
-       case PRIVATE_SCOPE:
-               out_addline_infile(out,m->line_no);
-               print_method(out,"static ","\n"," ","\n",m,FALSE);
-               print_method_body(m,TRUE);
+       doc = get_gtk_doc(m->id);
+       if(doc) {
+               out_printf(out, "%s", doc);
+               g_free(doc);
+       }
+       switch(m->method) {
+       case REGULAR_METHOD:
+               if(m->line_no>0)
+                       out_addline_infile(out,m->line_no);
+               if(m->scope == PRIVATE_SCOPE)
+                       print_method(out,"static ","\n",""," ","\n",
+                                    m,FALSE,FALSE);
+               else /* PUBLIC, PROTECTED */
+                       print_method(out, "", "\n", "", " ", "\n",
+                                    m, FALSE, FALSE);
+               print_method_body(m, TRUE);
                break;
-       case PRIVATE_SIGNAL_FIRST_METHOD:
-       case PRIVATE_SIGNAL_LAST_METHOD:
-               private = TRUE;
        case SIGNAL_FIRST_METHOD:
        case SIGNAL_LAST_METHOD:
-               out_addline_infile(out,m->line_no);
-               print_method(out,private?"static ":"","\n"," ","\n",m,FALSE);
+               if(m->line_no>0)
+                       out_addline_infile(out,m->line_no);
+               if(m->scope == PRIVATE_SCOPE)
+                       print_method(out,"static ","\n",""," ","\n",
+                                    m,FALSE,FALSE);
+               else /* PUBLIC, PROTECTED */
+                       print_method(out,"","\n",""," ","\n",m,FALSE,FALSE);
                out_addline_outfile(out);
                out_printf(out,"{\n");
                s = g_strdup(m->id);
                g_strup(s);
                if(strcmp(m->mtype->name,"void")==0 &&
                   m->mtype->stars==0) {
-                       GList *li;
                        print_preconditions(m);
                        if(((FuncArg *)m->args->data)->name)
                        out_printf(out,"\tgtk_signal_emit (GTK_OBJECT (%s),\n"
                                "\t\tobject_signals[%s_SIGNAL]",
                                ((FuncArg *)m->args->data)->name,s);
-                       for(li=m->args->next;li;li=g_list_next(li)) {
-                               FuncArg *fa = li->data;
-                               out_printf(out,",\n\t\t%s",fa->name);
-                       }
+                       put_signal_args(m);
                        out_printf(out,");\n}\n");
                } else {
-                       GList *li;
-                       out_printf(out,"\t");
-                       print_type(out,m->mtype,TRUE);
-                       out_printf(out,"return_val;\n");
+                       out_printf(out, "\t");
+                       print_type(out, m->mtype, TRUE);
+                       out_printf(out, "return_val = (");
+                       print_type(out, m->mtype, TRUE);
+                       if(m->defreturn)
+                               out_printf(out, ")(%s);\n", m->defreturn);
+                       else if(m->onerror)
+                               out_printf(out, ")(%s);\n", m->onerror);
+                       else
+                               out_printf(out, ")(0);\n");
                        print_preconditions(m);
                        out_printf(out,"\tgtk_signal_emit (GTK_OBJECT (%s),\n"
                                "\t\tobject_signals[%s_SIGNAL]",
                                ((FuncArg *)m->args->data)->name,s);
-                       for(li=m->args->next;li;li=g_list_next(li)) {
-                               FuncArg *fa = li->data;
-                               out_printf(out,",\n\t\t%s",fa->name);
-                       }
+                       put_signal_args(m);
                        out_printf(out,",\n\t\t&return_val);\n"
                                "\treturn return_val;\n}\n");
                }
 
-               if(!m->cbuf || !*m->cbuf->str)
+               if(!m->cbuf)
                        break;
-               out_addline_infile(out,m->line_no);
-               print_method(out,"static ","\n_real_"," ","\n",m,FALSE);
+               if(m->line_no>0)
+                       out_addline_infile(out,m->line_no);
+               print_method(out,"static ","\n___real_",""," ","\n",
+                            m,FALSE,FALSE);
                print_method_body(m,FALSE);
                break;
-       case PRIVATE_VIRTUAL_METHOD:
-               private = TRUE;
        case VIRTUAL_METHOD:
-               out_addline_infile(out,m->line_no);
-               print_method(out,private?"static ":"","\n"," ","\n",m,FALSE);
+               if(m->line_no>0)
+                       out_addline_infile(out,m->line_no);
+               if(m->scope==PRIVATE_SCOPE)
+                       print_method(out,"static ","\n",""," ","\n",
+                                    m,FALSE,FALSE);
+               else /* PUBLIC, PROTECTED */
+                       print_method(out,"","\n",""," ","\n",m,FALSE,FALSE);
                out_addline_outfile(out);
                out_printf(out,"{\n"
                        "\t%sClass *klass;\n",typebase);
@@ -1151,47 +1535,59 @@ put_method(Method *m)
                        out_printf(out,");\n}\n");
                } else {
                        GList *li;
-                       out_printf(out,"\t\treturn (*klass->%s)(%s",m->id,
+                       out_printf(out, "\t\treturn (*klass->%s)(%s",m->id,
                                   ((FuncArg *)m->args->data)->name);
                        for(li=m->args->next;li;li=g_list_next(li)) {
                                FuncArg *fa = li->data;
                                out_printf(out,",%s",fa->name);
                        }
-                       out_printf(out,");\n"
+                       out_printf(out, ");\n"
                                "\telse\n"
                                "\t\treturn (");
-                       print_type(out,m->mtype,TRUE);
-                       out_printf(out,")(%s);\n}\n",
-                               m->onerror?m->onerror:"0");
+                       print_type(out, m->mtype, TRUE);
+                       if(m->defreturn)
+                               out_printf(out, ")(%s);\n}\n", m->defreturn);
+                       else if(m->onerror)
+                               out_printf(out, ")(%s);\n}\n", m->onerror);
+                       else
+                               out_printf(out, ")(0);\n}\n");
                }
 
-               if(!m->cbuf || !*m->cbuf->str)
+               if(!m->cbuf)
                        break;
-               out_addline_infile(out,m->line_no);
-               print_method(out,"static ","\n_real_"," ","\n",m,FALSE);
+               if(m->line_no>0)
+                       out_addline_infile(out,m->line_no);
+               print_method(out,"static ","\n___real_",""," ","\n",
+                            m,FALSE,FALSE);
                print_method_body(m,FALSE);
                break;
        case OVERRIDE_METHOD:
-               out_addline_infile(out,m->line_no);
-               print_method(out,"static ","\n"," ","\n",m,FALSE);
+               if(!m->cbuf)
+                       break;
+               if(m->line_no>0)
+                       out_addline_infile(out,m->line_no);
+               print_method(out,"static ","\n",""," ","\n",
+                            m,FALSE,FALSE);
                s = replace_sep(m->otype,'_');
                g_strup(s);
+               args = get_arg_names_for_macro(m);
                if(is_void) {
-                       out_printf(out,"#define PARENT_HANDLER(args...) \\\n"
+                       out_printf(out,"#define PARENT_HANDLER(%s) \\\n"
                                   "\t{ if(%s_CLASS(parent_class)->%s) \\\n"
-                                  "\t\t(* %s_CLASS(parent_class)->%s)(##args); }\n",
-                                  s,m->id,s,m->id);
+                                  "\t\t(* %s_CLASS(parent_class)->%s)(%s); }\n",
+                                  args,s,m->id,s,m->id,args);
                } else {
-                       out_printf(out,"#define PARENT_HANDLER(args...) \\\n"
+                       out_printf(out,"#define PARENT_HANDLER(%s) \\\n"
                                   "\t((%s_CLASS(parent_class)->%s)? \\\n"
-                                  "\t\t(* %s_CLASS(parent_class)->%s)(##args): \\\n"
+                                  "\t\t(* %s_CLASS(parent_class)->%s)(%s): \\\n"
                                   "\t\t(",
-                                  s,m->id,s,m->id);
+                                  args,s,m->id,s,m->id,args);
                        out_printf(out,"(");
                        print_type(out,m->mtype,TRUE);
                        out_printf(out,")%s))\n",
                                   m->onerror?m->onerror:"0");
                }
+               g_free(args);
                g_free(s);
                print_method_body(m,TRUE);
                out_printf(out,"#undef PARENT_HANDLER\n");
@@ -1202,368 +1598,481 @@ put_method(Method *m)
 }
 
 static void
-check_duplicate(Class *c,Node *node,char *id, int line_no)
+open_files(void)
 {
-       GList *l;
-       for(l=c->nodes;l;l=g_list_next(l)) {
-               Node *n = l->data;
-               char *nid;
-               int nline_no;
-               char *s;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       nid = m->id;
-                       nline_no = m->line_no;
-               } else if(n->type == VARIABLE_NODE) {
-                       Variable *v = (Variable *)n;
-                       nid = v->id;
-                       nline_no = v->line_no;
-               } else
-                       continue;
-               if(n==node ||
-                  line_no>=nline_no ||
-                  strcmp(nid,id)!=0 ||
-                  n->type != node->type)
-                       continue;
-               s = g_strdup_printf("symbol '%s' redefined, "
-                                   "first defined on line %d",
-                                   id,line_no);
-               print_error(FALSE,s,nline_no);
-       }
-}
+       char *outfile,*outfileh,*outfileph;
 
-static void
-check_duplicate_symbols(Class *c)
-{
-       GList *l;
-       for(l=c->nodes;l;l=g_list_next(l)) {
-               Node *n = l->data;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       check_duplicate(c,n,m->id,m->line_no);
-               } else if(n->type == VARIABLE_NODE) {
-                       Variable *v = (Variable *)n;
-                       check_duplicate(c,n,v->id,v->line_no);
+       if(!for_cpp)
+               outfile = g_strconcat(filebase,".c",NULL);
+       else
+               outfile = g_strconcat(filebase,".cc",NULL);
+       if(no_touch_headers)
+               outfileh = g_strconcat("#gob#",filebase,".h#gob#",NULL);
+       else
+               outfileh = g_strconcat(filebase, ".h", NULL);
+
+       if((privates > 0 || protecteds > 0 || always_private_header) &&
+          !no_private_header)
+               outfileph = g_strconcat(filebase, "-private.h", NULL);
+       else
+               outfileph = NULL;
+
+       
+       if(no_write) {
+               devnull = fopen("/dev/null", "w");
+               if(!devnull)
+                       g_error("Cannot open null device");
+               out = devnull;
+               outh = devnull;
+               if(outfileph)
+                       outph = devnull;
+       } else {
+               out = fopen(outfile, "w");
+               if(!out) {
+                       g_error("Cannot open outfile: %s", outfile);
+               }
+               outh = fopen(outfileh, "w");
+               if(!outh)
+                       g_error("Cannot open outfile: %s", outfileh);
+               if(outfileph) {
+                       outph = fopen(outfileph, "w");
+                       if(!outph)
+                               g_error("Cannot open outfile: %s", outfileh);
                }
        }
 }
 
 static void
-check_bad_symbols(Class *c)
+put_argument_nongnu_wrappers(Class *c)
 {
-       GList *l;
-       for(l=c->nodes;l;l=g_list_next(l)) {
-               Node *n = l->data;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       if((m->scope == SIGNAL_LAST_METHOD ||
-                           m->scope == SIGNAL_FIRST_METHOD ||
-                           m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-                           m->scope == PRIVATE_SIGNAL_FIRST_METHOD ||
-                           m->scope == VIRTUAL_METHOD ||
-                           m->scope == PRIVATE_VIRTUAL_METHOD) &&
-                          strcmp(m->id,"__parent__")==0) {
-                               char *s;
-                               s = g_strdup_printf("'%s' not allowed as an "
-                                                   "identifier of signal "
-                                                   "or virtual methods",
-                                                   m->id);
-                               print_error(FALSE,s,m->line_no);
-                               g_free(s);
-                       }
-               } else if(n->type == VARIABLE_NODE) {
-                       Variable *v = (Variable *)n;
-                       if(strcmp(v->id,"_priv")==0 ||
-                          strcmp(v->id,"__parent__")==0) {
-                       char *s;
-                               s = g_strdup_printf("'%s' not allowed as a data "
-                                                   "member name",v->id);
-                               print_error(FALSE,s,v->line_no);
-                               g_free(s);
-                       }
+       GList *li;
+
+       if(arguments<0)
+               return;
+
+       for(li=c->nodes;li;li=g_list_next(li)) {
+               Node *n = li->data;
+               Argument *a = (Argument *)n;
+               char *aname;
+               char *cast;
+
+               if(n->type != ARGUMENT_NODE)
+                       continue;
+
+               aname = g_strdup(a->name);
+               g_strup(aname);
+
+               if(a->atype)
+                       cast = get_type(a->atype, TRUE);
+               else
+                       cast = g_strdup(get_cast(a->gtktype, TRUE));
+
+               if(cast) {
+                       if(a->set)
+                               out_printf(outh, "#define %s_ARG_%s(arg)    \t"
+                                          "\"%s\",(%s)(arg)\n",
+                                          macrobase, aname, a->name, cast);
+                       if(a->get)
+                               out_printf(outh, "#define %s_GET_ARG_%s(arg)\t"
+                                          "\"%s\",(%s*)(arg)\n",
+                                          macrobase, aname, a->name, cast);
+               } else {
+                       if(a->set)
+                               out_printf(outh, "#define %s_ARG_%s(arg)    \t"
+                                          "\"%s\",(arg)\n",
+                                          macrobase, aname, a->name);
+                       if(a->get)
+                               out_printf(outh, "#define %s_GET_ARG_%s(arg)\t"
+                                          "\"%s\",(arg)\n",
+                                          macrobase, aname, a->name);
                }
+               g_free(cast);
+               g_free(aname);
        }
 }
 
-
 static void
-check_duplicate_named(Class *c,Node *node,char *id, int line_no)
+put_argument_gnu_wrappers(Class *c)
 {
-       GList *l;
-       for(l=c->nodes;l;l=g_list_next(l)) {
-               Node *n = l->data;
-               char *nid;
-               int nline_no;
+       GList *li;
+
+       if(arguments<0)
+               return;
+
+       for(li=c->nodes;li;li=g_list_next(li)) {
+               Node *n = li->data;
+               Argument *a = (Argument *)n;
                char *s;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       if(m->scope == SIGNAL_LAST_METHOD ||
-                          m->scope == SIGNAL_FIRST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_FIRST_METHOD) {
-                               nid = m->id;
-                               nline_no = m->line_no;
-                       } else
-                               continue;
-               } else if(n->type == ARGUMENT_NODE) {
-                       Argument *a = (Argument *)n;
-                       nid = a->name;
-                       nline_no = a->line_no;
-               } else
-                       continue;
-               if(n==node ||
-                  line_no>=nline_no ||
-                  strcmp(nid,id)!=0)
+               char *cast;
+               if(n->type != ARGUMENT_NODE)
                        continue;
-               s = g_strdup_printf("named symbol (argument or signal) '%s' "
-                                   "redefined, first defined on line %d",
-                                   id,line_no);
-               print_error(FALSE,s,nline_no);
+               s = g_strdup(a->name);
+               g_strup(s);
+               if(a->atype)
+                       cast = get_type(a->atype,TRUE);
+               else
+                       cast = g_strdup(get_cast(a->gtktype, TRUE));
+               if(cast) {
+                       if(a->set)
+                               out_printf(outh, "#define %s_ARG_%s(arg)    \t"
+                                          "\"%s\",({%sz = (arg); z;})\n",
+                                          macrobase, s, a->name, cast);
+                       if(a->get)
+                               out_printf(outh, "#define %s_GET_ARG_%s(arg)\t"
+                                          "\"%s\",({%s*z = (arg); z;})\n",
+                                          macrobase, s, a->name, cast);
+               } else {
+                       if(a->set)
+                               out_printf(outh, "#define %s_ARG_%s(arg)    \t"
+                                          "\"%s\",(arg)\n",
+                                          macrobase, s, a->name);
+                       if(a->get)
+                               out_printf(outh, "#define %s_GET_ARG_%s(arg)\t"
+                                          "\"%s\",(arg)\n",
+                                          macrobase, s, a->name);
+               }
+               g_free(cast);
+               g_free(s);
        }
 }
 
 static void
-check_duplicate_signals_args(Class *c)
+print_ccode_block(CCode *cc)
 {
-       GList *l;
-       for(l=c->nodes;l;l=g_list_next(l)) {
-               Node *n = l->data;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       if(m->scope == SIGNAL_LAST_METHOD ||
-                          m->scope == SIGNAL_FIRST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_FIRST_METHOD)
-                               check_duplicate_named(c,n,m->id,m->line_no);
-               } else if(n->type == ARGUMENT_NODE) {
-                       Argument *a = (Argument *)n;
-                       check_duplicate_named(c,n,a->name,a->line_no);
+       FILE *fp;
+       switch(cc->cctype) {
+       case HT_CCODE:
+               /* HT code is printed exactly like normal header
+                  code but is printed before */
+       case H_CCODE:
+               fp = outh;
+               out_printf(fp, "\n");
+               break;
+       case AT_CCODE:
+               /* AT code is printed exactly like normal 'all'
+                  code but is printed before */
+       case A_CCODE:
+               if(outph) {
+                       out_printf(outph, "\n");
+                       out_printf(outph, "%s\n", cc->cbuf);
+                       out_addline_infile(outph, cc->line_no);
+                       out_addline_outfile(outph);
                }
+               out_printf(outh, "\n");
+               out_printf(outh, "%s\n", cc->cbuf);
+               fp = out;
+               out_printf(fp, "\n");
+               out_addline_infile(fp, cc->line_no);
+               break;
+       default:
+       case C_CCODE:
+               fp = out;
+               out_printf(fp,"\n");
+               out_addline_infile(fp, cc->line_no);
+               break;
+       case PH_CCODE:
+               if(outph)
+                       fp = outph;
+               else
+                       fp = out;
+               out_printf(fp, "\n");
+               out_addline_infile(fp, cc->line_no);
+               break;
        }
+       out_printf(fp, "%s\n", cc->cbuf);
+       if(cc->cctype == C_CCODE ||
+          cc->cctype == A_CCODE ||
+          cc->cctype == AT_CCODE ||
+          cc->cctype == PH_CCODE)
+               out_addline_outfile(fp);
 }
 
 static void
-check_public_new(Class *c)
+print_class_block(Class *c)
 {
        GList *l;
+       char *s;
+       gboolean printed_private = FALSE;
+
+       if(any_special) {
+               out_printf(out, "/* utility types we may need */\n");
+               if(special_array[SPECIAL_2POINTER])
+                       out_printf(out, "typedef struct { "
+                                  "gpointer a; gpointer b; "
+                                  "} ___twopointertype;\n");
+               if(special_array[SPECIAL_3POINTER])
+                       out_printf(out, "typedef struct { "
+                                  "gpointer a; gpointer b; "
+                                  "gpointer c; "
+                                  "} ___threepointertype;\n");
+               if(special_array[SPECIAL_INT_POINTER])
+                       out_printf(out, "typedef struct { "
+                                  "gint a; gpointer b; "
+                                  "} ___intpointertype;\n");
+               out_printf(out, "\n");
+       }
+
+       out_printf(outh, "\n/*\n"
+                  " * Type checking and casting macros\n"
+                  " */\n");
+       out_printf(outh,"#define %s\t"
+                  "(%s_get_type())\n",
+                  macrotype,funcbase);
+       out_printf(outh,"#define %s(obj)\t"
+                  "GTK_CHECK_CAST((obj),%s_get_type(),%s)\n",
+                  macrobase,funcbase,typebase);
+       out_printf(outh,"#define %s_CLASS(klass)\t"
+                  "GTK_CHECK_CLASS_CAST((klass),%s_get_type(),%sClass)\n",
+                  macrobase,funcbase,typebase);
+       out_printf(outh,"#define %s(obj)\t"
+                  "GTK_CHECK_TYPE((obj), %s_get_type ())\n\n",
+                  macrois,funcbase);
+
+       out_printf(out, "/* self casting macros */\n");
+       out_printf(out, "#define SELF(x) %s(x)\n", macrobase);
+       out_printf(out, "#define IS_SELF(x) %s(x)\n", macrois);
+       out_printf(out, "#define SELF_CLASS(x) %s_CLASS(x)\n\n", macrobase);
+
+       if(privates>0) {
+               out_printf(outh, "\n/* Private structure type */\n");
+               out_printf(outh,"typedef struct _%sPrivate %sPrivate;\n",
+                          typebase,typebase);
+       }
+
+       out_printf(outh, "\n/*\n"
+                  " * Main object structure\n"
+                  " */\n");
+       s = replace_sep(c->otype,'_');
+       g_strup(s);
+       out_printf(outh,"#ifndef __TYPEDEF_%s__\n"
+                  "#define __TYPEDEF_%s__\n",s,s);
+       g_free(s);
+       out_printf(outh,"typedef struct _%s %s;\n"
+                  "#endif\n",typebase,typebase);
+       out_printf(outh,"struct _%s {\n\t%s __parent__;\n",
+                  typebase,ptypebase);
        for(l=c->nodes;l;l=g_list_next(l)) {
+               static gboolean printed_public = FALSE;
                Node *n = l->data;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       if(m->scope!=PUBLIC_SCOPE &&
-                          strcmp(m->id,"new")==0)
-                               print_error(TRUE,
-                                           "'new' should be a public method",
-                                           m->line_no);
+               Variable *v = (Variable *)n;
+               if(n->type == VARIABLE_NODE &&
+                  v->scope == PUBLIC_SCOPE) {
+                       if(!printed_public) {
+                               out_printf(outh,"\t/*< public >*/\n");
+                               printed_public = TRUE;
+                       }
+                       put_variable((Variable *)n,outh);
                }
        }
-}
-
-static void
-check_vararg(Class *c)
-{
-       GList *l;
+       /* put protecteds always AFTER publics */
        for(l=c->nodes;l;l=g_list_next(l)) {
                Node *n = l->data;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       if(!m->vararg)
-                               continue;
-                       if(m->scope == OVERRIDE_METHOD ||
-                          m->scope == SIGNAL_LAST_METHOD ||
-                          m->scope == SIGNAL_FIRST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_FIRST_METHOD ||
-                          m->scope == VIRTUAL_METHOD ||
-                          m->scope == PRIVATE_VIRTUAL_METHOD) {
-                               print_error(FALSE,
-                                           "signals, overrides and virtuals, "
-                                           "can't have variable argument "
-                                           "lists",
-                                           m->line_no);
+               Variable *v = (Variable *)n;
+               if(n->type == VARIABLE_NODE &&
+                  v->scope == PROTECTED_SCOPE) {
+                       if(!printed_private) {
+                               out_printf(outh,"\t/*< private >*/\n");
+                               printed_private = TRUE;
                        }
+                       put_variable((Variable *)n,outh);
                }
        }
-}
+       if(privates>0) {
+               if(!printed_private)
+                       out_printf(outh,"\t/*< private >*/\n");
+               out_printf(outh,"\t%sPrivate *_priv;\n",typebase);
+       }
+       out_printf(outh,"};\n");
 
-static void
-check_firstarg(Class *c)
-{
-       GList *l;
-       for(l=c->nodes;l;l=g_list_next(l)) {
-               Node *n = l->data;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       if(m->args)
-                               continue;
-                       if(m->scope == OVERRIDE_METHOD ||
-                          m->scope == SIGNAL_LAST_METHOD ||
-                          m->scope == SIGNAL_FIRST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_FIRST_METHOD ||
-                          m->scope == VIRTUAL_METHOD ||
-                          m->scope == PRIVATE_VIRTUAL_METHOD) {
-                               print_error(FALSE,
-                                           "signals, overrides and virtuals, "
-                                           "can't have no arguments",
-                                           m->line_no);
+       if(privates>0) {
+               FILE *outfp;
+
+               /* if we are to stick this into the private
+                  header, if not stick it directly into the
+                  C file */
+               if(outph) 
+                       outfp = outph;
+               else
+                       outfp = out;
+
+               out_printf(outfp,"struct _%sPrivate {\n",
+                          typebase);
+               for(l=c->nodes;l;l=l->next) {
+                       Node *n = l->data;
+                       Variable *v = (Variable *)n;
+                       if(n->type == VARIABLE_NODE &&
+                          v->scope == PRIVATE_SCOPE) {
+                               out_addline_infile(outfp,v->line_no);
+                               put_variable(v,outfp);
                        }
                }
+               out_addline_outfile(outfp);
+               out_printf(outfp,"};\n");
        }
-}
 
-static int
-count_signals(Class *c)
-{
-       int num = 0;
-       GList *l;
+       out_printf(outh, "\n/*\n"
+                  " * Class definition\n"
+                  " */\n");
+       out_printf(outh,"typedef struct _%sClass %sClass;\n",
+                  typebase,typebase);
+       out_printf(outh,
+                  "struct _%sClass {\n\t%sClass __parent__;\n",
+                  typebase,ptypebase);
        for(l=c->nodes;l;l=g_list_next(l)) {
                Node *n = l->data;
-               if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       if(m->scope == SIGNAL_LAST_METHOD ||
-                          m->scope == SIGNAL_FIRST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_LAST_METHOD ||
-                          m->scope == PRIVATE_SIGNAL_FIRST_METHOD)
-                               num++;
-               }
+               if(n->type == METHOD_NODE)
+                       put_vs_method((Method *)n);
        }
-       return num;
-}
-
-static int
-count_arguments(Class *c)
-{
-       int num = 0;
-       GList *li;
+       out_printf(outh,"};\n\n");
 
-       for(li=c->nodes;li;li=g_list_next(li)) {
-               Node *n = li->data;
-               if(n->type == ARGUMENT_NODE)
-                       num ++;
+       out_printf(out,"/* here are local prototypes */\n");
+       if(arguments>0) {
+               out_printf(out,"static void ___object_set_arg "
+                          "(GtkObject *object, GtkArg *arg, "
+                          "guint arg_id);\n"
+                          "static void ___object_get_arg "
+                          "(GtkObject *object, GtkArg *arg, "
+                          "guint arg_id);\n");
        }
-       return num;
-}
 
-static int
-count_overrides(Class *c)
-{
-       int num = 0;
-       GList *l;
+       out_printf(outh, "\n/*\n"
+                  " * Public methods\n"
+                  " */\n");
+
+       out_printf(outh,"guint\t%s_get_type\t(void);\n",funcbase);
        for(l=c->nodes;l;l=g_list_next(l)) {
                Node *n = l->data;
                if(n->type == METHOD_NODE) {
-                       Method *m = (Method *)n;
-                       if(m->scope == OVERRIDE_METHOD)
-                               num++;
+                       put_pub_method((Method *)n);
+                       put_prot_method((Method *)n);
+                       put_priv_method_prot((Method *)n);
                }
        }
-       return num;
-}
 
-static int
-count_privates(Class *c)
-{
-       int num = 0;
-       GList *l;
-       for(l=c->nodes;l;l=g_list_next(l)) {
-               Node *n = l->data;
-               if(n->type == VARIABLE_NODE) {
-                       Variable *v = (Variable *)n;
-                       if(v->scope == PRIVATE_SCOPE)
-                               num++;
+/* this idea is less and less apealing to me */
+#if 0
+       if(!no_signal_connect) {
+               if(signals>0) {
+                       out_printf(outh, "\n/*\n"
+                                  " * Signal connection methods\n"
+                                  " */\n");
+               }
+
+               for(l=c->nodes;l;l=g_list_next(l)) {
+                       Node *n = l->data;
+                       if(n->type == METHOD_NODE)
+                               put_signal_connect((Method *)n);
                }
        }
-       return num;
-}
+#endif
 
 
-static void
-open_files(void)
-{
-       char *outfile,*outfileh,*outfileph;
+       /* argument wrapping macros */
+       if(arguments>0 && !no_gnu) {
+               out_printf(outh, "\n/*\n"
+                          " * Argument wrapping macros\n"
+                          " */\n");
+               out_printf(outh,"#ifdef __GNUC__\n");
+               put_argument_gnu_wrappers(c);
+               out_printf(outh,"#else /* __GNUC__ */\n");
+               put_argument_nongnu_wrappers(c);
+               out_printf(outh,"#endif /* __GNUC__ */\n\n");
+       } else if(arguments>0 && no_gnu) {
+               out_printf(outh, "\n/*\n"
+                          " * Argument wrapping macros\n"
+                          " */\n");
+               put_argument_nongnu_wrappers(c);
+       }
 
-       if(!for_cpp)
-               outfile = g_strconcat(filebase,".c",NULL);
-       else
-               outfile = g_strconcat(filebase,".cc",NULL);
-       if(no_touch_headers)
-               outfileh = g_strconcat("#gob#",filebase,".h#gob#",NULL);
-       else
-               outfileh = g_strconcat(filebase,".h",NULL);
+       if(signals>0) {
+               for(l=c->nodes;l;l=g_list_next(l)) {
+                       Node *n = l->data;
+                       if(n->type == METHOD_NODE)
+                               add_signal_prots((Method *)n);
+               }
+       }
 
-       if((privates>0 || always_private_header) &&
-          !no_private_header)
-               outfileph = g_strconcat(filebase,"-private.h",NULL);
-       else
-               outfileph = NULL;
+       add_enums(c);
 
-       
-       out = fopen(outfile,"w");
-       if(!out) {
-               g_error("Cannot open outfile: %s",outfile);
-       }
-       outh = fopen(outfileh,"w");
-       if(!outh) {
-               g_error("Cannot open outfile: %s",outfileh);
-       }
-       if(outfileph) {
-               outph = fopen(outfileph,"w");
-               if(!outph) {
-                       g_error("Cannot open outfile: %s",outfileh);
+       add_get_type();
+
+       if(any_method_to_alias(c)) {
+               if(no_gnu)
+                       make_method_nongnu_aliases(c);
+               else {
+                       out_printf(out,"\n#ifdef __GNUC__\n");
+                       make_method_gnu_aliases(c);
+                       out_printf(out,"#else /* __GNUC__ */\n");
+                       make_method_nongnu_aliases(c);
+                       out_printf(out,"#endif /* __GNUC__ */\n\n");
                }
        }
+
+       out_printf(out,"/* a macro for creating a new object of our type */\n");
+       out_printf(out,
+                  "#define GET_NEW ((%s *)gtk_type_new(%s_get_type()))\n\n",
+                  typebase, funcbase);
+
+       add_inits(c);
+
+       if(arguments>0) {
+               add_getset_arg(c, TRUE);
+               add_getset_arg(c, FALSE);
+       }
+
+       for(l=c->nodes;l;l=g_list_next(l)) {
+               Node *n = l->data;
+               if(n->type == METHOD_NODE)
+                       put_method((Method *)n);
+       }
+
+       add_bad_hack_to_avoid_unused_warnings(c);
 }
 
 static void
-generate_outfiles(void)
+print_version_macros(void)
 {
-       char *p;
-       GList *li;
-       time_t curtime;
-       gboolean found_header;
+       int major=0, minor=0, pl=0;
+       sscanf(VERSION, "%d.%d.%d", &major, &minor, &pl);
 
+       out_printf(out, "#define GOB_VERSION_MAJOR %d\n", major);
+       out_printf(out, "#define GOB_VERSION_MINOR %d\n", minor);
+       out_printf(out, "#define GOB_VERSION_PATCHLEVEL %d\n\n", pl);
+}
+
+static void
+print_file_comments(void)
+{
+       time_t curtime;
        time(&curtime);
-       out_printf(outh,"/* Generated by GOB (v%s)"
-              "   (do not edit directly) */\n\n",VERSION);
-       if(outph)
-               out_printf(outph,"/* Generated by GOB (v%s)"
-                          "   (do not edit directly) */\n\n",VERSION);
-       out_printf(out,"/* Generated by GOB (v%s) on %s"
-              "   (do not edit directly) */\n\n",VERSION,ctime(&curtime));
-       
-       p = replace_sep(((Class *)class)->otype,'_');
-       g_strup(p);
-       out_printf(outh,"#ifndef __%s_H__\n#define __%s_H__\n\n"
-               "#include <gtk/gtk.h>\n\n",p,p);
+       out_printf(outh, "/* Generated by GOB (v%s)"
+                  "   (do not edit directly) */\n\n", VERSION);
        if(outph)
-               out_printf(outph,"#ifndef __%s_PRIVATE_H__\n"
-                          "#define __%s_PRIVATE_H__\n\n"
-                          "#include \"%s.h\"\n\n",p,p,filebase);
-       g_free(p);
+               out_printf(outph, "/* Generated by GOB (v%s)"
+                          "   (do not edit directly) */\n\n", VERSION);
+       out_printf(out, "/* Generated by GOB (v%s) on %s"
+                  "   (do not edit directly) */\n\n",
+                  VERSION, ctime(&curtime));
+}
 
-       if(!for_cpp) {
-               out_printf(outh,"#ifdef __cplusplus\n"
-                          "extern \"C\" {\n"
-                          "#endif /* __cplusplus */\n\n");
-               if(outph)
-                       out_printf(outph,"#ifdef __cplusplus\n"
-                                  "extern \"C\" {\n"
-                                  "#endif /* __cplusplus */\n\n");
-       }
-       
-       p = g_strconcat(filebase,".h",NULL);
+static void
+print_includes(void)
+{
+       gboolean found_header;
+       char *p;
+
+       p = g_strconcat(filebase, ".h", NULL);
        found_header = TRUE;
-       if(!g_list_find_custom(include_files,p,(GCompareFunc)strcmp)) {
-               out_printf(out,"#include \"%s.h\"\n\n",filebase);
+       if(!g_list_find_custom(include_files, p, (GCompareFunc)strcmp)) {
+               out_printf(out, "#include \"%s.h\"\n\n", filebase);
                found_header = FALSE;
        }
        g_free(p);
 
        /* if we are creating a private header see if it was included */
        if(outph) {
-               p = g_strconcat(filebase,"-private.h",NULL);
-               if(!g_list_find_custom(include_files,p,(GCompareFunc)strcmp)) {
+               p = g_strconcat(filebase, "-private.h", NULL);
+               if(!g_list_find_custom(include_files, p, (GCompareFunc)strcmp)) {
                        out_printf(out,"#include \"%s-private.h\"\n\n",
                                   filebase);
                        if(found_header)
@@ -1579,176 +2088,116 @@ generate_outfiles(void)
                }
                g_free(p);
        }
+}
+
+static void
+print_header_prefixes(void)
+{
+       char *p;
 
+       p = replace_sep(((Class *)class)->otype, '_');
+       g_strup(p);
+       out_printf(outh, "#ifndef __%s_H__\n#define __%s_H__\n\n", p, p);
+       if(outph)
+               out_printf(outph, "#ifndef __%s_PRIVATE_H__\n"
+                          "#define __%s_PRIVATE_H__\n\n"
+                          "#include \"%s.h\"\n\n", p, p, filebase);
+       g_free(p);
+
+       if(!no_extern_c) {
+               out_printf(outh, "#ifdef __cplusplus\n"
+                          "extern \"C\" {\n"
+                          "#endif /* __cplusplus */\n\n");
+               if(outph)
+                       out_printf(outph, "#ifdef __cplusplus\n"
+                                  "extern \"C\" {\n"
+                                  "#endif /* __cplusplus */\n\n");
+       }
+}
+
+static void
+print_header_postfixes(void)
+{
+       if(!no_extern_c)
+               out_printf(outh, "\n#ifdef __cplusplus\n"
+                          "}\n"
+                          "#endif /* __cplusplus */\n");
+       out_printf(outh, "\n#endif\n");
+       if(outph) {
+               if(!no_extern_c)
+                       out_printf(outph, "\n#ifdef __cplusplus\n"
+                                  "}\n"
+                                  "#endif /* __cplusplus */\n");
+               out_printf(outph, "\n#endif\n");
+       }
+}
+
+static void
+print_all_top(void)
+{
+       GList *li;
+
+       /* print the AT_CCODE blocks */
        for(li=nodes;li;li=g_list_next(li)) {
                Node *node = li->data;
                if(node->type == CCODE_NODE) {
                        CCode *cc = (CCode *)node;
-                       FILE *fp;
-                       if(cc->header) {
-                               fp = outh;
-                               out_printf(fp,"\n");
-                       } else {
-                               fp = out;
-                               out_printf(fp,"\n");
-                               out_addline_infile(fp,cc->line_no);
-                       }
-                       out_printf(fp,"%s\n",cc->cbuf->str);
-                       if(!cc->header)
-                               out_addline_outfile(fp);
-               } else if(node->type == CLASS_NODE) {
-                       Class *c = (Class *)class;
-                       GList *l;
-
-                       out_printf(outh,"\n#define %s\t"
-                               "(%s_get_type())\n",
-                               macrotype,funcbase);
-                       out_printf(outh,"#define %s(obj)\t"
-                               "GTK_CHECK_CAST((obj),%s_get_type(),%s)\n",
-                               macrobase,funcbase,typebase);
-                       out_printf(outh,"#define %s_CLASS(klass)\t"
-                               "GTK_CHECK_CLASS_CAST((klass),%s_get_type(),%sClass)\n",
-                               macrobase,funcbase,typebase);
-                       out_printf(outh,"#define %s(obj)\t"
-                               "GTK_CHECK_TYPE((obj), %s_get_type ())\n\n",
-                               macrois,funcbase);
-
-                       if(privates>0)
-                               out_printf(outh,"\ntypedef struct _%sPrivate %sPrivate;\n",typebase,typebase);
-
-                       out_printf(outh,"\ntypedef struct _%s %s;\n",typebase,typebase);
-                       out_printf(outh,"struct _%s {\n\t%s __parent__;\n",
-                                  typebase,ptypebase);
-                       for(l=c->nodes;l;l=g_list_next(l)) {
-                               Node *n = l->data;
-                               Variable *v = (Variable *)n;
-                               if(n->type == VARIABLE_NODE &&
-                                  v->scope == PUBLIC_SCOPE)
-                                       put_variable((Variable *)n,outh);
-                       }
-                       if(privates>0)
-                               out_printf(outh,"\t%sPrivate *_priv;\n",typebase);
-                       out_printf(outh,"};\n");
-
-                       if(privates>0) {
-                               FILE *outfp;
-
-                               /* if we are to stick this into the private
-                                  header, otherwise stick it directly into the
-                                  C file */
-                               if(outph) 
-                                       outfp = outph;
-                               else
-                                       outfp = out;
-
-                               out_printf(outfp,"struct _%sPrivate {\n",
-                                          typebase);
-                               for(l=c->nodes;l;l=l->next) {
-                                       Node *n = l->data;
-                                       Variable *v = (Variable *)n;
-                                       if(n->type == VARIABLE_NODE &&
-                                          v->scope == PRIVATE_SCOPE) {
-                                               out_addline_infile(outfp,v->line_no);
-                                               put_variable(v,outfp);
-                                       }
-                               }
-                               out_addline_outfile(outfp);
-                               out_printf(outfp,"};\n");
-                       }
+                       if(cc->cctype==AT_CCODE)
+                               print_ccode_block((CCode *)node);
+               }
+       }
+}
 
-                       out_printf(outh,"\ntypedef struct _%sClass %sClass;\n",
-                               typebase,typebase);
-                       out_printf(outh,
-                               "struct _%sClass {\n\t%sClass __parent__;\n",
-                               typebase,ptypebase);
-                       for(l=c->nodes;l;l=g_list_next(l)) {
-                               Node *n = l->data;
-                               if(n->type == METHOD_NODE)
-                                       put_vs_method((Method *)n);
-                       }
-                       out_printf(outh,"};\n\n");
+static void
+print_header_top(void)
+{
+       GList *li;
 
-                       out_printf(outh,"guint\t%s_get_type\t(void);\n",funcbase);
-                       
-                       if(arguments>0) {
-                               out_printf(out,"static void __object_set_arg "
-                                          "(GtkObject *object, GtkArg *arg, "
-                                          "guint arg_id);\n"
-                                          "static void __object_get_arg "
-                                          "(GtkObject *object, GtkArg *arg, "
-                                          "guint arg_id);\n");
-                       }
+       /* mandatory include */
+       out_printf(outh,"#include <gtk/gtk.h>\n\n");
 
-                       for(l=c->nodes;l;l=g_list_next(l)) {
-                               Node *n = l->data;
-                               if(n->type == METHOD_NODE) {
-                                       put_pub_method((Method *)n);
-                                       put_priv_method_prot((Method *)n);
-                               }
-                       }
+       /* print the HT_CCODE blocks */
+       for(li=nodes;li;li=g_list_next(li)) {
+               Node *node = li->data;
+               if(node->type == CCODE_NODE) {
+                       CCode *cc = (CCode *)node;
+                       if(cc->cctype==HT_CCODE)
+                               print_ccode_block((CCode *)node);
+               }
+       }
+}
 
-                       if(signals>0) {
-                               for(l=c->nodes;l;l=g_list_next(l)) {
-                                       Node *n = l->data;
-                                       if(n->type == METHOD_NODE) {
-                                               add_signal_prots((Method *)n);
-                                       }
-                               }
-                       }
-                       
-                       add_enums(c);
-                       
-                       add_get_type();
+static void
+generate_outfiles(void)
+{
+       GList *li;
 
-                       make_method_pointers(c);
+       print_file_comments();
 
-                       out_printf(out,"#define GET_NEW (gtk_type_new(%s_get_type()))\n",
-                                  funcbase);
+       print_all_top();
 
-                       add_inits(c);
+       print_header_top();
 
-                       if(arguments>0) {
-                               add_getset_arg(c, TRUE);
-                               add_getset_arg(c, FALSE);
-                       }
+       print_header_prefixes();
 
-                       for(l=c->nodes;l;l=g_list_next(l)) {
-                               Node *n = l->data;
-                               if(n->type == METHOD_NODE) {
-                                       put_method((Method *)n);
-                               }
-                       }
+       print_version_macros();
 
-                       out_printf(out,"#undef GET_NEW\n");
+       print_includes();
 
-                       add_bad_hack_to_avoid_unused_warnings(c);
+       for(li=nodes;li;li=g_list_next(li)) {
+               Node *node = li->data;
+               if(node->type == CCODE_NODE) {
+                       CCode *cc = (CCode *)node;
+                       if(cc->cctype!=HT_CCODE)
+                               print_ccode_block((CCode *)node);
+               } else if(node->type == CLASS_NODE) {
+                       print_class_block((Class *)node);
                } else
                        g_assert_not_reached();
        }
 
-       if(!for_cpp)
-               out_printf(outh,"\n#ifdef __cplusplus\n"
-                          "}\n"
-                          "#endif /* __cplusplus */\n");
-       out_printf(outh,"\n#endif");
-       if(outph) {
-               if(!for_cpp)
-                       out_printf(outph,"\n#ifdef __cplusplus\n"
-                                  "}\n"
-                                  "#endif /* __cplusplus */\n");
-               out_printf(outph,"\n#endif");
-       }
-}
-
-#if 0
-static void
-usage(poptContext optCon, int exitcode, char *error, char *addl)
-{
-       poptPrintUsage(optCon, stderr, 0);
-       if (error) fprintf(stderr, "%s: %s", error, addl);
-       exit(exitcode);
+       print_header_postfixes();
 }
-#endif
 
 static void
 print_help(void)
@@ -1756,16 +2205,25 @@ print_help(void)
        fprintf(stderr,"Gob version %s\n\n",VERSION);
        fprintf(stderr,"Options:\n"
                "\t--help,-h,-?            Display this help\n"
+               "\t--version               Display version\n"
                "\t--exit-on-warn,-w       Exit with an error on warnings\n"
                "\t--no-exit-on-warn       Don't exit on warnings [default]\n"
                "\t--for-cpp               Create C++ files\n"
+               "\t--no-extern-c           Never print extern \"C\" into the "
+                                         "header\n"
+               "\t--no-gnu                Never use GNU extentions\n"
                "\t--no-touch-headers      Don't touch headers unless they "
                                          "really changed\n"
                "\t--always-private-header Always create a private header "
-                                         "file, even if it would be empty\n"
+                                         "file,\n"
+               "\t                        even if it would be empty\n"
                "\t--no-private-header     Don't create a private header, "
-                                         "put private structure inside c "
-                                         "file\n");
+                                         "put private\n"
+               "\t                        structure and protected "
+                                         "prototypes inside c file\n"
+               "\t--no-write,-n           Don't write output files, just "
+                                         "check syntax\n"
+               "\t--no-lines              Don't print '#line' to output\n");
 }
 
 static void
@@ -1781,7 +2239,7 @@ parse_options(int argc, char *argv[])
                if(no_opts || argv[i][0]!='-') {
                        /*must be a file*/
                        if(got_file) {
-                               fprintf(stderr,"Specify only one file!\n");
+                               fprintf(stderr, "Specify only one file!\n");
                                print_help();
                                exit(1);
                        }
@@ -1790,44 +2248,58 @@ parse_options(int argc, char *argv[])
                } else if(strcmp(argv[i],"--help")==0) {
                        print_help();
                        exit(0);
-               } else if(strcmp(argv[i],"--exit-on-warn")==0) {
+               } else if(strcmp(argv[i],"--version")==0) {
+                       fprintf(stderr, "Gob version %s\n", VERSION);
+                       exit(0);
+               } else if(strcmp(argv[i], "--exit-on-warn")==0) {
                        exit_on_warn = TRUE;
-               } else if(strcmp(argv[i],"--no-exit-on-warn")==0) {
+               } else if(strcmp(argv[i], "--no-exit-on-warn")==0) {
                        exit_on_warn = FALSE;
-               } else if(strcmp(argv[i],"--for-cpp")==0) {
+               } else if(strcmp(argv[i], "--for-cpp")==0) {
                        for_cpp = TRUE;
-               } else if(strcmp(argv[i],"--no-touch-headers")==0) {
+               } else if(strcmp(argv[i], "--no-touch-headers")==0) {
                        no_touch_headers = TRUE;
-               } else if(strcmp(argv[i],"--always-private-header")==0) {
+               } else if(strcmp(argv[i], "--always-private-header")==0) {
                        no_private_header = FALSE;
                        always_private_header = TRUE;
-               } else if(strcmp(argv[i],"--no-private-header")==0) {
+               } else if(strcmp(argv[i], "--no-private-header")==0) {
                        always_private_header = FALSE;
                        no_private_header = TRUE;
-               } else if(strcmp(argv[i],"--")==0) {
+               } else if(strcmp(argv[i], "--no-gnu")==0) {
+                       no_gnu = TRUE;
+               } else if(strcmp(argv[i], "--no-extern-c")==0) {
+                       no_extern_c = TRUE;
+               } else if(strcmp(argv[i], "--no-write")==0) {
+                       no_write = TRUE;
+               } else if(strcmp(argv[i], "--no-lines")==0) {
+                       no_lines = TRUE;
+               } else if(strcmp(argv[i], "--")==0) {
                        /*further arguments are files*/
                        no_opts = TRUE;
-               } else if(strncmp(argv[i],"--",2)==0) {
+               } else if(strncmp(argv[i], "--",2)==0) {
                        /*unknown long option*/
-                       fprintf(stderr,"Unknown option '%s'!\n",argv[i]);
+                       fprintf(stderr, "Unknown option '%s'!\n", argv[i]);
                        print_help();
                        exit(1);
                } else {
                        /*by now we know we have a string starting with
                          - which is a short option string*/
                        char *p = argv[i]+1;
-                       for(p=argv[i]+1;*p;p++) {
+                       for(p=argv[i]+1; *p; p++) {
                                switch(*p) {
                                case 'w':
                                        exit_on_warn=TRUE;
                                        break;
+                               case 'n':
+                                       no_write = TRUE;
+                                       break;
                                case 'h':
                                case '?':
                                        print_help();
                                        exit(0);
                                default:
                                        fprintf(stderr,
-                                               "Unknown option '%c'!\n",*p);
+                                               "Unknown option '%c'!\n", *p);
                                        print_help();
                                        exit(1);
                                }
@@ -1836,18 +2308,20 @@ parse_options(int argc, char *argv[])
        }
 }
 
+/* this is a somewhat ugly hack, but it appears to work */
 static void
 compare_and_move_header(void)
 {
-       char *hfnew = g_strconcat("#gob#",filebase,".h#gob#",NULL);
-       char *hf = g_strconcat(filebase,".h",NULL);
+       char *hfnew = g_strconcat("#gob#", filebase, ".h#gob#", NULL);
+       char *hf = g_strconcat(filebase, ".h", NULL);
        struct stat s;
        if(stat(hf,&s)==0) {
                char *s;
-               s = g_strdup_printf("cmp '%s' '%s' > /dev/null",hf,hfnew);
+               s = g_strdup_printf("cmp '%s' '%s' > /dev/null", hf, hfnew);
                if(system(s)==0) {
                        if(unlink(hfnew)!=0)
-                               print_error(FALSE,"Can't remove new header file",0);
+                               print_error(FALSE,
+                                           "Can't remove new header file", 0);
                        g_free(hfnew);
                        g_free(hf);
                        g_free(s);
@@ -1855,10 +2329,10 @@ compare_and_move_header(void)
                }
                g_free(s);
                if(unlink(hf)!=0)
-                       print_error(FALSE,"Can't remove old header file",0);
+                       print_error(FALSE, "Can't remove old header file", 0);
        }
        if(rename(hfnew,hf)!=0)
-               print_error(FALSE,"Can't rename new header file",0);
+               print_error(FALSE, "Can't rename new header file", 0);
        g_free(hfnew);
        g_free(hf);
 }
@@ -1866,43 +2340,12 @@ compare_and_move_header(void)
 int
 main(int argc, char *argv[])
 {
-#if 0
-       int c;
-       poptContext optCon;
-       
-       struct poptOption optionsTable[] = {
-               { "exit-on-warn", 'w', 0, &exit_on_warn, 0,
-                        "exit on warnings" },
-               POPT_AUTOHELP
-               { NULL, 0, 0, NULL, 0 }
-       };
-       
-       optCon = poptGetContext(NULL, argc, argv, optionsTable, 0);
-       poptSetOtherOptionHelp(optCon, "[OPTIONS]* [filename]");
-       
-       while ((c = poptGetNextOpt(optCon)) >= 0)
-               ;
-       
-       filename = poptGetArg(optCon);
-       if(!(poptPeekArg(optCon) == NULL))
-               usage(optCon, 1, "Specify only one file",
-                     ".e.g., filename.gob");
-                       
-       if (c < -1) {
-               /* an error occurred during option processing */
-               fprintf(stderr, "%s: %s\n",
-                       poptBadOption(optCon, POPT_BADOPTION_NOALIAS),
-                       poptStrerror(c));
-               return 1;
-       }
-#endif
-
-       parse_options(argc,argv);
+       parse_options(argc, argv);
        
        if(filename) {
-               yyin = fopen(filename,"r");
+               yyin = fopen(filename, "r");
                if(!yyin) {
-                       fprintf(stderr,"Error: can't open file '%s'\n",
+                       fprintf(stderr, "Error: can't open file '%s'\n",
                                filename);
                        exit(1);
                }
@@ -1913,7 +2356,7 @@ main(int argc, char *argv[])
        if(yyparse()!=0)
                g_error("Parsing errors, quitting");
        if(!class)
-               print_error(FALSE," no class defined",0);
+               print_error(FALSE, " no class defined", 0);
        
 
        exit_on_error = FALSE;
@@ -1922,35 +2365,46 @@ main(int argc, char *argv[])
        arguments = count_arguments((Class *)class);
        overrides = count_overrides((Class *)class);
        privates = count_privates((Class *)class);
+       protecteds = count_protecteds((Class *)class);
+       destructors = count_destructors((Class *)class);
+       initializers = count_initializers((Class *)class);
 
        make_bases();
        make_inits((Class *)class);
        if(privates>0)
-               make_destroy((Class *)class);
+               make_finalize((Class *)class);
        check_bad_symbols((Class *)class);
        check_duplicate_symbols((Class *)class);
        check_duplicate_signals_args((Class *)class);
        check_public_new((Class *)class);
        check_vararg((Class *)class);
        check_firstarg((Class *)class);
+       check_nonvoidempty((Class *)class);
+       check_signal_args((Class *)class);
+       check_argument_types((Class *)class);
 
        exit_on_error = TRUE;
        
        if(got_error)
                exit(1);
 
+       any_special = setup_special_array((Class *)class, special_array);
+
        open_files();
        
        generate_outfiles();
 
-       fclose(out);
-       fclose(outh);
+       if(devnull)
+               fclose(devnull);
+       else {
+               fclose(out);
+               fclose(outh);
+               if(outph)
+                       fclose(outph);
+       }
 
-       if(no_touch_headers)
+       if(no_touch_headers && !no_write)
                compare_and_move_header();
        
-#if 0
-       poptFreeContext(optCon);
-#endif
        return 0;
 }