[hackers] [surf] using char instead of gchar. || Enno Boland (tox)

From: <hg_AT_suckless.org>
Date: Fri, 16 Oct 2009 14:30:56 +0000 (UTC)

changeset: 126:ab5aa4a1cb65
tag: tip
user: Enno Boland (tox) <tox_AT_s01.de>
date: Fri Oct 16 16:33:18 2009 +0200
files: config.def.h surf.c
description:
using char instead of gchar.

diff -r f69b381f3788 -r ab5aa4a1cb65 config.def.h
--- a/config.def.h Fri Oct 16 15:35:11 2009 +0200
+++ b/config.def.h Fri Oct 16 16:33:18 2009 +0200
@@ -1,11 +1,11 @@
 /* modifier 0 means no modifier */
-static gchar *useragent = "Mozilla/5.0 (X11; U; Linux; en-us) AppleWebKit/531.2+ (KHTML, like Gecko, surf-"VERSION") Safari/531.2+";
-static gchar *progress = "#FF0000";
-static gchar *progress_trust = "#00FF00";
-static gchar *stylefile = ".surf/style.css";
-static gchar *scriptfile = ".surf/script.js";
-static gchar *cookiefile = ".surf/cookie.txt";
-static gchar *dldir = ".surf/dl";
+static char *useragent = "Mozilla/5.0 (X11; U; Linux; en-us) AppleWebKit/531.2+ (KHTML, like Gecko, surf-"VERSION") Safari/531.2+";
+static char *progress = "#FF0000";
+static char *progress_trust = "#00FF00";
+static char *stylefile = ".surf/style.css";
+static char *scriptfile = ".surf/script.js";
+static char *cookiefile = ".surf/cookie.txt";
+static char *dldir = ".surf/dl";
 #define MODKEY GDK_CONTROL_MASK
 static Key keys[] = {
     /* modifier keyval function arg Focus */
@@ -36,10 +36,9 @@
 };
 
 static Item items[] = {
- { "Open", loaduri, { .v = NULL } },
     { "New Window", newwindow, { .v = NULL } },
+ { "Reload", reload, { .b = FALSE } },
     { "Stop", stop, { 0 } },
     { "<===", navigate, { .i = -1 } },
     { "===>", navigate, { .i = +1 } },
- { "suckless", newwindow, { .v = (char *)"http://suckless.org" } },
 };
diff -r f69b381f3788 -r ab5aa4a1cb65 surf.c
--- a/surf.c Fri Oct 16 15:35:11 2009 +0200
+++ b/surf.c Fri Oct 16 16:33:18 2009 +0200
@@ -31,9 +31,10 @@
 
 typedef struct Client {
         GtkWidget *win, *scroll, *vbox, *uribar, *searchbar, *indicator;
+ GtkWidget **items;
         WebKitWebView *view;
         WebKitDownload *download;
- gchar *title, *linkhover;
+ char *title, *linkhover;
         gint progress;
         struct Client *next;
 } Client;
@@ -45,10 +46,10 @@
 } Item;
 
 typedef struct Cookie {
- gchar *name;
- gchar *value;
- gchar *domain;
- gchar *path;
+ char *name;
+ char *value;
+ char *domain;
+ char *path;
         struct Cookie *next;
 } Cookie;
 
@@ -76,29 +77,29 @@
 static GdkNativeWindow embed = 0;
 static gboolean showxid = FALSE;
 static gboolean ignore_once = FALSE;
-static gchar winid[64];
-static gchar *progname;
+static char winid[64];
+static char *progname;
 
-static const gchar *autouri(Client *c);
-static gchar *buildpath(const gchar *path);
+static const char *autouri(Client *c);
+static char *buildpath(const char *path);
 static void cleanup(void);
 static void clipboard(Client *c, const Arg *arg);
 static void context(WebKitWebView *v, GtkMenu *m, Client *c);
