simplify osx_write_post_select()
[paraslash.git] / sdl_gui.c
index bdc809b..97d0573 100644 (file)
--- a/sdl_gui.c
+++ b/sdl_gui.c
@@ -20,6 +20,7 @@
 
 #include "para.h"
 #include "string.h"
+#include "fd.h"
 
 
 #include <SDL/SDL.h>
@@ -42,7 +43,7 @@ static int width = 0;
 static int height = 0;
 
 extern const char *status_item_list[NUM_STAT_ITEMS];
-struct gengetopt_args_info args_info;
+struct sdl_gui_args_info args_info;
 
 #define FRAME_WIDTH 10
 #define FONT_HEIGHT 36
@@ -68,19 +69,19 @@ struct gengetopt_args_info args_info;
 
 
 struct stat_item{
-        char *name;
-        char *prefix;
-        char *postfix;
-        char *content;
-        unsigned x;
-        unsigned y;
-        unsigned w;
-        unsigned h;
+       const char *name;
+       const char *prefix;
+       const char *postfix;
+       char *content;
+       unsigned x;
+       unsigned y;
+       unsigned w;
+       unsigned h;
        Uint8 r;
        Uint8 g;
        Uint8 b;
-        int font;
-        int align;
+       int font;
+       int align;
 };
 
 struct font {
@@ -118,7 +119,7 @@ struct font fonts[] = {
 
 static struct stat_item stat_items[NUM_STAT_ITEMS];
 
-void para_log(__unused int ll, __unused char* fmt,...) /* no logging */
+void para_log(__a_unused int ll, __a_unused const char* fmt,...) /* no logging */
 {
 }
 
@@ -135,7 +136,6 @@ static void init_stat_items(void)
 
        s[SI_STATUS_BAR].prefix = "";
        s[SI_STATUS_BAR].postfix = "";
-       s[SI_STATUS_BAR].content = "";
        s[SI_STATUS_BAR].x = 0;
        s[SI_STATUS_BAR].y = 10;
        s[SI_STATUS_BAR].w = 100;
@@ -148,7 +148,6 @@ static void init_stat_items(void)
 
        s[SI_PLAY_TIME].prefix = "";
        s[SI_PLAY_TIME].postfix = "";
-       s[SI_PLAY_TIME].content = "";
        s[SI_PLAY_TIME].x = 35;
        s[SI_PLAY_TIME].y = 20;
        s[SI_PLAY_TIME].w = 65;
@@ -161,7 +160,6 @@ static void init_stat_items(void)
 
        s[SI_STATUS].prefix = "";
        s[SI_STATUS].postfix = "";
-       s[SI_STATUS].content = "";
        s[SI_STATUS].x = 35;
        s[SI_STATUS].y = 28;
        s[SI_STATUS].w = 12;
@@ -174,7 +172,6 @@ static void init_stat_items(void)
 
        s[SI_STATUS_FLAGS].prefix = " (";
        s[SI_STATUS_FLAGS].postfix = ")";
-       s[SI_STATUS_FLAGS].content = "";
        s[SI_STATUS_FLAGS].x = 47;
        s[SI_STATUS_FLAGS].y = 28;
        s[SI_STATUS_FLAGS].w = 15;
@@ -187,7 +184,6 @@ static void init_stat_items(void)
 
        s[SI_NUM_PLAYED].prefix = "#";
        s[SI_NUM_PLAYED].postfix = "";
-       s[SI_NUM_PLAYED].content = "0";
        s[SI_NUM_PLAYED].x = 62;
        s[SI_NUM_PLAYED].y = 28;
        s[SI_NUM_PLAYED].w = 13;
@@ -200,7 +196,6 @@ static void init_stat_items(void)
 
        s[SI_UPTIME].prefix = "Up: ";
        s[SI_UPTIME].postfix = "";
-       s[SI_UPTIME].content = "";
        s[SI_UPTIME].x = 75;
        s[SI_UPTIME].y = 28;
        s[SI_UPTIME].w = 25;
@@ -211,22 +206,20 @@ static void init_stat_items(void)
        s[SI_UPTIME].font = N_YELLOW;
        s[SI_UPTIME].align = RIGHT;
 
-       s[SI_DBTOOL].prefix = "dbtool: ";
-       s[SI_DBTOOL].postfix = "";
-       s[SI_DBTOOL].content = "no content yet";
-       s[SI_DBTOOL].x = 35;
-       s[SI_DBTOOL].y = 48;
-       s[SI_DBTOOL].w = 35;
-       s[SI_DBTOOL].h = FONT_HEIGHT;
-       s[SI_DBTOOL].r = 0;
-       s[SI_DBTOOL].g = 0;
-       s[SI_DBTOOL].b = 0;
-       s[SI_DBTOOL].font = N_YELLOW;
-       s[SI_DBTOOL].align = LEFT;
+       s[SI_SELECTOR].prefix = "selector: ";
+       s[SI_SELECTOR].postfix = "";
+       s[SI_SELECTOR].x = 35;
+       s[SI_SELECTOR].y = 48;
+       s[SI_SELECTOR].w = 35;
+       s[SI_SELECTOR].h = FONT_HEIGHT;
+       s[SI_SELECTOR].r = 0;
+       s[SI_SELECTOR].g = 0;
+       s[SI_SELECTOR].b = 0;
+       s[SI_SELECTOR].font = N_YELLOW;
+       s[SI_SELECTOR].align = LEFT;
 
        s[SI_FORMAT].prefix = "Format: ";
        s[SI_FORMAT].postfix = "";
-       s[SI_FORMAT].content = "";
        s[SI_FORMAT].x = 70;
        s[SI_FORMAT].y = 48;
        s[SI_FORMAT].w = 30;
@@ -239,7 +232,6 @@ static void init_stat_items(void)
 
        s[SI_MTIME].prefix = "MTime: ";
        s[SI_MTIME].postfix = "";
-       s[SI_MTIME].content = "";
        s[SI_MTIME].x = 35;
        s[SI_MTIME].y = 35;
        s[SI_MTIME].w = 65;
@@ -252,7 +244,6 @@ static void init_stat_items(void)
 
        s[SI_FILE_SIZE].prefix = "Size: ";
        s[SI_FILE_SIZE].postfix = "kb";
-       s[SI_FILE_SIZE].content = "";
        s[SI_FILE_SIZE].x = 35;
        s[SI_FILE_SIZE].y = 42;
        s[SI_FILE_SIZE].w = 20;
@@ -265,7 +256,6 @@ static void init_stat_items(void)
 
        s[SI_AUDIO_INFO1].prefix = "";
        s[SI_AUDIO_INFO1].postfix = "";
-       s[SI_AUDIO_INFO1].content = "";
        s[SI_AUDIO_INFO1].x = 0;
        s[SI_AUDIO_INFO1].y = 60;
        s[SI_AUDIO_INFO1].w = 100;
@@ -278,7 +268,6 @@ static void init_stat_items(void)
 
        s[SI_AUDIO_INFO2].prefix = "";
        s[SI_AUDIO_INFO2].postfix = "";
-       s[SI_AUDIO_INFO2].content = "";
        s[SI_AUDIO_INFO2].x = 0;
        s[SI_AUDIO_INFO2].y = 65;
        s[SI_AUDIO_INFO2].w = 100;
@@ -291,7 +280,6 @@ static void init_stat_items(void)
 
        s[SI_AUDIO_INFO3].prefix = "";
        s[SI_AUDIO_INFO3].postfix = "";
-       s[SI_AUDIO_INFO3].content = "";
        s[SI_AUDIO_INFO3].x = 0;
        s[SI_AUDIO_INFO3].y = 70;
        s[SI_AUDIO_INFO3].w = 100;
@@ -305,7 +293,6 @@ static void init_stat_items(void)
        s[SI_DBINFO1].name = "dbinfo1:";
        s[SI_DBINFO1].prefix = "";
        s[SI_DBINFO1].postfix = "";
-       s[SI_DBINFO1].content = "";
        s[SI_DBINFO1].x = 0;
        s[SI_DBINFO1].y = 83;
        s[SI_DBINFO1].w = 100;
@@ -318,7 +305,6 @@ static void init_stat_items(void)
 
        s[SI_DBINFO2].prefix = "";
        s[SI_DBINFO2].postfix = "";
-       s[SI_DBINFO2].content = "";
        s[SI_DBINFO2].x = 0;
        s[SI_DBINFO2].y = 88;
        s[SI_DBINFO2].w = 100;
@@ -332,7 +318,6 @@ static void init_stat_items(void)
        s[SI_DBINFO3].name = "dbinfo3:";
        s[SI_DBINFO3].prefix = "";
        s[SI_DBINFO3].postfix = "";
-       s[SI_DBINFO3].content = "";
        s[SI_DBINFO3].x = 0;
        s[SI_DBINFO3].y = 93;
        s[SI_DBINFO3].w = 100;
@@ -371,7 +356,7 @@ static void init_SDL(void)
        SDL_EventState(SDL_MOUSEBUTTONUP, SDL_IGNORE);
        /* Set the window manager title bar */
        SDL_WM_SetCaption("The Gui of death that makes you blind (paraslash "
-               VERSION ")", "SFont");
+               PACKAGE_VERSION ")", "SFont");
 }
 
 /*
@@ -450,7 +435,7 @@ static void tab2space(char *text)
        }
 }
 
-static void print_msg(char *msg)
+static void print_msg(const char *msg)
 {
        SFont_FontInfo *font = &(fonts[MSG_FONT].fontinfo);
        char *buf = strdup(msg);
@@ -499,7 +484,7 @@ SDLKey get_key(void)
 /*
  * print message, wait for key (blocking), return 1 for 'q', 0 else
  */
