]> git.draconx.ca Git - gob-dx.git/blobdiff - src/main.c
Release 0.93.4
[gob-dx.git] / src / main.c
index 09f59c254a86bc03f0a357645dbe8cf91a51d8ef..cdbfa42859f79a2ae5dd8d4daecc9df4be2dc3b7 100644 (file)
@@ -85,8 +85,7 @@ 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;
+gint private_header = PRIVATE_HEADER_ALWAYS;
 gboolean no_extern_c = FALSE;
 gboolean no_write = FALSE;
 gboolean no_lines = FALSE;
@@ -309,7 +308,7 @@ add_bad_hack_to_avoid_unused_warnings(Class *c)
                return;
        
        if(!no_gnu)
-               out_printf(out,"\n\n#ifndef __GNUC__\n");
+               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"
@@ -318,7 +317,7 @@ add_bad_hack_to_avoid_unused_warnings(Class *c)
                   "  extentions to make saner, faster code */\n"
                   "static void\n"
                   "___%s_really_bad_hack_to_avoid_warnings(void)\n"
-                  "{\n",funcbase);
+                  "{\n", funcbase);
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *node = li->data;
                if(node->type == METHOD_NODE) {
@@ -330,31 +329,31 @@ add_bad_hack_to_avoid_unused_warnings(Class *c)
                                continue;
 
                        /* in C++ mode we don't alias new */
-                       if(for_cpp && strcmp(m->id,"new")==0)
+                       if(for_cpp && strcmp(m->id, "new")==0)
                                continue;
 
-                       out_printf(out,"\t((void (*)(void))%s)();\n",m->id);
+                       out_printf(out, "\t((void (*)(void))%s)();\n", m->id);
                }
        }
        out_printf(out, "\t___%s_really_bad_hack_to_avoid_warnings();\n",
                   funcbase);
        if(!no_gnu)
-               out_printf(out,"}\n#endif /* __GNUC__ */\n\n");
+               out_printf(out, "}\n#endif /* __GNUC__ */\n\n");
        else
-               out_printf(out,"}\n\n");
+               out_printf(out, "}\n\n");
 }
 
 static void
 put_variable(Variable *v, FILE *fp)
 {
-       out_printf(fp,"\t");
-       print_type(fp,v->vtype,FALSE);
-       out_printf(fp,"%s%s;",v->id,
+       out_printf(fp, "\t");
+       print_type(fp, v->vtype, FALSE);
+       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");
+               out_printf(fp, " /* protected */");
+       out_printf(fp, "\n");
 }
 
 static void
@@ -367,10 +366,10 @@ put_vs_method(Method *m)
 
        /* if a signal mark it as such */
        if(m->method != VIRTUAL_METHOD)
-               print_method(outh,"\t/*signal*/","(* ","",") ",";\n",m,
-                            FALSE,TRUE);
+               print_method(outh, "\t/*signal*/", "(* ", "", ") ", ";\n", m,
+                            FALSE, TRUE);
        else
-               print_method(outh,"\t","(* ","",") ",";\n",m,FALSE,TRUE);
+               print_method(outh, "\t", "(* ", "", ") ", ";\n", m, FALSE, TRUE);
 }
 
 static void
@@ -379,7 +378,7 @@ put_pub_method(Method *m)
        if(m->scope != PUBLIC_SCOPE)
                return;
 
-       print_method(outh,"","\t","","\t",";\n",m,TRUE,FALSE);
+       print_method(outh, "", "\t", "", "\t", ";\n", m, TRUE, FALSE);
 }
 
 /* I'm starting not to like this idea */
@@ -427,9 +426,9 @@ put_prot_method(Method *m)
                return;
 
        if(outph)
-               print_method(outph,"","\t","","\t",";\n",m,FALSE,FALSE);
+               print_method(outph, "", "\t", "", "\t", ";\n", m, FALSE, FALSE);
        else
-               print_method(out,"","\t","","\t",";\n",m,FALSE,FALSE);
+               print_method(out, "", "\t", "", "\t", ";\n", m, FALSE, FALSE);
 }
 
 static void
@@ -462,12 +461,12 @@ make_func_arg(char *typename, int is_class, char *name)
        char *tn;
        
        if(is_class)
-               tn = g_strconcat(typename,":Class",NULL);
+               tn = g_strconcat(typename, ":Class", NULL);
        else
                tn = g_strdup(typename);
 
-       type = new_type(1,tn,NULL);
-       node = new_funcarg((Type *)type,name,NULL);
+       type = new_type(1, tn, NULL);
+       node = new_funcarg((Type *)type, name, NULL);
        return g_list_prepend(NULL, node);
 }
 
@@ -484,30 +483,30 @@ make_inits(Class *cl)
                        Method *m = (Method *)n;
                        if(m->method == INIT_METHOD) {
                                if(got_init)
-                                       print_error(FALSE,"init defined more then once",m->line_no);
+                                       print_error(FALSE, "init defined more then once", m->line_no);
                                got_init = TRUE;
                        } else if(m->method == CLASS_INIT_METHOD) {
                                if(got_class_init)
-                                       print_error(FALSE,"class_init defined more then once",m->line_no);
+                                       print_error(FALSE, "class_init defined more then once", m->line_no);
                                got_class_init = TRUE;
                        }
                }
        }
        if(!got_class_init) {
                node = new_method(NO_SCOPE, CLASS_INIT_METHOD,
-                                 (Type *)new_type(0, g_strdup("void"),NULL),
+                                 (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);
+               cl->nodes = g_list_prepend(cl->nodes, node);
        }
        if(!got_init) {
                node = new_method(NO_SCOPE, INIT_METHOD,
-                                 (Type *)new_type(0, g_strdup("void"),NULL),
+                                 (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);
+               cl->nodes = g_list_prepend(cl->nodes, node);
        }
 }
 
@@ -527,7 +526,7 @@ make_finalize(Class *cl)
                                        got_finalize = TRUE;
                                        break;
                                }