-static gchar *copystr(gchar **str, const gchar *src);
+static char *copystr(char **str, const char *src);
 static gboolean decidewindow(WebKitWebView *v, WebKitWebFrame *f, WebKitNetworkRequest *r, WebKitWebNavigationAction *n, WebKitWebPolicyDecision *p, Client *c);
 static void destroyclient(Client *c);
 static void destroywin(GtkWidget* w, Client *c);
-static void die(gchar *str);
+static void die(char *str);
 static void download(WebKitDownload *o, GParamSpec *pspec, Client *c);
 static void drawindicator(Client *c);
 static gboolean exposeindicator(GtkWidget *w, GdkEventExpose *e, Client *c);
 static gboolean initdownload(WebKitWebView *v, WebKitDownload *o, Client *c);
-static gchar *geturi(Client *c);
+static char *geturi(Client *c);
 static void hidesearch(Client *c, const Arg *arg);
 static void hideuri(Client *c, const Arg *arg);
 static void itemclick(GtkMenuItem *mi, Client *c);
 static gboolean keypress(GtkWidget *w, GdkEventKey *ev, Client *c);
-static void linkhover(WebKitWebView *v, const gchar* t, const gchar* l, Client *c);
+static void linkhover(WebKitWebView *v, const char* t, const char* l, Client *c);
 static void loadcommit(WebKitWebView *v, WebKitWebFrame *f, Client *c);
 static void loadstart(WebKitWebView *v, WebKitWebFrame *f, Client *c);
 static void loaduri(Client *c, const Arg *arg);
@@ -106,7 +107,7 @@
 static Client *newclient(void);
 static void newwindow(Client *c, const Arg *arg);
 static WebKitWebView *createwindow(WebKitWebView *v, WebKitWebFrame *f, Client *c);
-static void pasteuri(GtkClipboard *clipboard, const gchar *text, gpointer d);
+static void pasteuri(GtkClipboard *clipboard, const char *text, gpointer d);
 static GdkFilterReturn processx(GdkXEvent *xevent, GdkEvent *event, gpointer d);
 static void print(Client *c, const Arg *arg);
 static void proccookies(SoupMessage *m, Client *c);
@@ -115,17 +116,17 @@
 static void reload(Client *c, const Arg *arg);
 static void rereadcookies(void);
 static void sigchld(int unused);
-static void setcookie(gchar *name, gchar *val, gchar *dom, gchar *path, long exp);
+static void setcookie(char *name, char *val, char *dom, char *path, long exp);
 static void setup(void);
 static void spawn(Client *c, const Arg *arg);
-static void titlechange(WebKitWebView *v, WebKitWebFrame* frame, const gchar* title, Client *c);
+static void titlechange(WebKitWebView *v, WebKitWebFrame* frame, const char* title, Client *c);
 static void scroll(Client *c, const Arg *arg);
 static void searchtext(Client *c, const Arg *arg);
 static void source(Client *c, const Arg *arg);
 static void showsearch(Client *c, const Arg *arg);
 static void showuri(Client *c, const Arg *arg);
 static void stop(Client *c, const Arg *arg);
-static void titlechange(WebKitWebView *v, WebKitWebFrame* frame, const gchar* title, Client *c);
+static void titlechange(WebKitWebView *v, WebKitWebFrame* frame, const char* title, Client *c);
 static gboolean focusview(GtkWidget *w, GdkEventFocus *e, Client *c);
 static void usage(void);
 static void update(Client *c);
@@ -136,7 +137,7 @@
 /* configuration, allows nested code to access above variables */
 #include "config.h"
 
-const gchar *
+const char *
 autouri(Client *c) {
         if(GTK_WIDGET_HAS_FOCUS(c->uribar))
                 return gtk_entry_get_text(GTK_ENTRY(c->uribar));
@@ -145,9 +146,9 @@
         return NULL;
 }
 
