patch 8.1.1983: compiler nags for uninitialized variable and unused function

Problem:    Compiler nags for uninitialized variable and unused function.
Solution:   Add unnecessary initialization.  Move function inside #ifdef.

diff --git a/src/channel.c b/src/channel.c
index a023c9fd7706cdb3bb0e7cc1938aa5bb1c5fce7c..b4493cf752b29e9630008a57bf754eb66b36f8f5 100644 (file)
--- a/src/channel.c
+++ b/src/channel.c
@@ -55,9 +55,6 @@
 #endif
 
 static void channel_read(channel_T *channel, ch_part_T part, char *func);
-# if defined(MSWIN) || defined(FEAT_GUI_X11) || defined(FEAT_GUI_GTK)
-static channel_T *channel_fd2channel(sock_T fd, ch_part_T *partp);
-# endif
 static ch_mode_T channel_get_mode(channel_T *channel, ch_part_T part);
 static int channel_get_timeout(channel_T *channel, ch_part_T part);
 static ch_part_T channel_part_send(channel_T *channel);
@@ -505,7 +502,31 @@ free_unused_channels(int copyID, int mask)
 
 #if defined(FEAT_GUI) || defined(PROTO)
 
-#if defined(FEAT_GUI_X11) || defined(FEAT_GUI_GTK)
+# if defined(FEAT_GUI_X11) || defined(FEAT_GUI_GTK)
+/*
+ * Lookup the channel from the socket.  Set "partp" to the fd index.
+ * Returns NULL when the socket isn't found.
+ */
+    static channel_T *
+channel_fd2channel(sock_T fd, ch_part_T *partp)
+{
+    channel_T  *channel;
+    ch_part_T  part;
+
+    if (fd != INVALID_FD)
+       for (channel = first_channel; channel != NULL;
+                                                  channel = channel->ch_next)
+       {
+           for (part = PART_SOCK; part < PART_IN; ++part)
+               if (channel->ch_part[part].ch_fd == fd)
+               {
+                   *partp = part;
+                   return channel;
+               }
+       }
+    return NULL;
+}
+
     static void
 channel_read_fd(int fd)
 {
@@ -518,12 +539,12 @@ channel_read_fd(int fd)
     else
        channel_read(channel, part, "channel_read_fd");
 }
-#endif
+# endif
 
 /*
  * Read a command from netbeans.
  */
-#ifdef FEAT_GUI_X11
+# ifdef FEAT_GUI_X11
     static void
 messageFromServerX11(XtPointer clientData,
                  int *unused1 UNUSED,
@@ -531,10 +552,10 @@ messageFromServerX11(XtPointer clientData,
 {
     channel_read_fd((int)(long)clientData);
 }
-#endif
+# endif
 
-#ifdef FEAT_GUI_GTK
-# if GTK_CHECK_VERSION(3,0,0)
+# ifdef FEAT_GUI_GTK
+#  if GTK_CHECK_VERSION(3,0,0)
     static gboolean
 messageFromServerGtk3(GIOChannel *unused1 UNUSED,
                  GIOCondition unused2 UNUSED,
@@ -544,7 +565,7 @@ messageFromServerGtk3(GIOChannel *unused1 UNUSED,
     return TRUE; /* Return FALSE instead in case the event source is to
                  * be removed after this function returns. */
 }
-# else
+#  else
     static void
 messageFromServerGtk2(gpointer clientData,
                  gint unused1 UNUSED,
@@ -552,8 +573,8 @@ messageFromServerGtk2(gpointer clientData,
 {
     channel_read_fd((int)(long)clientData);
 }
+#  endif
 # endif
-#endif
 
     static void
 channel_gui_register_one(channel_T *channel, ch_part_T part)
@@ -674,7 +695,7 @@ channel_gui_unregister(channel_T *channel)
        channel_gui_unregister_one(channel, part);
 }
 
-#endif
+#endif  // FEAT_GUI
 
 static char *e_cannot_connect = N_("E902: Cannot connect to port");
 
@@ -3764,33 +3785,6 @@ theend:
     free_job_options(&opt);
 }
 
-# if defined(MSWIN) || defined(FEAT_GUI_X11) || defined(FEAT_GUI_GTK) \
-       || defined(PROTO)
-/*
- * Lookup the channel from the socket.  Set "partp" to the fd index.
- * Returns NULL when the socket isn't found.
- */
-    static channel_T *
-channel_fd2channel(sock_T fd, ch_part_T *partp)
-{
-    channel_T  *channel;
-    ch_part_T  part;
-
-    if (fd != INVALID_FD)
-       for (channel = first_channel; channel != NULL;
-                                                  channel = channel->ch_next)
-       {
-           for (part = PART_SOCK; part < PART_IN; ++part)
-               if (channel->ch_part[part].ch_fd == fd)
-               {
-                   *partp = part;
-                   return channel;
-               }
-       }
-    return NULL;
-}
-# endif
-
 # if defined(MSWIN) || defined(FEAT_GUI) || defined(PROTO)
 /*
  * Check the channels for anything that is ready to be read.

diff --git a/src/memline.c b/src/memline.c
index b6f0300516ccfd1f883bde70f7ff82485e6d2362..08a99f7f428cf66410a7c512621f1c95e5be2022 100644 (file)
--- a/src/memline.c
+++ b/src/memline.c
@@ -2716,7 +2716,7 @@ add_text_props_for_append(
     int                count;
     int                n;
     char_u     *props;
-    int                new_len;
+    int                new_len = 0;  // init for gcc
     char_u     *new_line;
     textprop_T prop;
 

diff --git a/src/version.c b/src/version.c
index 515564d642ea4b053ac7a348d7b5a15b1a444b20..bf67e38c75b68738300987de591988a4ecdcc135 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -757,6 +757,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1983,
 /**/
     1982,
 /**/