-                               print_error(FALSE,"finalize method override "
+                               print_error(FALSE, "finalize method override "
                                            "of class other then Gtk:Object",
                                            m->line_no);
                        }
@@ -535,14 +534,14 @@ make_finalize(Class *cl)
        }
        if(!got_finalize) {
                node = new_method(NO_SCOPE, OVERRIDE_METHOD,
-                                 (Type *)new_type(0,g_strdup("void"),NULL),
+                                 (Type *)new_type(0, g_strdup("void"), NULL),
                                  g_strdup("Gtk:Object"),
                                  NULL, NULL, g_strdup("finalize"),
-                                 make_func_arg("Gtk:Object",FALSE,g_strdup("o")),
+                                 make_func_arg("Gtk:Object", FALSE, g_strdup("o")),
                                  NULL, NULL,
                                  g_strdup("PARENT_HANDLER (o);\n"),
-                                 0,0,FALSE);
-               cl->nodes = g_list_append(cl->nodes,node);
+                                 0, 0, FALSE);
+               cl->nodes = g_list_append(cl->nodes, node);
                overrides++;
        }
 }
@@ -561,7 +560,7 @@ 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) {
+               if(strcmp(a->data, b->data)!=0) {
                        return FALSE;
                }
        }
@@ -577,7 +576,7 @@ 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))
+               if(is_list_equal(mm->gtktypes, m->gtktypes))
                        return mm;
        }
        return NULL;
@@ -589,16 +588,16 @@ 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)) {
+               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);
+                                          (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);
+                                          (char *)li->data, i);
                        }
                }
        }
@@ -619,7 +618,7 @@ add_signal_prots(Method *m)
                return;
 
        if(!marsh)
-               marsh = g_hash_table_new(NULL,NULL);
+               marsh = g_hash_table_new(NULL, NULL);
        
        if(strcmp(m->gtktypes->data, "NONE")==0 &&
           strcmp(m->gtktypes->next->data, "NONE")==0)
@@ -628,12 +627,12 @@ add_signal_prots(Method *m)
        /* 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);
+               s = g_hash_table_lookup(marsh, mm);
+               g_hash_table_insert(marsh, m, s);
                return;
        }
        
-       s = g_strdup_printf("Sig%d",sig++);
+       s = g_strdup_printf("Sig%d", sig++);
        
        g_hash_table_insert(marsh, m, s);
        eq_signal_methods = g_list_prepend(eq_signal_methods, m);
@@ -648,12 +647,12 @@ add_signal_prots(Method *m)
        }
        out_printf(out, "gpointer);\n"); 
        
-       out_printf(out,"\nstatic void\n"
+       out_printf(out, "\nstatic void\n"
                "___marshal_%s (GtkObject * object,\n"
                "\tGtkSignalFunc func,\n"
                "\tgpointer func_data,\n"
                "\tGtkArg * args)\n"
-               "{\n",s);
+               "{\n", s);
        
        if(strcmp(m->gtktypes->data, "NONE")==0) {
                out_printf(out, "\t___%s rfunc;\n\n"
@@ -678,9 +677,9 @@ static void
 add_enums(Class *c)
 {
        GList *li;
-       out_printf(out,"\n");
+       out_printf(out, "\n");
        if(signals>0) {
-               out_printf(out,"enum {\n");
+               out_printf(out, "enum {\n");
                for(li=c->nodes;li;li=g_list_next(li)) {
                        Node *n = li->data;
                        if(n->type == METHOD_NODE) {
@@ -689,22 +688,22 @@ add_enums(Class *c)
                                   m->method == SIGNAL_FIRST_METHOD) {
                                        char *s = g_strdup(m->id);
                                        g_strup(s);
-                                       out_printf(out,"\t%s_SIGNAL,\n",s);
+                                       out_printf(out, "\t%s_SIGNAL,\n", s);
                                        g_free(s);
                                }
                        }
                }
-               out_printf(out,"\tLAST_SIGNAL\n};\n\n");
+               out_printf(out, "\tLAST_SIGNAL\n};\n\n");
        }
        if(arguments>0) {
-               out_printf(out,"enum {\n\tARG_0");
+               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,",\n\tARG_%s",s);
+                               out_printf(out, ",\n\tARG_%s", s);
                                g_free(s);
                        }
                }
@@ -716,7 +715,7 @@ add_enums(Class *c)
                           "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);
+       out_printf(out, "static %sClass *parent_class = NULL;\n\n", ptypebase);
 }
 
 static void
@@ -741,8 +740,8 @@ add_get_type(void)
                "\t}\n\n"
                "\treturn type;\n"
                "}\n\n",
-               funcbase,typebase,typebase,typebase,
-               funcbase,funcbase,pfuncbase);
+               funcbase, typebase, typebase, typebase,
+               funcbase, funcbase, pfuncbase);
 }
 
 static void
@@ -767,21 +766,21 @@ add_overrides(Class *c, char *oname, gboolean did_gtk_obj)
                
                s = remove_sep(m->otype);
                
-               if(g_hash_table_lookup(done,s)) {
+               if(g_hash_table_lookup(done, s)) {
                        g_free(s);
                        continue;
                }
-               g_hash_table_insert(done,s,s);
+               g_hash_table_insert(done, s, s);
 
-               f = replace_sep(m->otype,'_');
+               f = replace_sep(m->otype, '_');
                g_strdown(f);
 
-               out_printf(out,"\t%sClass *%s_class = (%sClass *)%s;\n",
-                       s,f,s,oname);
+               out_printf(out, "\t%sClass *%s_class = (%sClass *)%s;\n",
+                       s, f, s, oname);
                
                g_free(f);
        }
-       g_hash_table_foreach(done,(GHFunc)g_free,NULL);
+       g_hash_table_foreach(done, (GHFunc)g_free, NULL);
        g_hash_table_destroy(done);
 }
 
@@ -814,7 +813,7 @@ make_run_signal_flags(Method *m, gboolean last)
                char *flag = li->data;
                int i;
                for(i=0;flags[i];i++) {
-                       if(strcmp(flags[i],flag)==0)
+                       if(strcmp(flags[i], flag)==0)
                                break;
                }
                /* if we haven't found it in our list */
