--- a/gntcolors.c Tue Aug 13 20:49:55 2013 +0530
+++ b/gntcolors.c Fri Aug 23 23:46:41 2013 +0530
@@ -296,7 +296,7 @@
return (hascolors ? COLOR_PAIR(pair) :
((pair == GNT_COLOR_NORMAL || pair == GNT_COLOR_HIGHLIGHT_D ||
- pair == GNT_COLOR_TITLE_D || pair == GNT_COLOR_DISABLED) ? 0 : A_STANDOUT));
+ pair == GNT_COLOR_TITLE_D || pair == GNT_COLOR_DISABLED) ? 0 : (int)A_STANDOUT)); int gnt_color_add_pair(int fg, int bg)
--- a/gntentry.c Tue Aug 13 20:49:55 2013 +0530
+++ b/gntentry.c Fri Aug 23 23:46:41 2013 +0530
@@ -184,7 +184,7 @@
show_suggest_dropdown(GntEntry *entry)
@@ -815,7 +815,7 @@
for (str = text; *str; str = next)
next = g_utf8_find_next_char(str, NULL);
@@ -834,7 +834,7 @@
if (entry->max && g_utf8_pointer_to_offset(entry->start, entry->end) >= entry->max)
- if (entry->end + len - entry->start >= entry->buffer)
+ if ((gsize)(entry->end + len - entry->start) >= entry->buffer) /* This will cause the buffer to grow */
char *tmp = g_strdup(entry->start);
@@ -1184,8 +1184,11 @@
g_return_if_fail(entry->history != NULL); /* Need to set_history_length first */
- if (g_list_length(entry->history) >= entry->histlength)
+ if (entry->histlength >= 0 && + g_list_length(entry->history) >= (gsize)entry->histlength) entry->history = g_list_first(entry->history);
g_free(entry->history->data);
--- a/gntentry.h Tue Aug 13 20:49:55 2013 +0530
+++ b/gntentry.h Fri Aug 23 23:46:41 2013 +0530
@@ -178,7 +178,7 @@
* Set the length of history for the entry box.
* @param entry The entry box.
- * @param num The maximum length of the history.
+ * @param num The maximum length of the history, -1 for unlimited. void gnt_entry_set_history_length(GntEntry *entry, int num);
--- a/gntmenu.c Tue Aug 13 20:49:55 2013 +0530
+++ b/gntmenu.c Fri Aug 23 23:46:41 2013 +0530
@@ -79,7 +79,7 @@
GntMenu *menu = GNT_MENU(widget);
if (menu->type == GNT_MENU_TOPLEVEL) {
wbkgdset(widget->window, '\0' | gnt_color_pair(GNT_COLOR_HIGHLIGHT));
@@ -277,7 +277,7 @@
gnt_menu_key_pressed(GntWidget *widget, const char *text)
GntMenu *menu = GNT_MENU(widget);
- int current = menu->selected;
+ guint current = menu->selected; @@ -304,9 +304,10 @@
if (menu->type == GNT_MENU_TOPLEVEL) {
if (strcmp(text, GNT_KEY_LEFT) == 0) {
- if (menu->selected < 0)
+ if (menu->selected == 0) menu->selected = g_list_length(menu->list) - 1;
} else if (strcmp(text, GNT_KEY_RIGHT) == 0) {
if (menu->selected >= g_list_length(menu->list))
--- a/gntmenu.h Tue Aug 13 20:49:55 2013 +0530
+++ b/gntmenu.h Fri Aug 23 23:46:41 2013 +0530
@@ -65,7 +65,7 @@
/* This will keep track of its immediate submenu which is visible so that
* keystrokes can be passed to it. */
--- a/gntstyle.c Tue Aug 13 20:49:55 2013 +0530
+++ b/gntstyle.c Fri Aug 23 23:46:41 2013 +0530
@@ -168,7 +168,7 @@
for (i = 1; i < MAX_WORKSPACES; ++i) {
--- a/gnttextview.c Tue Aug 13 20:49:55 2013 +0530
+++ b/gnttextview.c Fri Aug 23 23:46:41 2013 +0530
@@ -513,7 +513,7 @@
GntTextFormatFlags flags, const char *tagname)
GntWidget *widget = GNT_WIDGET(view);
GList *list = view->list;
--- a/gntwm.c Tue Aug 13 20:49:55 2013 +0530
+++ b/gntwm.c Fri Aug 23 23:46:41 2013 +0530
@@ -427,6 +427,7 @@
w = wm->cws->ordered->data;
orgpos = pos = g_list_index(wm->cws->list, w);
+ g_return_if_fail(pos < 0); @@ -434,7 +435,7 @@
wid = g_list_last(wm->cws->list)->data;
pos = g_list_length(wm->cws->list) - 1;
- } else if (pos >= g_list_length(wm->cws->list)) {
+ } else if ((guint)pos >= g_list_length(wm->cws->list)) { wid = wm->cws->list->data;