more miscellaneous cleanups

git-svn-id: svn://svn.savannah.gnu.org/nano/trunk/nano@2662 35c25a1d-7b9e-4130-9fde-d3aeb78583b8

diff --git a/src/chars.c b/src/chars.c
index 2c8b12b24e888541155e7760a14fce19eff9c5d3..01c89310fd12724fec71b928377f832f3db405b5 100644 (file)
--- a/src/chars.c
+++ b/src/chars.c
@@ -247,8 +247,10 @@ int mb_cur_max(void)
 }
 
 /* Convert the value in chr to a multibyte character with the same
- * wide character value as chr, if possible.  Return the (dynamically
- * allocated) multibyte character and its length. */
+ * wide character value as chr, if possible.  If the conversion
+ * succeeds, return the (dynamically allocated) multibyte character and
+ * its length.  Otherwise, return an undefined (dynamically allocated)
+ * multibyte character and a length of zero. */
 char *make_mbchar(int chr, int *chr_mb_len)
 {
     char *chr_mb;
@@ -264,8 +266,6 @@ char *make_mbchar(int chr, int *chr_mb_len)
            wctomb(NULL, 0);
            *chr_mb_len = 0;
        }
-
-       align(&chr_mb);
     } else {
 #endif
        *chr_mb_len = 1;
@@ -277,74 +277,6 @@ char *make_mbchar(int chr, int *chr_mb_len)
     return chr_mb;
 }
 
-#ifdef ENABLE_NANORC
-/* Check if the string str is a valid multibyte string.  Return TRUE if
- * it is, and FALSE otherwise. */
-bool is_valid_mbstring(const char *str)
-{
-    assert(str != NULL);
-
-    return 
-#ifdef NANO_WIDE
-       (!ISSET(NO_UTF8)) ?
-       (mbstowcs(NULL, str, (size_t)-1) != (size_t)-1) :
-#endif
-
-       TRUE;
-}
-#endif /* ENABLE_NANORC */
-
-#ifdef NANO_EXTRA
-/* Convert the string str to a valid multibyte string with the same wide
- * character values as str.  Return the (dynamically allocated)
- * multibyte string. */
-char *make_valid_mbstring(const char *str)
-{
-    assert(str != NULL);
-
-#ifdef NANO_WIDE
-    if (!ISSET(NO_UTF8)) {
-       char *chr_mb = charalloc(MB_CUR_MAX);
-       char *str_mb = charalloc((MB_CUR_MAX * strlen(str)) + 1);
-       size_t str_mb_len = 0;
-
-       while (*str != '\0') {
-           int chr_mb_len, i;
-           bool bad_chr;
-
-           chr_mb_len = parse_mbchar(str, chr_mb, &bad_chr, NULL);
-
-           if (bad_chr) {
-               char *bad_chr_mb;
-               int bad_chr_mb_len;
-
-               bad_chr_mb = make_mbchar((unsigned char)*chr_mb,
-                   &bad_chr_mb_len);
-
-               for (i = 0; i < bad_chr_mb_len; i++)
-                   str_mb[str_mb_len + i] = bad_chr_mb[i];
-               str_mb_len += bad_chr_mb_len;
-
-               free(bad_chr_mb);
-           } else {
-               for (i = 0; i < chr_mb_len; i++)
-                   str_mb[str_mb_len + i] = chr_mb[i];
-               str_mb_len += chr_mb_len;
-           }
-
-           str += chr_mb_len;
-       }
-
-       free(chr_mb);
-       null_at(&str_mb, str_mb_len);
-
-       return str_mb;
-     } else
-#endif
-       return mallocstrcpy(NULL, str);
-}
-#endif /* NANO_EXTRA */
-
 /* Parse a multibyte character from buf.  Return the number of bytes
  * used.  If chr isn't NULL, store the multibyte character in it.  If
  * bad_chr isn't NULL, set it to TRUE if we have a bad multibyte
@@ -885,3 +817,71 @@ char *mbstrchr(const char *s, char *c)
        return strchr(s, *c);
 }
 #endif /* !DISABLE_JUSTIFY */