@@ -826,7 +825,7 @@ make_run_signal_flags(Method *m, gboolean last)
                        print_error(TRUE, s, m->line_no);
                        g_free(s);
                }
-               g_string_sprintfa(gs, " | GTK_RUN_%s",flag);
+               g_string_sprintfa(gs, " | GTK_RUN_%s", flag);
        }
 
        {
@@ -842,7 +841,7 @@ add_signals(Class *c)
 {
        GList *li;
 
-       out_printf(out,"\n");
+       out_printf(out, "\n");
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *n = li->data;
                char *mar, *sig, *flags;
@@ -861,7 +860,7 @@ add_signals(Class *c)
 
                if(g_hash_table_lookup(marsh, m))
                        mar = g_strconcat("___marshal_",
-                                         (char *)g_hash_table_lookup(marsh,m),
+                                         (char *)g_hash_table_lookup(marsh, m),
                                          NULL);
                else
                        mar = g_strdup("gtk_signal_default_marshaller");
@@ -871,16 +870,16 @@ add_signals(Class *c)
                sig = g_strdup(m->id);
                g_strup(sig);
                flags = make_run_signal_flags(m, last);
-               out_printf(out,"\tobject_signals[%s_SIGNAL] =\n"
+               out_printf(out, "\tobject_signals[%s_SIGNAL] =\n"
                        "\t\tgtk_signal_new (\"%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,
+                       sig, m->id,
                        flags,
-                       typebase,m->id,mar,(char *)m->gtktypes->data,
+                       typebase, m->id, mar, (char *)m->gtktypes->data,
                        is_none?0:g_list_length(m->gtktypes->next));
                g_free(mar);
                g_free(sig);
@@ -889,13 +888,13 @@ add_signals(Class *c)
                if(!is_none) {
                        GList *l;
                        for(l=m->gtktypes->next;l;l=g_list_next(l))
-                               out_printf(out,",\n\t\t\tGTK_TYPE_%s",
+                               out_printf(out, ",\n\t\t\tGTK_TYPE_%s",
                                        (char *)l->data);
                }
 
-               out_printf(out,");\n");
+               out_printf(out, ");\n");
        }
-       out_printf(out,"\tgtk_object_class_add_signals (gtk_object_class,\n"
+       out_printf(out, "\tgtk_object_class_add_signals (gtk_object_class,\n"
                "\t\tobject_signals, LAST_SIGNAL);\n\n");
 }
 
@@ -905,7 +904,7 @@ set_def_handlers(Class *c, char *oname)
        GList *li;
        gboolean set_line = FALSE;
 
-       out_printf(out,"\n");
+       out_printf(out, "\n");
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *n = li->data;
                Method *m = (Method *)n;
@@ -929,17 +928,17 @@ set_def_handlers(Class *c, char *oname)
                        s = replace_sep(m->otype, '_');
                        g_strdown(s);
                        if(m->cbuf)
-                               out_printf(out,"\t%s_class->%s = %s_%s;\n",
+                               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",
+                               out_printf(out, "\t%s_class->%s = NULL;\n",
                                           s, m->id);
                } else {
                        if(m->cbuf)
-                               out_printf(out,"\t%s->%s = ___real_%s_%s;\n",
+                               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",
+                               out_printf(out, "\t%s->%s = NULL;\n",
                                        oname, m->id);
                }
        }
@@ -959,7 +958,7 @@ make_arguments(Class *c)
                NULL
        };
 
-       out_printf(out,"\n");
+       out_printf(out, "\n");
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *n = li->data;
                Argument *a;
@@ -981,16 +980,16 @@ make_arguments(Class *c)
                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 "
+                       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);
+                                           "set automatically", a->line_no);
                                continue;
                        }
                        for(i=0;argflags[i];i++) {
-                               if(strcmp(argflags[i],flag)==0)
+                               if(strcmp(argflags[i], flag)==0)
                                        break;
                        }
                        /* if we haven't found it in our list */
@@ -999,15 +998,15 @@ make_arguments(Class *c)
                                s = g_strdup_printf("Unknown flag '%s' used, "
                                                    "perhaps it was misspelled",
                                                    flag);
-                               print_error(TRUE,s,a->line_no);
+                               print_error(TRUE, s, a->line_no);
                                g_free(s);
                        }
-                       g_string_sprintfa(flags, " | GTK_ARG_%s",flag);
+                       g_string_sprintfa(flags, " | GTK_ARG_%s", flag);
                }
 
                s = g_strdup(a->name);
                g_strup(s);
