Code

m4/ax_check*: update to serial 3
[ncmpc.git] / src / main.c
index c597e96c766a122d3a6cda681c802f8de84e5ab9..4c0f881e9d8311b564fa1b2e3782a5f6bb085b6e 100644 (file)
@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License along
  * with this program; if not, write to the Free Software Foundation, Inc.,
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-*/
+ */
 
 #include "config.h"
 #include "ncmpc.h"
@@ -27,7 +27,7 @@
 #include "ncu.h"
 #include "screen.h"
 #include "screen_utils.h"
-#include "screen_message.h"
+#include "screen_status.h"
 #include "strfsong.h"
 #include "i18n.h"
 #include "player_command.h"
@@ -75,7 +75,6 @@ static void
 update_xterm_title(void)
 {
        static char title[BUFSIZE];
-       char tmp[BUFSIZE];
        struct mpd_status *status = NULL;
        const struct mpd_song *song = NULL;
 
@@ -84,6 +83,7 @@ update_xterm_title(void)
                song = mpd->song;
        }
 
+       char tmp[BUFSIZE];
        if (options.xterm_title_format && status && song &&
            mpd_status_get_state(status) == MPD_STATE_PLAY)
                strfsong(tmp, BUFSIZE, options.xterm_title_format, song);
@@ -112,15 +112,16 @@ exit_and_cleanup(void)
        }
 }
 
+#ifndef WIN32
 static void
-catch_sigint(G_GNUC_UNUSED int sig)
+catch_sigint(gcc_unused int sig)
 {
        g_main_loop_quit(main_loop);
 }
 
 
 static void
-catch_sigcont(G_GNUC_UNUSED int sig)
+catch_sigcont(gcc_unused int sig)
 {
        char irrelevant = 'a';
        if (1 != write(sigwinch_pipes[1], &irrelevant, 1))
@@ -136,8 +137,8 @@ sigstop(void)
 }
 
 static gboolean
-sigwinch_event(G_GNUC_UNUSED GIOChannel *source,
-               G_GNUC_UNUSED GIOCondition condition, G_GNUC_UNUSED gpointer data)
+sigwinch_event(gcc_unused GIOChannel *source,
+               gcc_unused GIOCondition condition, gcc_unused gpointer data)
 {
        char ignoreme[64];
        if (1 > read(sigwinch_pipes[0], ignoreme, 64))
@@ -151,18 +152,19 @@ sigwinch_event(G_GNUC_UNUSED GIOChannel *source,
 }
 
 static void
-catch_sigwinch(G_GNUC_UNUSED int sig)
+catch_sigwinch(gcc_unused int sig)
 {
        char irrelevant = 'a';
        if (1 != write(sigwinch_pipes[1], &irrelevant, 1))
                exit(EXIT_FAILURE);
 }
+#endif /* WIN32 */
 
 static void
 idle_callback(enum mpd_error error,
-             G_GNUC_UNUSED enum mpd_server_error server_error,
+             gcc_unused enum mpd_server_error server_error,
              const char *message, enum mpd_idle events,
-             G_GNUC_UNUSED void *ctx);
+             gcc_unused void *ctx);
 
 static gboolean
 timer_mpd_update(gpointer data);
@@ -269,23 +271,6 @@ default_settings_name(void)
 
        return name;
 #else