+
+#ifdef ENABLE_NANORC
+/* Check if the string s is a valid multibyte string.  Return TRUE if it
+ * is, and FALSE otherwise. */
+bool is_valid_mbstring(const char *s)
+{
+    assert(s != NULL);
+
+    return 
+#ifdef NANO_WIDE
+       (!ISSET(NO_UTF8)) ?
+       (mbstowcs(NULL, s, (size_t)-1) != (size_t)-1) :
+#endif
+
+       TRUE;
+}
+#endif /* ENABLE_NANORC */
+
+#ifdef NANO_EXTRA
+/* Convert the string s to a valid multibyte string with the same wide
+ * character values as s.  Return the (dynamically allocated)
+ * multibyte string. */
+char *make_valid_mbstring(const char *s)
+{
+    assert(s != NULL);
+
+#ifdef NANO_WIDE
+    if (!ISSET(NO_UTF8)) {
+       char *chr_mb = charalloc(MB_CUR_MAX);
+       char *s_mb = charalloc((MB_CUR_MAX * strlen(s)) + 1);
+       size_t s_mb_len = 0;
+
+       while (*s != '\0') {
+           int chr_mb_len, i;
+           bool bad_chr;
+
+           chr_mb_len = parse_mbchar(s, chr_mb, &bad_chr, NULL);
+
+           if (bad_chr) {
+               char *bad_chr_mb;
+               int bad_chr_mb_len;
+
+               bad_chr_mb = make_mbchar((unsigned char)*chr_mb,
+                   &bad_chr_mb_len);
+
+               for (i = 0; i < bad_chr_mb_len; i++)
+                   s_mb[s_mb_len + i] = bad_chr_mb[i];
+               s_mb_len += bad_chr_mb_len;
+
+               free(bad_chr_mb);
+           } else {
+               for (i = 0; i < chr_mb_len; i++)
+                   s_mb[s_mb_len + i] = chr_mb[i];
+               s_mb_len += chr_mb_len;
+           }
+
+           s += chr_mb_len;
+       }
+
+       free(chr_mb);
+       null_at(&s_mb, s_mb_len);
+
+       return s_mb;
+     } else
+#endif
+       return mallocstrcpy(NULL, s);
+}
+#endif /* NANO_EXTRA */

diff --git a/src/files.c b/src/files.c
index a3f94821009db3a06cb744eba65d701c5b6265de..f343cf048987c4a9246a5bcdc374eb79121501aa 100644 (file)
--- a/src/files.c
+++ b/src/files.c
@@ -1697,7 +1697,8 @@ int write_file(const char *name, FILE *f_open, bool tmp, int append,
        /* Update originalfilestat to reference the file as it is now. */
        stat(filename, &originalfilestat);
 #endif
-       statusbar(P_("Wrote %lu line", "Wrote %lu lines", lineswritten),
+       statusbar(P_("Wrote %lu line", "Wrote %lu lines",
+               (unsigned long)lineswritten),
                (unsigned long)lineswritten);
        UNSET(MODIFIED);
        titlebar(NULL);

diff --git a/src/proto.h b/src/proto.h
index fac5ab275fee9192c9893bbf46e8e1dcaca0239e..bd4592b0d348e016aac8929fee8588c8ea83a18e 100644 (file)
--- a/src/proto.h
+++ b/src/proto.h
@@ -181,12 +181,6 @@ char *control_mbrep(const char *c, char *crep, int *crep_len);
 int mbwidth(const char *c);
 int mb_cur_max(void);
 char *make_mbchar(int chr, int *chr_mb_len);
-#ifdef ENABLE_NANORC
-bool is_valid_mbstring(const char *str);
-#endif
-#ifdef NANO_EXTRA
-char *make_valid_mbstring(const char *str);
-#endif
 int parse_mbchar(const char *buf, char *chr, bool *bad_chr, size_t
        *col);
 size_t move_mbleft(const char *buf, size_t pos);
@@ -225,6 +219,12 @@ bool has_blank_mbchars(const char *s);
 #endif
 char *mbstrchr(const char *s, char *c);
 #endif
+#ifdef ENABLE_NANORC
+bool is_valid_mbstring(const char *s);
+#endif
+#ifdef NANO_EXTRA
+char *make_valid_mbstring(const char *s);
+#endif
 
 /* Public functions in color.c. */
 #ifdef ENABLE_COLOR

diff --git a/src/search.c b/src/search.c
index 56edfa8786ab91c194e792db6d13525dd5ca76a7..07aa492f583b4bb99c68d5c465ecda94cf84c890 100644 (file)
--- a/src/search.c
+++ b/src/search.c
@@ -946,8 +946,9 @@ void do_replace(void)
     edit_refresh();
 
     if (numreplaced >= 0)
-       statusbar(P_("Replaced %ld occurrence", "Replaced %ld occurrences",
-               (long)numreplaced), (long)numreplaced);
+       statusbar(P_("Replaced %lu occurrence",
+               "Replaced %lu occurrences", (unsigned long)numreplaced),
+               (unsigned long)numreplaced);
 
     replace_abort();
 }