From: Christian Heller Date: Sun, 8 Sep 2013 14:01:31 +0000 (+0200) Subject: All members of structs Win and WinMeta are now officially visible externally, removed... X-Git-Tag: tce~968 X-Git-Url: https://plomlompom.com/repos/%7B%7Bdb.prefix%7D%7D/%7B%7B%20web_path%20%7D%7D/%7B%7Bprefix%7D%7D/static/edit?a=commitdiff_plain;h=918a7b6c22411b67f5c5aa2aa4d1c166ad765aca;p=plomrogue All members of structs Win and WinMeta are now officially visible externally, removed "_" prefixes. --- diff --git a/src/command_db.h b/src/command_db.h index 6b08700..f0baef9 100644 --- a/src/command_db.h +++ b/src/command_db.h @@ -19,7 +19,7 @@ struct Command { uint8_t id; /* unique identifier of command */ char * dsc_short; /* short string name of command to be used internally */ - char * dsc_long; /* long string description of command for the user */ + char * dsc_long; /* long string description of command for the user */ }; struct CommandDB diff --git a/src/wincontrol.c b/src/wincontrol.c index 582b631..5ebc057 100644 --- a/src/wincontrol.c +++ b/src/wincontrol.c @@ -55,7 +55,7 @@ static void set_winconf(struct World * world, char id); /* Get WinConf by "id"; get id of WinConf mothering "win". */ static struct WinConf * get_winconf_by_id(struct World * world, char id); -/* Get (Win->_draw) function identified by "c"; NULL if c not mapped to one. */ +/* 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. */ @@ -422,13 +422,13 @@ extern void save_win_configs(struct World * world) FILE * file = try_fopen(path_tmp, "w", world, f_name); char line[6]; - struct Win * w_p = world->wmeta->_chain_start; + struct Win * w_p = world->wmeta->chain_start; uint8_t i = 0; while (0 != w_p) { struct WinConf * wc = get_winconf_by_win(world, w_p); line[i] = wc->id; - w_p = w_p->_next; + w_p = w_p->next; i++; } line[i] = '\n'; @@ -458,12 +458,12 @@ extern void toggle_winconfig(struct World * world, struct Win * win) struct WinConf * wcp = get_winconf_by_win(world, win); if (0 == wcp->view) { - win->_draw = draw_winconf; + win->draw = draw_winconf; wcp->view = 1; } else { - win->_draw = get_drawfunc_by_char(wcp->draw); + win->draw = get_drawfunc_by_char(wcp->draw); wcp->view = 0; } } diff --git a/src/windows.c b/src/windows.c index 080880c..df96743 100644 --- a/src/windows.c +++ b/src/windows.c @@ -66,16 +66,16 @@ static uint8_t refit_pad(struct WinMeta * wmeta) { /* Determine rightmost window column. */ uint32_t lastwincol = 0; - struct Win * w_p = wmeta->_chain_start; + struct Win * w_p = wmeta->chain_start; while (w_p != 0) { - if ((uint32_t) w_p->_start.x + (uint32_t) w_p->frame.size.x + if ((uint32_t) w_p->start.x + (uint32_t) w_p->frame.size.x > lastwincol + 1) { - lastwincol = (uint32_t) w_p->_start.x + lastwincol = (uint32_t) w_p->start.x + (uint32_t) w_p->frame.size.x - 1; } - w_p = w_p->_next; + w_p = w_p->next; } /* Only resize the pad if the rightmost window column has changed. */ @@ -108,15 +108,15 @@ static uint8_t update_wins(struct WinMeta * wmeta, struct Win * w) } WINDOW * subpad_test = subpad(wmeta->padframe.curses_win, w->frame.size.y, w->frame.size.x, - w->_start.y, w->_start.x); + w->start.y, w->start.x); if (NULL == subpad_test) { return 1; } w->frame.curses_win = subpad_test; - if (0 != w->_next) + if (0 != w->next) { - return update_wins(wmeta, w->_next); + return update_wins(wmeta, w->next); } return 0; } @@ -126,29 +126,29 @@ static uint8_t update_wins(struct WinMeta * wmeta, struct Win * w) static void place_win(struct WinMeta * wmeta, struct Win * w) { /* First window goes into the upper-left corner. */ - w->_start.x = 0; - w->_start.y = 1; /* Leave space for title bar. */ - if (0 != w->_prev) + w->start.x = 0; + w->start.y = 1; /* Leave space for title bar. */ + if (0 != w->prev) { /* Non-first window fallbacks to: fit rightwards of rightmost border. */ - struct Win * w_top = w->_prev; - while (w_top->_start.y != 1) + struct Win * w_top = w->prev; + while (w_top->start.y != 1) { - w_top = w_top->_prev; + w_top = w_top->prev; } - w->_start.x = w_top->_start.x + w_top->frame.size.x + 1; + w->start.x = w_top->start.x + w_top->frame.size.x + 1; /* Fit window below its predecessor if that one directly thrones over * empty space wide and high enough. */ - uint16_t w_prev_maxy = w->_prev->_start.y - + getmaxy(w->_prev->frame.curses_win); - if ( w->frame.size.x <= w->_prev->frame.size.x + uint16_t w_prev_maxy = w->prev->start.y + + getmaxy(w->prev->frame.curses_win); + if ( w->frame.size.x <= w->prev->frame.size.x && w->frame.size.y < wmeta->padframe.size.y - w_prev_maxy) { - w->_start.x = w->_prev->_start.x; - w->_start.y = w_prev_maxy + 1; + w->start.x = w->prev->start.x; + w->start.y = w_prev_maxy + 1; } /* Failing that, try to open a new sub column below the nearest @@ -156,29 +156,29 @@ static void place_win(struct WinMeta * wmeta, struct Win * w) */ else { - struct Win * w_up = w->_prev; + struct Win * w_up = w->prev; struct Win * w_upup = w_up; uint16_t widthdiff; while (w_up != w_top) { - w_upup = w_up->_prev; + w_upup = w_up->prev; while (1) { - if (w_up->_start.y != w_upup->_start.y) + if (w_up->start.y != w_upup->start.y) { break; } - w_upup = w_upup->_prev; + w_upup = w_upup->prev; } - w_prev_maxy = w_upup->_start.y + w_prev_maxy = w_upup->start.y + getmaxy(w_upup->frame.curses_win); - widthdiff = (w_upup->_start.x + w_upup->frame.size.x) - - (w_up->_start.x + w_up->frame.size.x); + widthdiff = (w_upup->start.x + w_upup->frame.size.x) + - (w_up->start.x + w_up->frame.size.x); if ( w->frame.size.y < wmeta->padframe.size.y - w_prev_maxy && w->frame.size.x < widthdiff) { - w->_start.x = w_up->_start.x + w_up->frame.size.x + 1 ; - w->_start.y = w_prev_maxy + 1; + w->start.x = w_up->start.x + w_up->frame.size.x + 1 ; + w->start.y = w_prev_maxy + 1; break; } w_up = w_upup; @@ -199,10 +199,10 @@ static void destroy_win(struct Win * w) static void draw_wins(struct Win * w) { - w->_draw(w); - if (0 != w->_next) + w->draw(w); + if (0 != w->next) { - draw_wins(w->_next); + draw_wins(w->next); } } @@ -212,15 +212,15 @@ static void draw_win_borderlines(struct Win * w, char active, WINDOW * pad) { /* Draw vertical and horizontal border lines. */ uint16_t y, x; - for (y = w->_start.y; y <= w->_start.y + w->frame.size.y; y++) + for (y = w->start.y; y <= w->start.y + w->frame.size.y; y++) { - mvwaddch(pad, y, w->_start.x - 1, '|'); - mvwaddch(pad, y, w->_start.x + w->frame.size.x, '|'); + mvwaddch(pad, y, w->start.x - 1, '|'); + mvwaddch(pad, y, w->start.x + w->frame.size.x, '|'); } - for (x = w->_start.x; x <= w->_start.x + w->frame.size.x; x++) + for (x = w->start.x; x <= w->start.x + w->frame.size.x; x++) { - mvwaddch(pad, w->_start.y - 1, x, '-'); - mvwaddch(pad, w->_start.y + w->frame.size.y, x, '-'); + mvwaddch(pad, w->start.y - 1, x, '-'); + mvwaddch(pad, w->start.y + w->frame.size.y, x, '-'); } /* Draw as much as possible of the title into center of top border line. */ @@ -228,21 +228,21 @@ static void draw_win_borderlines(struct Win * w, char active, WINDOW * pad) if (w->frame.size.x >= min_title_length_visible) { uint16_t title_offset = 0; - if (w->frame.size.x > strlen(w->_title) + 2) + if (w->frame.size.x > strlen(w->title) + 2) { - title_offset = (w->frame.size.x - (strlen(w->_title) + 2)) / 2; + title_offset = (w->frame.size.x - (strlen(w->title) + 2)) / 2; } /* +2 is for padding/decoration */ - uint16_t length_visible = strnlen(w->_title, w->frame.size.x - 2); + uint16_t length_visible = strnlen(w->title, w->frame.size.x - 2); char title[length_visible + 3]; char decoration = ' '; if (1 == active) { decoration = '$'; } - memcpy(title + 1, w->_title, length_visible); + memcpy(title + 1, w->title, length_visible); title[0] = title[length_visible + 1] = decoration; title[length_visible + 2] = '\0'; - mvwaddstr(pad, w->_start.y - 1, w->_start.x + title_offset, title); + mvwaddstr(pad, w->start.y - 1, w->start.x + title_offset, title); } } @@ -257,9 +257,9 @@ static void draw_wins_borderlines(struct Win * w, struct Win * w_active, active = 1; } draw_win_borderlines(w, active, pad); - if (0 != w->_next) + if (0 != w->next) { - draw_wins_borderlines(w->_next, w_active, pad); + draw_wins_borderlines(w->next, w_active, pad); } } @@ -267,14 +267,14 @@ static void draw_wins_borderlines(struct Win * w, struct Win * w_active, static void draw_wins_bordercorners(struct Win * w, WINDOW * pad) { - mvwaddch(pad, w->_start.y - 1, w->_start.x - 1, '+'); - mvwaddch(pad, w->_start.y - 1, w->_start.x + w->frame.size.x, '+'); - mvwaddch(pad, w->_start.y + w->frame.size.y, w->_start.x - 1, '+'); + mvwaddch(pad, w->start.y - 1, w->start.x - 1, '+'); + mvwaddch(pad, w->start.y - 1, w->start.x + w->frame.size.x, '+'); + mvwaddch(pad, w->start.y + w->frame.size.y, w->start.x - 1, '+'); mvwaddch(pad, - w->_start.y + w->frame.size.y, w->_start.x + w->frame.size.x, '+'); - if (0 != w->_next) + w->start.y + w->frame.size.y, w->start.x + w->frame.size.x, '+'); + if (0 != w->next) { - draw_wins_bordercorners(w->_next, pad); + draw_wins_bordercorners(w->next, pad); } } @@ -282,40 +282,40 @@ static void draw_wins_bordercorners(struct Win * w, WINDOW * pad) static void shift_win_forward(struct WinMeta * wmeta) { - if (wmeta->active == wmeta->_chain_end) + if (wmeta->active == wmeta->chain_end) { - wmeta->_chain_end = wmeta->active->_prev; - wmeta->_chain_end->_next = 0; - wmeta->active->_next = wmeta->_chain_start; - wmeta->active->_next->_prev = wmeta->active; - wmeta->_chain_start = wmeta->active; - wmeta->_chain_start->_prev = 0; + wmeta->chain_end = wmeta->active->prev; + wmeta->chain_end->next = 0; + wmeta->active->next = wmeta->chain_start; + wmeta->active->next->prev = wmeta->active; + wmeta->chain_start = wmeta->active; + wmeta->chain_start->prev = 0; } else { - struct Win * old_prev = wmeta->active->_prev; - struct Win * old_next = wmeta->active->_next; - if (wmeta->_chain_end == wmeta->active->_next) + struct Win * old_prev = wmeta->active->prev; + struct Win * old_next = wmeta->active->next; + if (wmeta->chain_end == wmeta->active->next) { - wmeta->_chain_end = wmeta->active; - wmeta->active->_next = 0; + wmeta->chain_end = wmeta->active; + wmeta->active->next = 0; } else { - wmeta->active->_next = old_next->_next; - wmeta->active->_next->_prev = wmeta->active; + wmeta->active->next = old_next->next; + wmeta->active->next->prev = wmeta->active; } - if (wmeta->_chain_start == wmeta->active) + if (wmeta->chain_start == wmeta->active) { - wmeta->_chain_start = old_next; + wmeta->chain_start = old_next; } else { - old_prev->_next = old_next; + old_prev->next = old_next; } - old_next->_prev = old_prev; - old_next->_next = wmeta->active; - wmeta->active->_prev = old_next; + old_next->prev = old_prev; + old_next->next = wmeta->active; + wmeta->active->prev = old_next; } } @@ -323,40 +323,40 @@ static void shift_win_forward(struct WinMeta * wmeta) static void shift_win_backward(struct WinMeta * wmeta) { - if (wmeta->active == wmeta->_chain_start) + if (wmeta->active == wmeta->chain_start) { - wmeta->_chain_start = wmeta->active->_next; - wmeta->_chain_start->_prev = 0; - wmeta->active->_prev = wmeta->_chain_end; - wmeta->active->_prev->_next = wmeta->active; - wmeta->_chain_end = wmeta->active; - wmeta->_chain_end->_next = 0; + wmeta->chain_start = wmeta->active->next; + wmeta->chain_start->prev = 0; + wmeta->active->prev = wmeta->chain_end; + wmeta->active->prev->next = wmeta->active; + wmeta->chain_end = wmeta->active; + wmeta->chain_end->next = 0; } else { - struct Win * old_prev = wmeta->active->_prev; - struct Win * old_next = wmeta->active->_next; - if (wmeta->_chain_start == wmeta->active->_prev) + struct Win * old_prev = wmeta->active->prev; + struct Win * old_next = wmeta->active->next; + if (wmeta->chain_start == wmeta->active->prev) { - wmeta->_chain_start = wmeta->active; - wmeta->active->_prev = 0; + wmeta->chain_start = wmeta->active; + wmeta->active->prev = 0; } else { - wmeta->active->_prev = old_prev->_prev; - wmeta->active->_prev->_next = wmeta->active; + wmeta->active->prev = old_prev->prev; + wmeta->active->prev->next = wmeta->active; } - if (wmeta->_chain_end == wmeta->active) + if (wmeta->chain_end == wmeta->active) { - wmeta->_chain_end = old_prev; + wmeta->chain_end = old_prev; } else { - old_next->_prev = old_prev; + old_next->prev = old_prev; } - old_prev->_next = old_next; - old_prev->_prev = wmeta->active; - wmeta->active->_next = old_prev; + old_prev->next = old_next; + old_prev->prev = wmeta->active; + wmeta->active->next = old_prev; } } @@ -365,7 +365,7 @@ static void shift_win_backward(struct WinMeta * wmeta) extern uint8_t init_win_meta(WINDOW * screen, struct WinMeta ** wmp) { struct WinMeta * wmeta = malloc(sizeof(struct WinMeta)); - wmeta->_screen = screen; + wmeta->screen = screen; uint32_t maxy_test = getmaxy(screen); uint32_t maxx_test = getmaxx(screen); if (maxy_test > UINT16_MAX || maxx_test > UINT16_MAX) @@ -374,8 +374,8 @@ extern uint8_t init_win_meta(WINDOW * screen, struct WinMeta ** wmp) } wmeta->padframe.size.y = maxy_test; wmeta->padframe.size.x = maxx_test; - wmeta->_chain_start = 0; - wmeta->_chain_end = 0; + wmeta->chain_start = 0; + wmeta->chain_end = 0; wmeta->pad_offset = 0; WINDOW * pad_test = newpad(wmeta->padframe.size.y, 1); if (NULL == pad_test) @@ -399,17 +399,17 @@ extern uint8_t init_win(struct WinMeta * wmeta, struct Win ** wp, char * title, { return 1; } - w->_prev = 0; - w->_next = 0; - w->frame.curses_win = 0; - w->_title = malloc(strlen(title) + 1); - if (NULL == w->_title) + w->prev = 0; + w->next = 0; + w->frame.curses_win = 0; + w->title = malloc(strlen(title) + 1); + if (NULL == w->title) { return 1; } - sprintf(w->_title, "%s", title); + sprintf(w->title, "%s", title); w->data = data; - w->_draw = func; + w->draw = func; if (0 < width) { w->frame.size.x = width; @@ -446,7 +446,7 @@ extern void free_win(struct Win * win) { delwin(win->frame.curses_win); } - free(win->_title); + free(win->title); free(win); } @@ -454,17 +454,17 @@ extern void free_win(struct Win * win) extern uint8_t append_win(struct WinMeta * wmeta, struct Win * w) { - if (0 != wmeta->_chain_start) + if (0 != wmeta->chain_start) { - w->_prev = wmeta->_chain_end; - wmeta->_chain_end->_next = w; + w->prev = wmeta->chain_end; + wmeta->chain_end->next = w; } else { wmeta->active = w; - wmeta->_chain_start = w; + wmeta->chain_start = w; } - wmeta->_chain_end = w; + wmeta->chain_end = w; return update_wins(wmeta, w); } @@ -474,23 +474,23 @@ extern uint8_t suspend_win(struct WinMeta * wmeta, struct Win * w) { destroy_win(w); - if (wmeta->_chain_start != w) + if (wmeta->chain_start != w) { - w->_prev->_next = w->_next; + w->prev->next = w->next; } else { - wmeta->_chain_start = w->_next; + wmeta->chain_start = w->next; } char pad_refitted = 0; - if (wmeta->_chain_end != w) + if (wmeta->chain_end != w) { - w->_next->_prev = w->_prev; + w->next->prev = w->prev; if (wmeta->active == w) { - wmeta->active = w->_next; + wmeta->active = w->next; } - uint8_t test = update_wins(wmeta, w->_next);/* Positioning of */ + uint8_t test = update_wins(wmeta, w->next); /* Positioning of */ if (0 != test) /* successor windows may */ { /* be affected / need */ return test; /* correction. Note that */ @@ -499,15 +499,15 @@ extern uint8_t suspend_win(struct WinMeta * wmeta, struct Win * w) } /* later need for that. */ else { - wmeta->_chain_end = w->_prev; + wmeta->chain_end = w->prev; if (wmeta->active == w) { - wmeta->active = w->_prev; + wmeta->active = w->prev; } } - w->_prev = 0; - w->_next = 0; + w->prev = 0; + w->next = 0; if (0 == pad_refitted) { @@ -551,24 +551,24 @@ extern void cycle_active_win(struct WinMeta * wmeta, char dir) { if ('f' == dir) { - if (wmeta->active->_next != 0) + if (wmeta->active->next != 0) { - wmeta->active = wmeta->active->_next; + wmeta->active = wmeta->active->next; } else { - wmeta->active = wmeta->_chain_start; + wmeta->active = wmeta->chain_start; } } else { - if (wmeta->active->_prev != 0) + if (wmeta->active->prev != 0) { - wmeta->active = wmeta->active->_prev; + wmeta->active = wmeta->active->prev; } else { - wmeta->active = wmeta->_chain_end; + wmeta->active = wmeta->chain_end; } } } @@ -579,7 +579,7 @@ extern void cycle_active_win(struct WinMeta * wmeta, char dir) extern uint8_t shift_active_win(struct WinMeta * wmeta, char dir) { if ( 0 == wmeta->active /* No shifting with < 2 */ - || wmeta->_chain_start == wmeta->_chain_end) /* windows visible. */ + || wmeta->chain_start == wmeta->chain_end) /* windows visible. */ { return 0; } @@ -591,7 +591,7 @@ extern uint8_t shift_active_win(struct WinMeta * wmeta, char dir) { shift_win_backward(wmeta); } - return update_wins(wmeta, wmeta->_chain_start); + return update_wins(wmeta, wmeta->chain_start); } @@ -600,16 +600,16 @@ extern uint8_t draw_all_wins(struct WinMeta * wmeta) { /* Empty everything before filling it a-new. */ erase(); - wnoutrefresh(wmeta->_screen); + wnoutrefresh(wmeta->screen); werase(wmeta->padframe.curses_win); - if (wmeta->_chain_start) + if (wmeta->chain_start) { /* Draw windows' contents first, then their borders. */ - draw_wins(wmeta->_chain_start); - draw_wins_borderlines(wmeta->_chain_start, wmeta->active, + draw_wins(wmeta->chain_start); + draw_wins_borderlines(wmeta->chain_start, wmeta->active, wmeta->padframe.curses_win); - draw_wins_bordercorners(wmeta->_chain_start,wmeta->padframe.curses_win); + draw_wins_bordercorners(wmeta->chain_start,wmeta->padframe.curses_win); /* Draw virtual screen scroll hints. */ if (wmeta->pad_offset > 0) diff --git a/src/windows.h b/src/windows.h index 4914a27..f5f4056 100644 --- a/src/windows.h +++ b/src/windows.h @@ -58,16 +58,13 @@ struct Frame /* If Frame is Win's "frame", "size" is the */ struct Win { - /* members supposed to be used ONLY INTERNALLY */ - struct Win * _prev; /* chain pointers; if 0, they mark the start or end */ - struct Win * _next; /* of the chain; if both are 0, Win is outside chain */ - struct yx_uint16 _start; /* upper left corner of "frame" WINDOW */ - char * _title; /* title to be used in window title bar */ - void (* _draw) (struct Win *); /* how to draw window content ("data") */ - - /* members to be available EXTERNALLY */ + struct Win * prev; /* chain pointers; if 0, they mark the start or end */ + struct Win * next; /* of the chain; if both are 0, Win is outside chain */ + struct yx_uint16 start; /* upper left corner of "frame" WINDOW */ + char * title; /* title to be used in window title bar */ + void (* draw) (struct Win *); /* how to draw window content ("data") */ struct Frame frame; - void * data; /* window content to be drawn by _draw() */ + void * data; /* window content to be drawn by _draw() */ }; @@ -77,15 +74,12 @@ struct Win */ struct WinMeta { - /* members supposed to be used ONLY INTERNALLY */ - WINDOW * _screen; /* ncurses' pointer to the terminal screen */ - struct Win * _chain_start; /* first Win in chain; its _prev == 0 */ - struct Win * _chain_end; /* last Win in chain; its _next == 0 */ - - /* members to be available EXTERNALLY */ - uint16_t pad_offset; /* number of cells view is moved to the right */ - struct Frame padframe; /* virtual screen fitted into terminal screen */ - struct Win * active; /* Win highlighted/selected for manipulation */ + WINDOW * screen; /* ncurses' pointer to the terminal screen */ + struct Win * chain_start; /* first Win in chain; its _prev == 0 */ + struct Win * chain_end; /* last Win in chain; its _next == 0 */ + uint16_t pad_offset; /* number of cells view is moved to the right */ + struct Frame padframe; /* virtual screen fitted into terminal screen */ + struct Win * active; /* Win highlighted/selected for manipulation */ };