-               out_printf(out,"\tgtk_object_add_arg_type(\"%s::%s\",\n"
+               out_printf(out, "\tgtk_object_add_arg_type(\"%s::%s\",\n"
                        "\t\tGTK_TYPE_%s,\n"
                        "\t\t%s,\n"
                        "\t\tARG_%s);\n",
@@ -1059,14 +1058,14 @@ add_inits(Class *c)
                m = (Method *)n;
                if(m->method == INIT_METHOD) {
                        if(m->line_no > 0)
-                               out_addline_infile(out,m->line_no);
+                               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");
+                       out_printf(out, "{\n");
                        if(privates > 0) {
-                               out_printf(out,"\t%s->_priv = "
+                               out_printf(out, "\t%s->_priv = "
                                           "g_new0 (%sPrivate, 1);\n",
                                           ((FuncArg *)m->args->data)->name,
                                           typebase);
@@ -1077,9 +1076,11 @@ add_inits(Class *c)
                                    li != NULL;
                                    li = li->next) {
                                        Node *n = li->data;
-                                       if(n->type != VARIABLE_NODE)
+                                       Variable *v = (Variable *)n;
+                                       if(n->type != VARIABLE_NODE ||
+                                          v->scope == CLASS_SCOPE)
                                                continue;
-                                       print_initializer(m, (Variable *)n);
+                                       print_initializer(m, v);
                                }
                        }
                } else if(m->method == CLASS_INIT_METHOD) {
@@ -1089,7 +1090,7 @@ add_inits(Class *c)
                                     m, FALSE, FALSE);
                        if(m->line_no > 0)
                                out_addline_outfile(out);
-                       out_printf(out,"{\n");
+                       out_printf(out, "{\n");
                        if(signals>0 ||
                           arguments>0)
                                out_printf(out,
@@ -1102,11 +1103,25 @@ add_inits(Class *c)
                                add_overrides(c,
                                              ((FuncArg *)m->args->data)->name,
                                              (signals>0 || arguments>0));
+
+                       if(initializers > 0) {
+                               GList *li;
+                               for(li = ((Class *)class)->nodes;
+                                   li != NULL;
+                                   li = li->next) {
+                                       Node *n = li->data;
+                                       Variable *v = (Variable *)n;
+                                       if(n->type != VARIABLE_NODE ||
+                                          v->scope != CLASS_SCOPE)
+                                               continue;
+                                       print_initializer(m, v);
+                               }
+                       }
                        
-                       out_printf(out,"\n\tparent_class = ");
+                       out_printf(out, "\n\tparent_class = ");
                        if(for_cpp)
-                               out_printf(out,"(%sClass *)",ptypebase);
-                       out_printf(out,"gtk_type_class (%s_get_type ());\n",
+                               out_printf(out, "(%sClass *)", ptypebase);
+                       out_printf(out, "gtk_type_class (%s_get_type ());\n",
                                   pfuncbase);
 
                        if(signals>0)
@@ -1121,15 +1136,15 @@ add_inits(Class *c)
                        continue;
 
                if(m->cbuf) {
-                       out_printf(out," {\n");
-                       out_addline_infile(out,m->ccode_line);
-                       out_printf(out,"%s\n",m->cbuf);
+                       out_printf(out, " {\n");
+                       out_addline_infile(out, m->ccode_line);
+                       out_printf(out, "%s\n", m->cbuf);
                        out_addline_outfile(out);
-                       out_printf(out," }\n");
+                       out_printf(out, " }\n");
                } else {
-                       out_printf(out,"return;\n");
+                       out_printf(out, "return;\n");
                }
-               out_printf(out,"}\n");
+               out_printf(out, "}\n");
        }
 }
 
@@ -1137,15 +1152,15 @@ static void
 add_getset_arg(Class *c, gboolean is_set)
 {
        GList *li;
-       out_printf(out,"\nstatic void\n"
-               "___object_%s_arg (GtkObject *object,\n"
-               "\tGtkArg *arg,\n"
-               "\tguint arg_id)\n"
-               "{\n"
-               "\t%s *self;\n\n"
-               "\tself = %s (object);\n\n"
-               "\tswitch (arg_id) {\n",
-               is_set?"set":"get",typebase,macrobase);
+       out_printf(out, "\nstatic void\n"
+                  "___object_%s_arg (GtkObject *object,\n"
+                  "\tGtkArg *arg,\n"
+                  "\tguint arg_id)\n"
+                  "{\n"
+                  "\t%s *self;\n\n"
+                  "\tself = %s (object);\n\n"
+                  "\tswitch (arg_id) {\n",
+                  is_set?"set":"get", typebase, macrobase);
 
        for(li=c->nodes;li;li=g_list_next(li)) {
                Node *n = li->data;
@@ -1189,11 +1204,11 @@ add_getset_arg(Class *c, gboolean is_set)
                                                   "foo; })\n",
                                                   cast, a->gtktype);
                                }
-                               out_printf(out,"#else /* __GNUC__ */\n");
+                               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, "#endif /* __GNUC__ */\n\n");
                        }
                        out_printf(out, "\t\t{\n");
                        g_free(cast);
@@ -1210,13 +1225,13 @@ add_getset_arg(Class *c, gboolean is_set)
                g_free(s);
                if(line_no > 0)
                        out_addline_infile(out, line_no);
-               out_printf(out,"%s\n",cbuf);
+               out_printf(out, "%s\n", cbuf);
                if(line_no > 0)
                        out_addline_outfile(out);
