From 3f2b7a8d7dc649f5afecedf7b630ae964ec5bfe4 Mon Sep 17 00:00:00 2001 From: Christian Heller Date: Mon, 25 Nov 2013 00:21:09 +0100 Subject: [PATCH] Minor code-stylistic and comment improvements mostly in wincontrol library. --- src/main.c | 2 +- src/rexit.c | 4 ++-- src/rexit.h | 2 +- src/wincontrol.c | 30 ++++++++++++++---------------- src/windows.c | 6 +++--- 5 files changed, 21 insertions(+), 23 deletions(-) diff --git a/src/main.c b/src/main.c index 39837fe..4571438 100644 --- a/src/main.c +++ b/src/main.c @@ -159,7 +159,7 @@ int main(int argc, char *argv[]) keypad(world.wmeta->screen, TRUE); raw(); load_interface_conf(); - set_cleanup_flag(CLEANUP_INTERFACE_CONF); + set_cleanup_flag(CLEANUP_INTERFACE); /* Focus map on player. */ struct MapObj * player = get_player(); diff --git a/src/rexit.c b/src/rexit.c index c96adc9..691cb68 100644 --- a/src/rexit.c +++ b/src/rexit.c @@ -9,7 +9,7 @@ #include "map.h" /* for Map struct */ #include "keybindings.h" /* for free_keybindings() */ #include "command_db.h" /* for free_command_db() */ -#include "windows.h" /* for Win struct, free_win(), free_winmeta_and_endwin() */ +#include "windows.h" /* for Win struct, free_winmeta_and_endwin() */ #include "map_objects.h" /* for free_map_objects, free_map_object_defs() */ #include "misc.h" /* for unload_interface_conf() */ #include "map_object_actions.h" /* for free_map_object_actions() */ @@ -48,7 +48,7 @@ static void cleanup() { free(world.map->cells); } - if (cleanup_flags & CLEANUP_INTERFACE_CONF) + if (cleanup_flags & CLEANUP_INTERFACE) { unload_interface_conf(); } diff --git a/src/rexit.h b/src/rexit.h index a28efc2..ab8e8ec 100644 --- a/src/rexit.h +++ b/src/rexit.h @@ -26,7 +26,7 @@ enum cleanup_flag CLEANUP_MAPOBJACTS = 0x0010, CLEANUP_MAP_OBJECTS = 0x0020, CLEANUP_MAP_OBJECT_DEFS = 0x0040, - CLEANUP_INTERFACE_CONF = 0x0080 + CLEANUP_INTERFACE = 0x0080 }; extern void set_cleanup_flag(enum cleanup_flag flag); diff --git a/src/wincontrol.c b/src/wincontrol.c index 7830636..658a8ce 100644 --- a/src/wincontrol.c +++ b/src/wincontrol.c @@ -4,10 +4,8 @@ #include /* for free() */ #include /* for strlen(), strchr(), strstr() */ #include /* for uint8_t, uint16_t */ -#include /* for access(), unlink() */ #include "windows.h" /* for suspend_win(), append_win(), reset_pad_offset(), - * resize_active_win(), init_win(), free_win(), - * structs Win, WinMeta + * resize_active_win(), init_win(), free_win(), struct Win */ #include "yx_uint16.h" /* for yx_uint16 struct */ #include "main.h" /* for world global */ @@ -15,10 +13,9 @@ * try_fgets(), try_fclose_unlink_rename(), try_fwrite() */ #include "rexit.h" /* for exit_err() */ -#include "main.h" /* for world global */ -#include "draw_wins.h" /* for draw_win_map(), draw_win_info(), draw_win_og(), +#include "draw_wins.h" /* for draw_win_map(), draw_win_info(), draw_win_log(), * draw_win_available_keybindings(), - * draw_win_keybindings_global(), draw_win_inventory(), + * draw_win_inventory(), draw_win_keybindings_global(), * draw_win_keybindings_winconf_geometry(), * draw_win_keybindings_winconf_keybindings(), * draw_winconf_geometry(), draw_winconf_keybindings() @@ -33,7 +30,7 @@ /* Return string "prefix" + "id"; malloc()'s string, remember to call free()! */ static char * string_prefixed_id(char * prefix, char id); -/* Create Winconf, init ->view/height_type/width_type to 0, ->id to "id". */ +/* Initializes Winconf: .view/.height_type/.width_type to 0, .id to "id". */ static void create_winconf(char id, struct WinConf * wcp); /* Initialize Winconf of "id" from appropriate config file.*/ @@ -45,7 +42,7 @@ static void init_win_from_winconf(char id); /* Save title, draw function, size of window identified by "id" to conffile. */ static void save_win_config(char id); -/* Free data pointed to inside WinConf struct. */ +/* Free data pointed to inside individual WinConf struct of "id". */ static void free_winconf_data(char id); /* Write geometry of a window to its WinConf, as positive or negative values @@ -59,7 +56,7 @@ static struct WinConf * get_winconf_by_id(char id); /* Get (Win->draw) function identified by "c"; NULL if c not mapped to one. */ static void * get_drawfunc_by_char(char c); -/* Iterate over bytes of world.winconf_ids array. Re-start after null byte. */ +/* Iterate over chars of world.winconf_ids array. Re-start after null byte. */ static char get_next_winconf_id(); @@ -306,11 +303,9 @@ static char get_next_winconf_id() if (0 == c) { i = 0; + return c; } - else - { - i++; - } + i++; return c; } @@ -342,12 +337,13 @@ extern struct Win * get_win_by_id(char id) extern void init_winconfs() { char * f_name = "init_winconfs()"; - char * err_o = "Trouble in init_winconfs() with opendir()."; - char * err_r = "Trouble in init_winconfs() with readdir()."; - char * err_c = "Trouble in init_winconfs() with closedir()."; + char * err_o = trouble_msg(f_name, "opendir()"); + char * err_r = trouble_msg(f_name, "readdir()"); + char * err_c = trouble_msg(f_name, "closedir()"); DIR * dp = opendir("config/windows"); exit_err(NULL == dp, err_o); + free(err_o); struct dirent * fn; errno = 0; char * winconf_ids = try_malloc(256, f_name); @@ -364,7 +360,9 @@ extern void init_winconfs() } winconf_ids[i] = '\0'; exit_err(errno, err_r); + free(err_r); exit_err(closedir(dp), err_c); + free(err_c); world.winconf_ids = try_malloc(strlen(winconf_ids) + 1, f_name); memcpy(world.winconf_ids, winconf_ids, strlen(winconf_ids) + 1); free(winconf_ids); diff --git a/src/windows.c b/src/windows.c index a0c42f1..42f2859 100644 --- a/src/windows.c +++ b/src/windows.c @@ -323,10 +323,10 @@ static void draw_wins_borderlines(struct Win * w) static void draw_wins_bordercorners(struct Win * w) { mvwaddch(world.wmeta->pad, w->start.y - 1, w->start.x - 1, '+'); - mvwaddch(world.wmeta->pad, w->start.y - 1, w->start.x + w->framesize.x, '+'); - mvwaddch(world.wmeta->pad, w->start.y + w->framesize.y, w->start.x - 1, '+'); + mvwaddch(world.wmeta->pad, w->start.y - 1, w->start.x + w->framesize.x,'+'); + mvwaddch(world.wmeta->pad, w->start.y + w->framesize.y, w->start.x - 1,'+'); mvwaddch(world.wmeta->pad, w->start.y + w->framesize.y, - w->start.x + w->framesize.x,'+'); + w->start.x + w->framesize.x, '+'); if (0 != w->next) { draw_wins_bordercorners(w->next); -- 2.30.2