-       return g_strdup(options.host);
-#endif
-}
-
-static char *
-connection_settings_name(const struct mpd_connection *connection)
-{
-#if LIBMPDCLIENT_CHECK_VERSION(2,4,0)
-       const struct mpd_settings *settings =
-               mpd_connection_get_settings(connection);
-       if (settings == NULL)
-               return g_strdup(_("unknown"));
-
-       return settings_name(settings);
-#else
-       (void)connection;
-
        /*
         * localhost is actually not correct, we only know that
         * mpd_connection_new() has connected to the "default host".
@@ -300,43 +285,38 @@ connection_settings_name(const struct mpd_connection *connection)
  * broken.  It tries to recover by reconnecting periodically.
  */
 static gboolean
-timer_reconnect(G_GNUC_UNUSED gpointer data)
+timer_reconnect(gcc_unused gpointer data)
 {
-       bool success;
-       struct mpd_connection *connection;
-
        assert(!mpdclient_is_connected(mpd));
 
        reconnect_source_id = 0;
 
        char *name = default_settings_name();
        screen_status_printf(_("Connecting to %s...  [Press %s to abort]"),
-                            name, get_key_names(CMD_QUIT,0) );
+                            name, get_key_names(CMD_QUIT, false));
        g_free(name);
        doupdate();
 
        mpdclient_disconnect(mpd);
-       success = mpdclient_connect(mpd,
-                                   options.host, options.port,
-                                   5000,
-                                   options.password);
-       if (!success) {
+       if (!mpdclient_connect(mpd, options.host, options.port,
+                              options.timeout_ms,
+                              options.password)) {
                /* try again in 5 seconds */
                reconnect_source_id = g_timeout_add(5000,
                                                    timer_reconnect, NULL);
                return FALSE;
        }
 
-       connection = mpdclient_get_connection(mpd);
+       struct mpd_connection *connection = mpdclient_get_connection(mpd);
 
 #ifndef NCMPC_MINI
-       /* quit if mpd is pre 0.11.0 - song id not supported by mpd */
-       if (mpd_connection_cmp_server_version(connection, 0, 12, 0) < 0) {
+       /* quit if mpd is pre 0.14 - song id not supported by mpd */
+       if (mpd_connection_cmp_server_version(connection, 0, 16, 0) < 0) {
                const unsigned *version =
                        mpd_connection_get_server_version(connection);
-               screen_status_printf(_("Error: MPD version %d.%d.%d is to old (%s needed)"),
+               screen_status_printf(_("Error: MPD version %d.%d.%d is too old (%s needed)"),
                                     version[0], version[1], version[2],
-                                    "0.12.0");
+                                    "0.16.0");
                mpdclient_disconnect(mpd);
                doupdate();
 
@@ -347,20 +327,14 @@ timer_reconnect(G_GNUC_UNUSED gpointer data)
        }
 #endif
 
-       if (mpd_connection_cmp_server_version(connection,
-                                             0, 14, 0) >= 0)
-               mpd->source = mpd_glib_new(connection,
-                                          idle_callback, mpd);
+       mpd->source = mpd_glib_new(connection,
+                                  idle_callback, mpd);
 
-       name = connection_settings_name(connection);
-       screen_status_printf(_("Connected to %s"), name);
-       g_free(name);
+       screen_status_clear_message();
        doupdate();
 
        /* update immediately */
-       mpd->events = MPD_IDLE_DATABASE|MPD_IDLE_STORED_PLAYLIST|
-               MPD_IDLE_QUEUE|MPD_IDLE_PLAYER|MPD_IDLE_MIXER|MPD_IDLE_OUTPUT|
-               MPD_IDLE_OPTIONS|MPD_IDLE_UPDATE;
+       mpd->events = MPD_IDLE_ALL;
 
        do_mpd_update();
 
@@ -391,7 +365,7 @@ idle_callback(enum mpd_error error, enum mpd_server_error server_error,
 
        c->idle = false;
 
-       assert(mpdclient_get_connection(c) != NULL);
+       assert(mpdclient_is_connected(c));
 
        if (error != MPD_ERROR_SUCCESS) {
                char *allocated;
@@ -439,7 +413,7 @@ idle_callback(enum mpd_error error, enum mpd_server_error server_error,
 }
 
 static gboolean
-timer_mpd_update(G_GNUC_UNUSED gpointer data)
+timer_mpd_update(gcc_unused gpointer data)
 {
        do_mpd_update();
 
@@ -482,15 +456,14 @@ int do_input_event(command_t cmd)
 }
 
 static gboolean
-keyboard_event(G_GNUC_UNUSED GIOChannel *source,
-              G_GNUC_UNUSED GIOCondition condition,
-              G_GNUC_UNUSED gpointer data)
+keyboard_event(gcc_unused GIOChannel *source,
+              gcc_unused GIOCondition condition,
+              gcc_unused gpointer data)
 {
-       command_t cmd;
-
        begin_input_event();
 
-       if ((cmd=get_keyboard_command()) != CMD_NONE)
+       command_t cmd = get_keyboard_command();
+       if (cmd != CMD_NONE)
                if (do_input_event(cmd) != 0)
                        return FALSE;
 
@@ -504,7 +477,7 @@ keyboard_event(G_GNUC_UNUSED GIOChannel *source,
  * message every 10 seconds.
  */
 static gboolean
-timer_check_key_bindings(G_GNUC_UNUSED gpointer data)
+timer_check_key_bindings(gcc_unused gpointer data)
 {
        char buf[256];
 #ifdef ENABLE_KEYDEF_SCREEN
@@ -527,7 +500,7 @@ timer_check_key_bindings(G_GNUC_UNUSED gpointer data)
           and this is a hint for the user what to press to correct
           that */
        g_snprintf(comment, sizeof(comment), _("press %s for the key editor"),
-                  get_key_names(CMD_SCREEN_KEYDEF, 0));
+                  get_key_names(CMD_SCREEN_KEYDEF, false));
        g_strlcat(buf, comment, sizeof(buf));
        g_strlcat(buf, ")", sizeof(buf));
 #endif
@@ -542,18 +515,11 @@ timer_check_key_bindings(G_GNUC_UNUSED gpointer data)
 int
 main(int argc, const char *argv[])
 {
-       struct sigaction act;
 #ifdef ENABLE_LOCALE
-       const char *charset = NULL;
-#endif
-       GIOChannel *keyboard_channel;
-#ifdef ENABLE_LIRC
-       int lirc_socket;
-       GIOChannel *lirc_channel = NULL;
+#ifndef ENABLE_NLS
+       gcc_unused
 #endif
-       GIOChannel *sigwinch_channel = NULL;
-
-#ifdef ENABLE_LOCALE
+       const char *charset = NULL;
        /* time and date formatting */
        setlocale(LC_TIME,"");
        /* care about sorting order etc */
@@ -592,7 +558,9 @@ main(int argc, const char *argv[])
        /* parse command line options - 2 pass */
        options_parse(argc, argv);
 
+#ifndef WIN32
        /* setup signal behavior - SIGINT */
+       struct sigaction act;
        sigemptyset(&act.sa_mask);
        act.sa_flags = 0;
        act.sa_handler = catch_sigint;
@@ -641,6 +609,7 @@ main(int argc, const char *argv[])
                perror("sigaction(SIGPIPE)");
                exit(EXIT_FAILURE);
        }
+#endif
 
        ncu_init();
 
@@ -658,18 +627,21 @@ main(int argc, const char *argv[])
        main_loop = g_main_loop_new(NULL, FALSE);
 
        /* watch out for keyboard input */
-       keyboard_channel = g_io_channel_unix_new(STDIN_FILENO);
+       GIOChannel *keyboard_channel = g_io_channel_unix_new(STDIN_FILENO);
        g_io_add_watch(keyboard_channel, G_IO_IN, keyboard_event, NULL);
 
 #ifdef ENABLE_LIRC
        /* watch out for lirc input */
-       lirc_socket = ncmpc_lirc_open();
+       int lirc_socket = ncmpc_lirc_open();
+       GIOChannel *lirc_channel = NULL;
        if (lirc_socket >= 0) {
                lirc_channel = g_io_channel_unix_new(lirc_socket);
                g_io_add_watch(lirc_channel, G_IO_IN, lirc_event, NULL);
        }
 #endif
 
+#ifndef WIN32
+       GIOChannel *sigwinch_channel = NULL;
        if (!pipe(sigwinch_pipes) &&
                !fcntl(sigwinch_pipes[1], F_SETFL, O_NONBLOCK)) {
                sigwinch_channel = g_io_channel_unix_new(sigwinch_pipes[0]);
@@ -679,6 +651,7 @@ main(int argc, const char *argv[])
                perror("sigwinch pipe creation failed");
                exit(EXIT_FAILURE);
        }
+#endif
 
        /* attempt to connect */
        reconnect_source_id = g_timeout_add(1, timer_reconnect, NULL);