-               out_printf(out,"\t\t}\n\t\tbreak;\n"
+               out_printf(out, "\t\t}\n\t\tbreak;\n"
                        "#undef ARG\n");
        }
-       out_printf(out,"\tdefault:\n\t\tbreak;\n\t}\n}\n");
+       out_printf(out, "\tdefault:\n\t\tbreak;\n\t}\n}\n");
 }
 
 static void
@@ -1233,7 +1248,7 @@ print_checks(Method *m, FuncArg *fa)
                char *s;
                /* point to the method prot in .gob for failed checks */
                if(m->line_no > 0)
-                       out_addline_infile(out,m->line_no);
+                       out_addline_infile(out, m->line_no);
                if(is_void)
                        out_printf(out, "\tg_return_if_fail (");
                else
@@ -1254,22 +1269,22 @@ print_checks(Method *m, FuncArg *fa)
                        g_free(s);
                        break;
                case LT_CHECK:
-                       out_printf(out,"%s < %s",fa->name,ch->number);
+                       out_printf(out, "%s < %s", fa->name, ch->number);
                        break;
                case GT_CHECK:
-                       out_printf(out,"%s > %s",fa->name,ch->number);
+                       out_printf(out, "%s > %s", fa->name, ch->number);
                        break;
                case LE_CHECK:
-                       out_printf(out,"%s <= %s",fa->name,ch->number);
+                       out_printf(out, "%s <= %s", fa->name, ch->number);
                        break;
                case GE_CHECK:
-                       out_printf(out,"%s >= %s",fa->name,ch->number);
+                       out_printf(out, "%s >= %s", fa->name, ch->number);
                        break;
                case EQ_CHECK:
-                       out_printf(out,"%s == %s",fa->name,ch->number);
+                       out_printf(out, "%s == %s", fa->name, ch->number);
                        break;
                case NE_CHECK:
-                       out_printf(out,"%s != %s",fa->name,ch->number);
+                       out_printf(out, "%s != %s", fa->name, ch->number);
                        break;
                }
                if(is_void)
@@ -1291,7 +1306,7 @@ print_preconditions(Method *m)
        for(li=m->args;li;li=g_list_next(li)) {
                FuncArg *fa = li->data;
                if(fa->checks)
-                       print_checks(m,fa);
+                       print_checks(m, fa);
        }
        if(m->line_no>0)
                out_addline_outfile(out);
@@ -1344,7 +1359,7 @@ 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",
+                       out_printf(out, "\t%s *___self = %s (%s);\n",
                                   typebase, macrobase,
                                   ((FuncArg *)m->args->data)->name);
                }