-static SDLKey hit_key(char *msg)
+static SDLKey hit_key(const char *msg)
 {
        SDLKey sym;
 
@@ -520,7 +505,7 @@ static SDLKey hit_key(char *msg)
  */
 static int command_handler(void)
 {
-       FILE *pipe;
+       FILE *f;
        unsigned count = 0;
        char text[MAXLINE]="";
        char buf[MAXLINE]="";
@@ -536,15 +521,15 @@ static int command_handler(void)
        if (text[0] == '!') {
                if (text[1] == '\0')
                        return 1;
-               pipe = popen(text + 1, "r");
+               f = popen(text + 1, "r");
        } else {
                sprintf(buf, BINDIR "/para_client %s 2>&1", text);
-               pipe = popen(buf, "r");
+               f = popen(buf, "r");
        }
-       if (!pipe)
+       if (!f)
                return 0;
        fill_output_rect();
-       while(fgets(text, MAXLINE - 1, pipe)) {
+       while(fgets(text, MAXLINE - 1, f)) {
                int len;
 
                tab2space(text);
@@ -569,7 +554,7 @@ static int command_handler(void)
        update_all();
        hit_key("Hit any key to return");
 out:   fill_output_rect();
-       pclose(pipe);
+       pclose(f);
        return 1;
 }
 
@@ -600,14 +585,14 @@ SDL_Surface *load_jpg(void)
        SDL_RWops *rwop;
        int fds[3] = {0, 1, 0};
        pid_t pid;
-       FILE *pipe;
+       FILE *f;
 
        if (para_exec_cmdline_pid(&pid, args_info.pic_cmd_arg, fds) < 0)
                return NULL;
-       pipe = fdopen(fds[1], "r");
-       if (!pipe)
+       f = fdopen(fds[1], "r");
+       if (!f)
                return NULL;
-       if (!(rwop = SDL_RWFromFP(pipe, 0)))
+       if (!(rwop = SDL_RWFromFP(f, 0)))
                return NULL;
        return IMG_LoadJPG_RW(rwop);
 }
@@ -704,7 +689,7 @@ void update_status(char *buf)
  * Read stat line from pipe if pipe is ready, call update_status to
  * display information.
  */
-static int draw_status(int pipe)
+static int draw_status(int fd)
 {
        fd_set rfds;
        int ret;
@@ -713,14 +698,12 @@ static int draw_status(int pipe)
        tv.tv_sec = 0;
        tv.tv_usec = 3000000;
        FD_ZERO(&rfds);
-       FD_SET(pipe, &rfds);
-       ret = select(pipe + 1, &rfds, NULL, NULL, &tv);
-//     printf("select returned %d\n", ret);
+       FD_SET(fd, &rfds);
+       ret = para_select(fd + 1, &rfds, NULL, &tv);
        if (ret <= 0)
                return 0;
-       if (read_audiod_pipe(pipe, update_status) > 0)
+       if (read_audiod_pipe(fd, update_status) > 0)
                return 1;
-//     clear_all_items();
        free(stat_items[SI_STATUS_BAR].content);
        stat_items[SI_STATUS_BAR].content =
                para_strdup("audiod not running!?\n");
@@ -756,10 +739,10 @@ static int configfile_exists(void)
  */
 int main(int argc, char *argv[])
 {
-       int i, ret, pipe;
+       int i, ret, fd = -1;
        SDLKey sym;
 
-       cmdline_parser(argc, argv, &args_info);
+       sdl_gui_cmdline_parser(argc, argv, &args_info);
        ret = configfile_exists();
 //     printf("w=%i,h=%i,ret=%i, cf=%s\n", width, height, ret, args_info.config_file_arg);
 
@@ -769,14 +752,13 @@ int main(int argc, char *argv[])
                exit(EXIT_FAILURE);
        }
        if (ret)
-               cmdline_parser_configfile(args_info.config_file_arg,
+               sdl_gui_cmdline_parser_configfile(args_info.config_file_arg,
                        &args_info, 0, 0, 0);
        signal(SIGCHLD, SIG_IGN);
        width = args_info.width_arg;
        height = args_info.height_arg;
 //     printf("w=%i,h=%i,ret=%i, cf=%s\n", width, height, ret, args_info.config_file_arg);
        init_stat_items();
-       pipe = para_open_audiod_pipe(args_info.stat_cmd_arg);
        init_SDL();
        for (i = 0; fonts[i].name[0]; i++) {
                char buf[MAXLINE];
@@ -792,10 +774,15 @@ int main(int argc, char *argv[])
                update_input();
        }
        for (;;) {
-               ret = draw_status(pipe);
+               if (fd < 0) {
+                       fd = para_open_audiod_pipe(args_info.stat_cmd_arg);
+                       if (fd < 0)
+                               clean_exit(EXIT_FAILURE);
+               }
+               ret = draw_status(fd);
                if (ret < 0) {
-                       close(pipe);
-                       pipe = -1;
+                       close(fd);
+                       fd = -1;
                }
                if (SDL_QuitRequested())
                        clean_exit(0);
@@ -819,12 +806,6 @@ int main(int argc, char *argv[])
                                || sym == SDLK_COMPOSE
                                )
                                continue;
-                       if (pipe < 0) {
-//                             printf("closing pipe\n");
-                               kill(0, SIGINT);
-                               close(pipe);
-//                             printf("pipe closed\n");
-                       }
                        fill_input_rect();
                        update_input();
                        if (!command_handler())
@@ -833,7 +814,6 @@ int main(int argc, char *argv[])
                        print_help();
                        update_pic();
                        SDL_UpdateRect(screen, 0, 0, 0, 0);
-                       pipe = para_open_audiod_pipe(args_info.stat_cmd_arg);
                        break;
                }
        }