summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorrcyeske <rcyeske>2001-03-02 05:36:09 +0000
committerrcyeske <rcyeske>2001-03-02 05:36:09 +0000
commit58e95be4eab6d82257214d76414c9eef901c5915 (patch)
treedba695f796439ef0deb375d6248d5843c7d5771b
parent11932954e0122c115be23862aabfacb565a4cbed (diff)
downloadratpoison-58e95be4eab6d82257214d76414c9eef901c5915.zip
* actions.c (cmd_source): Open the file. Error report as
appropriate. (cmd_windows): Call current_screen() to find the current screen. (cmd_clock): Show the clock even if no windows are open. (cmd_clock): Use xmalloc, remove error check. (initialize_default_keybindings): Use xmalloc. (cmd_bind): Use xmalloc. (cmd_rename): Use xmalloc, remove error check. (add_keybinding): Use xrealloc. (replace_keybinding): Use xrealloc. * actions.c: Use PRINT_DEBUG instead of fprintf. Put useful error text in calls to message(). (cmd_select): Show the window list if there is no such window number.
-rw-r--r--src/ChangeLog8
-rw-r--r--src/actions.c33
2 files changed, 16 insertions, 25 deletions
diff --git a/src/ChangeLog b/src/ChangeLog
index 58d8802..8de2f21 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -2,6 +2,14 @@
* actions.c (cmd_source): Open the file. Error report as
appropriate.
+ (cmd_windows): Call current_screen() to find the current screen.
+ (cmd_clock): Show the clock even if no windows are open.
+ (cmd_clock): Use xmalloc, remove error check.
+ (initialize_default_keybindings): Use xmalloc.
+ (cmd_bind): Use xmalloc.
+ (cmd_rename): Use xmalloc, remove error check.
+ (add_keybinding): Use xrealloc.
+ (replace_keybinding): Use xrealloc.
* main.c (read_rc_file): Take a file pointer rather than a
filename.
diff --git a/src/actions.c b/src/actions.c
index f3ebef8..32c8059 100644
--- a/src/actions.c
+++ b/src/actions.c
@@ -57,7 +57,7 @@ add_keybinding (int keysym, int state, char *cmd)
{
/* double the key table size */
key_actions_table_size *= 2;
- key_actions = (rp_action*) realloc (key_actions, sizeof (rp_action) * key_actions_table_size);
+ key_actions = (rp_action*) xrealloc (key_actions, sizeof (rp_action) * key_actions_table_size);
PRINT_DEBUG ("realloc()ed key_table %d\n", key_actions_table_size);
}
@@ -83,7 +83,7 @@ void
initialize_default_keybindings (void)
{
key_actions_table_size = 1;
- key_actions = (rp_action*) malloc (sizeof (rp_action) * key_actions_table_size);
+ key_actions = (rp_action*) xmalloc (sizeof (rp_action) * key_actions_table_size);
key_actions_last = 0;
prefix_key.sym = KEY_PREFIX;
@@ -223,7 +223,7 @@ cmd_bind (void *data)
return;
}
- keydesc = (char*) malloc (strlen (data + 1));
+ keydesc = (char*) xmalloc (strlen (data + 1));
sscanf (data, "%s", keydesc);
cmd = data + strlen (keydesc);
@@ -440,12 +440,7 @@ cmd_rename (void *data)
if (*winname)
{
free (rp_current_window->name);
- rp_current_window->name = malloc (sizeof (char) * strlen (winname) + 1);
- if (rp_current_window->name == NULL)
- {
- PRINT_ERROR ("Out of memory\n");
- exit (EXIT_FAILURE);
- }
+ rp_current_window->name = xmalloc (sizeof (char) * strlen (winname) + 1);
strcpy (rp_current_window->name, winname);
@@ -634,16 +629,10 @@ cmd_newwm(void *data)
void
cmd_clock (void *data)
{
- screen_info *s;
char *msg;
time_t timep;
- msg = malloc(9);
- if(msg == NULL)
- {
- PRINT_DEBUG ("Error allocation memory in show_clock()\n");
- return;
- }
+ msg = xmalloc (9);
timep = time(NULL);
if(timep == ((time_t)-1))
@@ -655,11 +644,8 @@ cmd_clock (void *data)
strncpy(msg, ctime(&timep) + 11, 8); /* FIXME: a little bit hardcoded looking */
msg[8] = '\0';
- if (rp_current_window)
- {
- s = rp_current_window->scr;
- message (msg);
- }
+ message (msg);
+
free(msg);
}
@@ -670,10 +656,7 @@ cmd_windows (void *data)
{
screen_info *s;
- if (rp_current_window)
- s = rp_current_window->scr;
- else
- s = &screens[0];
+ s = current_screen ();
if (!hide_bar (s)) show_bar (s);
}