@@ -1354,13 +1369,14 @@ put_in_gen_code(Method *m)
                            li != NULL;
                            li = li->next) {
                                Node *n = li->data;
-                               if(n->type == VARIABLE_NODE)
-                                       print_destructor("___self",
-                                                        (Variable *)n);
+                               Variable *v = (Variable *)n;
+                               if(n->type == VARIABLE_NODE &&
+                                  v->scope != CLASS_SCOPE)
+                                       print_destructor("___self", v);
                        }
                }
                if(privates > 0) {
-                       out_printf(out,"\tg_free (___self->_priv);\n"
+                       out_printf(out, "\tg_free (___self->_priv);\n"
                                   "\t___self->_priv = NULL;\n",
                                   macrobase,
                                   ((FuncArg *)m->args->data)->name,
@@ -1373,7 +1389,7 @@ put_in_gen_code(Method *m)
 static void
 print_method_body(Method *m, int pre)
 {
-       out_printf(out,"{\n");
+       out_printf(out, "{\n");
        if(pre)
                print_preconditions(m);
 
@@ -1402,7 +1418,7 @@ put_signal_args(Method *m)
 {
        GList *li;
        GList *ali;
-       for(ali = m->gtktypes->next,li=m->args->next;
+       for(ali = m->gtktypes->next, li=m->args->next;
            li && ali;
            li=li->next, ali=ali->next) {
                FuncArg *fa = li->data;
@@ -1411,7 +1427,7 @@ put_signal_args(Method *m)
                   the we know all the types */
                g_assert(cast);
 
-               out_printf(out,",\n\t\t(%s)%s",cast,
+               out_printf(out, ",\n\t\t(%s)%s", cast,
                           fa->name);
        }
 }
@@ -1436,11 +1452,11 @@ get_arg_names_for_macro(Method *m)
 static void
 put_method(Method *m)
 {
-       char *s,*args,*doc;
+       char *s, *args, *doc;
        gboolean is_void;
-       is_void = (strcmp(m->mtype->name,"void")==0 &&
+       is_void = (strcmp(m->mtype->name, "void")==0 &&
                   m->mtype->stars == 0);
-       out_printf(out,"\n");
+       out_printf(out, "\n");
        doc = get_gtk_doc(m->id);
        if(doc) {
                out_printf(out, "%s", doc);
@@ -1449,10 +1465,10 @@ put_method(Method *m)
        switch(m->method) {
        case REGULAR_METHOD:
                if(m->line_no>0)
-                       out_addline_infile(out,m->line_no);
+                       out_addline_infile(out, m->line_no);
                if(m->scope == PRIVATE_SCOPE)
-                       print_method(out,"static ","\n",""," ","\n",
-                                    m,FALSE,FALSE);
+                       print_method(out, "static ", "\n", "", " ", "\n",
+                                    m, FALSE, FALSE);
                else /* PUBLIC, PROTECTED */
                        print_method(out, "", "\n", "", " ", "\n",
                                     m, FALSE, FALSE);
@@ -1461,25 +1477,25 @@ put_method(Method *m)
        case SIGNAL_FIRST_METHOD:
        case SIGNAL_LAST_METHOD:
                if(m->line_no>0)
-                       out_addline_infile(out,m->line_no);
+                       out_addline_infile(out, m->line_no);
                if(m->scope == PRIVATE_SCOPE)
-                       print_method(out,"static ","\n",""," ","\n",
-                                    m,FALSE,FALSE);
+                       print_method(out, "static ", "\n", "", " ", "\n",
+                                    m, FALSE, FALSE);
                else /* PUBLIC, PROTECTED */
-                       print_method(out,"","\n",""," ","\n",m,FALSE,FALSE);
+                       print_method(out, "", "\n", "", " ", "\n", m, FALSE, FALSE);
                out_addline_outfile(out);
-               out_printf(out,"{\n");
+               out_printf(out, "{\n");
                s = g_strdup(m->id);
                g_strup(s);
-               if(strcmp(m->mtype->name,"void")==0 &&
+               if(strcmp(m->mtype->name, "void")==0 &&
                   m->mtype->stars==0) {
                        print_preconditions(m);
                        if(((FuncArg *)m->args->data)->name)
-                       out_printf(out,"\tgtk_signal_emit (GTK_OBJECT (%s),\n"
+                       out_printf(out, "\tgtk_signal_emit (GTK_OBJECT (%s),\n"
                                "\t\tobject_signals[%s_SIGNAL]",
-                               ((FuncArg *)m->args->data)->name,s);
+                               ((FuncArg *)m->args->data)->name, s);
                        put_signal_args(m);
-                       out_printf(out,");\n}\n");
+                       out_printf(out, ");\n}\n");
                } else {
                        out_printf(out, "\t");
                        print_type(out, m->mtype, TRUE);
@@ -1492,54 +1508,54 @@ put_method(Method *m)
                        else
                                out_printf(out, ")(0);\n");
                        print_preconditions(m);
-                       out_printf(out,"\tgtk_signal_emit (GTK_OBJECT (%s),\n"
+                       out_printf(out, "\tgtk_signal_emit (GTK_OBJECT (%s),\n"
                                "\t\tobject_signals[%s_SIGNAL]",
-                               ((FuncArg *)m->args->data)->name,s);
+                               ((FuncArg *)m->args->data)->name, s);
                        put_signal_args(m);
-                       out_printf(out,",\n\t\t&return_val);\n"
+                       out_printf(out, ",\n\t\t&return_val);\n"
                                "\treturn return_val;\n}\n");
                }
 
                if(!m->cbuf)
                        break;
                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);
+                       out_addline_infile(out, m->line_no);
+               print_method(out, "static ", "\n___real_", "", " ", "\n",
+                            m, FALSE, FALSE);
+               print_method_body(m, FALSE);
                break;
        case VIRTUAL_METHOD:
                if(m->line_no>0)
-                       out_addline_infile(out,m->line_no);
+                       out_addline_infile(out, m->line_no);
                if(m->scope==PRIVATE_SCOPE)
-                       print_method(out,"static ","\n",""," ","\n",
-                                    m,FALSE,FALSE);
+                       print_method(out, "static ", "\n", "", " ", "\n",
+                                    m, FALSE, FALSE);
                else /* PUBLIC, PROTECTED */
-                       print_method(out,"","\n",""," ","\n",m,FALSE,FALSE);
+                       print_method(out, "", "\n", "", " ", "\n", m, FALSE, FALSE);
                out_addline_outfile(out);
-               out_printf(out,"{\n"
-                       "\t%sClass *klass;\n",typebase);
+               out_printf(out, "{\n"
+                       "\t%sClass *klass;\n", typebase);
                print_preconditions(m);
-               out_printf(out,"\tklass = %s_CLASS(GTK_OBJECT(%s)->klass);\n\n"
+               out_printf(out, "\tklass = %s_CLASS(GTK_OBJECT(%s)->klass);\n\n"
                        "\tif(klass->%s)\n",
                        macrobase, ((FuncArg *)m->args->data)->name, m->id);
-               if(strcmp(m->mtype->name,"void")==0 &&
+               if(strcmp(m->mtype->name, "void")==0 &&
                   m->mtype->stars==0) {
                        GList *li;
-                       out_printf(out,"\t\t(*klass->%s)(%s",m->id,
+                       out_printf(out, "\t\t(*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, ",%s", fa->name);
                        }
-                       out_printf(out,");\n}\n");
+                       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, ",%s", fa->name);
                        }
                        out_printf(out, ");\n"
                                "\telse\n"
@@ -1556,41 +1572,41 @@ put_method(Method *m)
                if(!m->cbuf)
                        break;
                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);
+                       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:
                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,'_');
+                       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(%s) \\\n"
+                       out_printf(out, "#define PARENT_HANDLER(%s) \\\n"
                                   "\t{ if(%s_CLASS(parent_class)->%s) \\\n"
                                   "\t\t(* %s_CLASS(parent_class)->%s)(%s); }\n",
-                                  args,s,m->id,s,m->id,args);
+                                  args, s, m->id, s, m->id, args);
                } else {
-                       out_printf(out,"#define PARENT_HANDLER(%s) \\\n"
+                       out_printf(out, "#define PARENT_HANDLER(%s) \\\n"
                                   "\t((%s_CLASS(parent_class)->%s)? \\\n"
                                   "\t\t(* %s_CLASS(parent_class)->%s)(%s): \\\n"
                                   "\t\t(",
-                                  args,s,m->id,s,m->id,args);
-                       out_printf(out,"(");
-                       print_type(out,m->mtype,TRUE);
-                       out_printf(out,")%s))\n",
+                                  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");
+               print_method_body(m, TRUE);
+               out_printf(out, "#undef PARENT_HANDLER\n");
                break;
        default:
                break;
@@ -1600,19 +1616,20 @@ put_method(Method *m)
 static void
 open_files(void)
 {
-       char *outfile,*outfileh,*outfileph;
+       char *outfile, *outfileh, *outfileph;
 
        if(!for_cpp)
-               outfile = g_strconcat(filebase,".c",NULL);
+               outfile = g_strconcat(filebase, ".c", NULL);
        else
-               outfile = g_strconcat(filebase,".cc",NULL);
+               outfile = g_strconcat(filebase, ".cc", NULL);
        if(no_touch_headers)
-               outfileh = g_strconcat("#gob#",filebase,".h#gob#",NULL);
+               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)
+       if((privates > 0 || protecteds > 0 ||
+           private_header == PRIVATE_HEADER_ALWAYS) &&
+          !private_header != PRIVATE_HEADER_NEVER)
                outfileph = g_strconcat(filebase, "-private.h", NULL);
        else
                outfileph = NULL;
@@ -1709,7 +1726,7 @@ put_argument_gnu_wrappers(Class *c)
                s = g_strdup(a->name);
                g_strup(s);
                if(a->atype)
-                       cast = get_type(a->atype,TRUE);
+                       cast = get_type(a->atype, TRUE);
                else
                        cast = g_strdup(get_cast(a->gtktype, TRUE));
                if(cast) {
@@ -1767,7 +1784,7 @@ print_ccode_block(CCode *cc)
        default:
        case C_CCODE:
                fp = out;
-               out_printf(fp,"\n");
+               out_printf(fp, "\n");
                out_addline_infile(fp, cc->line_no);
                break;
        case PH_CCODE:
@@ -1815,18 +1832,18 @@ print_class_block(Class *c)
        out_printf(outh, "\n/*\n"
                   " * Type checking and casting macros\n"
                   " */\n");
-       out_printf(outh,"#define %s\t"
+       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"
+                  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);
+                  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);
@@ -1835,22 +1852,22 @@ print_class_block(Class *c)
 
        if(privates>0) {
                out_printf(outh, "\n/* Private structure type */\n");
-               out_printf(outh,"typedef struct _%sPrivate %sPrivate;\n",
-                          typebase,typebase);
+               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,'_');
+       s = replace_sep(c->otype, '_');
        g_strup(s);
-       out_printf(outh,"#ifndef __TYPEDEF_%s__\n"
-                  "#define __TYPEDEF_%s__\n",s,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);
+       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;
@@ -1858,10 +1875,10 @@ print_class_block(Class *c)
                if(n->type == VARIABLE_NODE &&
                   v->scope == PUBLIC_SCOPE) {
                        if(!printed_public) {
-                               out_printf(outh,"\t/*< public >*/\n");
+                               out_printf(outh, "\t/*< public >*/\n");
                                printed_public = TRUE;
                        }
-                       put_variable((Variable *)n,outh);
+                       put_variable((Variable *)n, outh);
                }
        }
        /* put protecteds always AFTER publics */
@@ -1871,18 +1888,18 @@ print_class_block(Class *c)
                if(n->type == VARIABLE_NODE &&
                   v->scope == PROTECTED_SCOPE) {
                        if(!printed_private) {
-                               out_printf(outh,"\t/*< private >*/\n");
+                               out_printf(outh, "\t/*< private >*/\n");
                                printed_private = TRUE;
                        }
-                       put_variable((Variable *)n,outh);
+                       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, "\t/*< private >*/\n");
+               out_printf(outh, "\t%sPrivate *_priv;\n", typebase);
        }
-       out_printf(outh,"};\n");
+       out_printf(outh, "};\n");
 
        if(privates>0) {
                FILE *outfp;
@@ -1895,39 +1912,47 @@ print_class_block(Class *c)
                else
                        outfp = out;
 
-               out_printf(outfp,"struct _%sPrivate {\n",
+               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_infile(outfp, v->line_no);
+                               put_variable(v, outfp);
                        }
                }
                out_addline_outfile(outfp);
-               out_printf(outfp,"};\n");
+               out_printf(outfp, "};\n");
        }
 
        out_printf(outh, "\n/*\n"
                   " * Class definition\n"
                   " */\n");
-       out_printf(outh,"typedef struct _%sClass %sClass;\n",
-                  typebase,typebase);
+       out_printf(outh, "typedef struct _%sClass %sClass;\n",
+                  typebase, typebase);
        out_printf(outh,
                   "struct _%sClass {\n\t%sClass __parent__;\n",
-                  typebase,ptypebase);
+                  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");
+       /* put class scope variables */
+       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 == CLASS_SCOPE)
+                       put_variable((Variable *)n, outh);
+       }
+       out_printf(outh, "};\n\n");
 
-       out_printf(out,"/* here are local prototypes */\n");
+       out_printf(out, "/* here are local prototypes */\n");
        if(arguments>0) {
-               out_printf(out,"static void ___object_set_arg "
+               out_printf(out, "static void ___object_set_arg "
                           "(GtkObject *object, GtkArg *arg, "
                           "guint arg_id);\n"
                           "static void ___object_get_arg "
@@ -1939,7 +1964,7 @@ print_class_block(Class *c)
                   " * Public methods\n"
                   " */\n");
 
-       out_printf(outh,"guint\t%s_get_type\t(void);\n",funcbase);
+       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) {
@@ -1972,11 +1997,11 @@ print_class_block(Class *c)
                out_printf(outh, "\n/*\n"
                           " * Argument wrapping macros\n"
                           " */\n");
-               out_printf(outh,"#ifdef __GNUC__\n");
+               out_printf(outh, "#ifdef __GNUC__\n");
                put_argument_gnu_wrappers(c);
-               out_printf(outh,"#else /* __GNUC__ */\n");
+               out_printf(outh, "#else /* __GNUC__ */\n");
                put_argument_nongnu_wrappers(c);
-               out_printf(outh,"#endif /* __GNUC__ */\n\n");
+               out_printf(outh, "#endif /* __GNUC__ */\n\n");
        } else if(arguments>0 && no_gnu) {
                out_printf(outh, "\n/*\n"
                           " * Argument wrapping macros\n"
@@ -2000,15 +2025,15 @@ print_class_block(Class *c)
                if(no_gnu)
                        make_method_nongnu_aliases(c);
                else {
-                       out_printf(out,"\n#ifdef __GNUC__\n");
+                       out_printf(out, "\n#ifdef __GNUC__\n");
                        make_method_gnu_aliases(c);
-                       out_printf(out,"#else /* __GNUC__ */\n");
+                       out_printf(out, "#else /* __GNUC__ */\n");
                        make_method_nongnu_aliases(c);
-                       out_printf(out,"#endif /* __GNUC__ */\n\n");
+                       out_printf(out, "#endif /* __GNUC__ */\n\n");
                }
        }
 
-       out_printf(out,"/* a macro for creating a new object of our type */\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);
@@ -2073,7 +2098,7 @@ print_includes(void)
        if(outph) {
                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",
+                       out_printf(out, "#include \"%s-private.h\"\n\n",
                                   filebase);
                        if(found_header)
                                print_error(TRUE,
@@ -2154,7 +2179,7 @@ print_header_top(void)
        GList *li;
 
        /* mandatory include */
-       out_printf(outh,"#include <gtk/gtk.h>\n\n");
+       out_printf(outh, "#include <gtk/gtk.h>\n\n");
 
        /* print the HT_CCODE blocks */
        for(li=nodes;li;li=g_list_next(li)) {
@@ -2202,8 +2227,8 @@ generate_outfiles(void)
 static void
 print_help(void)
 {
-       fprintf(stderr,"Gob version %s\n\n",VERSION);
-       fprintf(stderr,"Options:\n"
+       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"
@@ -2216,7 +2241,10 @@ print_help(void)
                                          "really changed\n"
                "\t--always-private-header Always create a private header "
                                          "file,\n"
-               "\t                        even if it would be empty\n"
+               "\t                        even if it would be empty "
+                                         "[default]\n"
+               "\t--ondemand-private-header Create private header only when "
+                                         "needed\n"
                "\t--no-private-header     Don't create a private header, "
                                          "put private\n"
                "\t                        structure and protected "
@@ -2245,10 +2273,10 @@ parse_options(int argc, char *argv[])
                        }
                        filename = argv[i];
                        got_file = TRUE;
-               } else if(strcmp(argv[i],"--help")==0) {
+               } else if(strcmp(argv[i], "--help")==0) {
                        print_help();
                        exit(0);
-               } else if(strcmp(argv[i],"--version")==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) {
@@ -2259,12 +2287,12 @@ parse_options(int argc, char *argv[])
                        for_cpp = TRUE;
                } else if(strcmp(argv[i], "--no-touch-headers")==0) {
                        no_touch_headers = TRUE;
+               } else if(strcmp(argv[i], "--ondemand-private-header")==0) {
+                       private_header == PRIVATE_HEADER_ONDEMAND;
                } else if(strcmp(argv[i], "--always-private-header")==0) {
-                       no_private_header = FALSE;
-                       always_private_header = TRUE;
+                       private_header == PRIVATE_HEADER_ALWAYS;
                } else if(strcmp(argv[i], "--no-private-header")==0) {
-                       always_private_header = FALSE;
-                       no_private_header = TRUE;
+                       private_header == PRIVATE_HEADER_NEVER;
                } else if(strcmp(argv[i], "--no-gnu")==0) {
                        no_gnu = TRUE;
                } else if(strcmp(argv[i], "--no-extern-c")==0) {
@@ -2276,7 +2304,7 @@ parse_options(int argc, char *argv[])
                } 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]);
                        print_help();
@@ -2315,7 +2343,7 @@ compare_and_move_header(void)
        char *hfnew = g_strconcat("#gob#", filebase, ".h#gob#", NULL);
        char *hf = g_strconcat(filebase, ".h", NULL);
        struct stat s;
-       if(stat(hf,&s)==0) {
+       if(stat(hf, &s)==0) {
                char *s;
                s = g_strdup_printf("cmp '%s' '%s' > /dev/null", hf, hfnew);
                if(system(s)==0) {
@@ -2331,7 +2359,7 @@ compare_and_move_header(void)
                if(unlink(hf)!=0)
                        print_error(FALSE, "Can't remove old header file", 0);
        }
-       if(rename(hfnew,hf)!=0)
+       if(rename(hfnew, hf)!=0)
                print_error(FALSE, "Can't rename new header file", 0);
        g_free(hfnew);
        g_free(hf);