-gchar *
-buildpath(const gchar *path) {
- gchar *apath, *p;
+char *
+buildpath(const char *path) {
+ char *apath, *p;
         FILE *f;
 
         /* creating directory */
@@ -189,22 +190,24 @@
 void
 context(WebKitWebView *v, GtkMenu *m, Client *c) {
         int i;
- GtkWidget *item;
+ GtkContainer *parent;
 
         gtk_widget_hide_all(GTK_WIDGET(m));
         gtk_widget_show(GTK_WIDGET(m));
         for(i = 0; i < LENGTH(items); i++) {
- item = gtk_menu_item_new_with_label(items[i].label);
- gtk_menu_shell_append(GTK_MENU_SHELL(m), item);
- g_signal_connect(G_OBJECT(item), "activate",
+ parent = GTK_CONTAINER(gtk_widget_get_parent(c->items[i]));
+ if(parent)
+ gtk_container_remove(parent, c->items[i]);
+ gtk_menu_shell_append(GTK_MENU_SHELL(m), c->items[i]);
+ g_signal_connect(G_OBJECT(c->items[i]), "activate",
                                 G_CALLBACK(itemclick), c);
- gtk_widget_show(item);
+ gtk_widget_show(c->items[i]);
         }
 }
 
-gchar *
-copystr(gchar **str, const gchar *src) {
- gchar *tmp;
+char *
+copystr(char **str, const char *src) {
+ char *tmp;
         tmp = g_strdup(src);
 
         if(str && *str) {
@@ -216,6 +219,7 @@
 
 void
 destroyclient(Client *c) {
+ int i;
         Client *p;
 
         gtk_widget_destroy(GTK_WIDGET(c->view));
@@ -224,6 +228,10 @@
         gtk_widget_destroy(c->searchbar);
         gtk_widget_destroy(c->vbox);
         gtk_widget_destroy(c->win);
+ for(i = 0; i < LENGTH(items); i++)
+ gtk_widget_destroy(c->items[i]);
+ free(c->items);
+
         for(p = clients; p && p->next != c; p = p->next);
         if(p)
                 p->next = c->next;
@@ -252,7 +260,7 @@
 }
 
 void
-die(gchar *str) {
+die(char *str) {
         fputs(str, stderr);
         exit(EXIT_FAILURE);
 }
@@ -260,7 +268,7 @@
 void
 drawindicator(Client *c) {
         gint width;
- gchar *uri;
+ char *uri;
         GtkWidget *w;
         GdkGC *gc;
         GdkColor fg;
@@ -299,8 +307,8 @@
 
 gboolean
 initdownload(WebKitWebView *view, WebKitDownload *o, Client *c) {
- const gchar *filename;
- gchar *uri, *html;
+ const char *filename;
+ char *uri, *html;
 
         stop(c, NULL);
         c->download = o;
@@ -322,11 +330,11 @@
         return TRUE;
 }
 
-gchar *
+char *
 geturi(Client *c) {
- gchar *uri;
+ char *uri;
 
- if(!(uri = (gchar *)webkit_web_view_get_uri(c->view)))
+ if(!(uri = (char *)webkit_web_view_get_uri(c->view)))
                 uri = copystr(NULL, "about:blank");
         return uri;
 }
@@ -346,7 +354,7 @@
 void
 itemclick(GtkMenuItem *mi, Client *c) {
         int i;
- const gchar *label;
+ const char *label;
 
         label = gtk_menu_item_get_label(mi);
         for(i = 0; i < LENGTH(items); i++)
@@ -381,7 +389,7 @@
 }
 
 void
-linkhover(WebKitWebView *v, const gchar* t, const gchar* l, Client *c) {
+linkhover(WebKitWebView *v, const char* t, const char* l, Client *c) {
         if(l)
                 c->linkhover = copystr(&c->linkhover, l);
         else if(c->linkhover) {
@@ -393,7 +401,7 @@
 
 void
 loadcommit(WebKitWebView *view, WebKitWebFrame *f, Client *c) {
- gchar *uri;
+ char *uri;
 
         ignore_once = TRUE;
         uri = geturi(c);
@@ -410,8 +418,8 @@
 
 void
 loaduri(Client *c, const Arg *arg) {
- gchar *u;
- const gchar *uri = (gchar *)arg->v;
+ char *u;
+ const char *uri = (char *)arg->v;
 
         if(!uri)
                 uri = autouri(c);
@@ -434,9 +442,10 @@
 
 Client *
 newclient(void) {
+ int i;
         Client *c;
         WebKitWebSettings *settings;
- gchar *uri;
+ char *uri;
 
         if(!(c = calloc(1, sizeof(Client))))
                 die("Cannot malloc!\n");
@@ -452,6 +461,14 @@
         g_signal_connect(G_OBJECT(c->win), "destroy", G_CALLBACK(destroywin), c);
         g_signal_connect(G_OBJECT(c->win), "key-press-event", G_CALLBACK(keypress), c);
 
+ if(!(c->items = calloc(1, sizeof(GtkWidget *) * LENGTH(items))))
+ die("Cannot malloc!\n");
+
+ /* contextmenu */
+ for(i = 0; i < LENGTH(items); i++)
+ c->items[i] = gtk_menu_item_new_with_label(items[i].label);
+
+
         /* VBox */
         c->vbox = gtk_vbox_new(FALSE, 0);
 
@@ -534,9 +551,9 @@
 void
 newwindow(Client *c, const Arg *arg) {
         guint i = 0;
- const gchar *cmd[7], *uri;
+ const char *cmd[7], *uri;
         const Arg a = { .v = (void *)cmd };
- gchar tmp[64];
+ char tmp[64];
 
         cmd[i++] = progname;
         if(embed) {
@@ -562,7 +579,7 @@
 }
 
 void
-pasteuri(GtkClipboard *clipboard, const gchar *text, gpointer d) {
+pasteuri(GtkClipboard *clipboard, const char *text, gpointer d) {
         Arg arg = {.v = text };
         if(text != NULL)
                 loaduri((Client *) d, &arg);
@@ -663,7 +680,7 @@
 }
 
 void
-setcookie(gchar *name, gchar *val, gchar *dom, gchar *path, long exp) {
+setcookie(char *name, char *val, char *dom, char *path, long exp) {
 
 }
 
@@ -712,7 +729,7 @@
 
 void
 searchtext(Client *c, const Arg *arg) {
- const gchar *text;
+ const char *text;
         gboolean forward = *(gboolean *)arg;
         text = gtk_entry_get_text(GTK_ENTRY(c->searchbar));
         webkit_web_view_search_text(c->view, text, FALSE, forward, TRUE);
@@ -721,7 +738,7 @@
 
 void
 showuri(Client *c, const Arg *arg) {
- gchar *uri;
+ char *uri;
 
         hidesearch(c, NULL);
         uri = geturi(c);
@@ -745,15 +762,15 @@
                 if(dpy)
                         close(ConnectionNumber(dpy));
                 setsid();
- execvp(((gchar **)arg->v)[0], (gchar **)arg->v);
- fprintf(stderr, "tabbed: execvp %s", ((gchar **)arg->v)[0]);
+ execvp(((char **)arg->v)[0], (char **)arg->v);
+ fprintf(stderr, "tabbed: execvp %s", ((char **)arg->v)[0]);
                 perror(" failed");
                 exit(0);
         }
 }
 
 void
-titlechange(WebKitWebView *v, WebKitWebFrame *f, const gchar *t, Client *c) {
+titlechange(WebKitWebView *v, WebKitWebFrame *f, const char *t, Client *c) {
         c->title = copystr(&c->title, t);
         update(c);
 }
@@ -773,7 +790,7 @@
 
 void
 update(Client *c) {
- gchar *t;
+ char *t;
 
         if(c->progress != 100)
                 t = g_strdup_printf("%s [%i%%]", c->title, c->progress);
@@ -796,7 +813,7 @@
 void
 windowobjectcleared(GtkWidget *w, WebKitWebFrame *frame, JSContextRef js, JSObjectRef win, Client *c) {
         JSStringRef jsscript;
- gchar *script;
+ char *script;
         JSValueRef exception = NULL;
         GError *error;
         
Received on Fri Oct 16 2009 - 14:30:56 UTC

This archive was generated by hypermail 2.2.0 : Fri Oct 16 2009 - 14:36:06 UTC