encoding: Always use "utf-8" as default for &encoding

Preverve the locale-depedency for fileencodings=default
This commit is contained in:
Björn Linse 2015-11-10 22:44:41 +01:00
parent cb62594042
commit 49b06a8f2a
3 changed files with 31 additions and 63 deletions

View File

@ -798,6 +798,8 @@ EXTERN bool enc_utf8 INIT(= false); /* UTF-8 encoded Unicode */
EXTERN int enc_latin1like INIT(= TRUE); /* 'encoding' is latin1 comp. */ EXTERN int enc_latin1like INIT(= TRUE); /* 'encoding' is latin1 comp. */
EXTERN int has_mbyte INIT(= 0); /* any multi-byte encoding */ EXTERN int has_mbyte INIT(= 0); /* any multi-byte encoding */
/// Encoding used when 'fencs' is set to "default"
EXTERN char_u *fenc_default INIT(= NULL);
/* /*
* To speed up BYTELEN() we fill a table with the byte lengths whenever * To speed up BYTELEN() we fill a table with the byte lengths whenever

View File

@ -2417,11 +2417,8 @@ char_u *enc_canonize(char_u *enc) FUNC_ATTR_NONNULL_RET
int i; int i;
if (STRCMP(enc, "default") == 0) { if (STRCMP(enc, "default") == 0) {
/* Use the default encoding as it's found by set_init_1(). */ // Use the default encoding as found by set_init_1().
char_u *r = get_encoding_default(); return vim_strsave(fenc_default);
if (r == NULL)
r = (char_u *)"latin1";
return vim_strsave(r);
} }
/* copy "enc" to allocated memory, with room for two '-' */ /* copy "enc" to allocated memory, with room for two '-' */

View File

@ -776,59 +776,38 @@ void set_init_1(void)
/* Parse default for 'listchars'. */ /* Parse default for 'listchars'. */
(void)set_chars_option(&p_lcs); (void)set_chars_option(&p_lcs);
/* enc_locale() will try to find the encoding of the current locale. */ // enc_locale() will try to find the encoding of the current locale.
// This will be used when 'default' is used as encoding specifier
// in 'fileencodings'
char_u *p = enc_locale(); char_u *p = enc_locale();
if (p != NULL) { if (p == NULL) {
char_u *save_enc; // use utf-8 as 'default' if locale encoding can't be detected.
p = vim_strsave((char_u *)"utf-8");
}
fenc_default = p;
/* Try setting 'encoding' and check if the value is valid. // Initialize multibyte (utf-8) handling
* If not, go back to the default "utf-8". */ mb_init();
save_enc = p_enc;
p_enc = (char_u *) p;
if (STRCMP(p_enc, "gb18030") == 0) {
/* We don't support "gb18030", but "cp936" is a good substitute
* for practical purposes, thus use that. It's not an alias to
* still support conversion between gb18030 and utf-8. */
p_enc = vim_strsave((char_u *)"cp936");
xfree(p);
}
if (mb_init() == NULL) {
opt_idx = findoption((char_u *)"encoding");
if (opt_idx >= 0) {
options[opt_idx].def_val[VI_DEFAULT] = p_enc;
options[opt_idx].flags |= P_DEF_ALLOCED;
}
#if defined(MSWIN) || defined(MACOS) #if defined(MSWIN) || defined(MACOS)
if (STRCMP(p_enc, "latin1") == 0 if (STRCMP(p_enc, "latin1") == 0
|| enc_utf8 || enc_utf8
) { ) {
/* Adjust the default for 'isprint' and 'iskeyword' to match /* Adjust the default for 'isprint' and 'iskeyword' to match
* latin1. */ * latin1. */
set_string_option_direct((char_u *)"isp", -1, set_string_option_direct((char_u *)"isp", -1,
ISP_LATIN1, OPT_FREE, SID_NONE); ISP_LATIN1, OPT_FREE, SID_NONE);
set_string_option_direct((char_u *)"isk", -1, set_string_option_direct((char_u *)"isk", -1,
ISK_LATIN1, OPT_FREE, SID_NONE); ISK_LATIN1, OPT_FREE, SID_NONE);
opt_idx = findoption((char_u *)"isp"); opt_idx = findoption((char_u *)"isp");
if (opt_idx >= 0) if (opt_idx >= 0)
options[opt_idx].def_val[VIM_DEFAULT] = ISP_LATIN1; options[opt_idx].def_val[VIM_DEFAULT] = ISP_LATIN1;
opt_idx = findoption((char_u *)"isk"); opt_idx = findoption((char_u *)"isk");
if (opt_idx >= 0) if (opt_idx >= 0)
options[opt_idx].def_val[VIM_DEFAULT] = ISK_LATIN1; options[opt_idx].def_val[VIM_DEFAULT] = ISK_LATIN1;
(void)init_chartab(); (void)init_chartab();
}
#endif
} else {
xfree(p_enc);
// mb_init() failed; fallback to utf8 and try again.
p_enc = save_enc;
mb_init();
}
} else {
// enc_locale() failed; initialize the default (utf8).
mb_init();
} }
#endif
// Don't change &encoding when resetting to defaults with ":set all&". // Don't change &encoding when resetting to defaults with ":set all&".
opt_idx = findoption((char_u *)"encoding"); opt_idx = findoption((char_u *)"encoding");
@ -4669,16 +4648,6 @@ char_u *get_highlight_default(void)
return (char_u *)NULL; return (char_u *)NULL;
} }
char_u *get_encoding_default(void)
{
int i;
i = findoption((char_u *)"enc");
if (i >= 0)
return options[i].def_val[VI_DEFAULT];
return (char_u *)NULL;
}
/* /*
* Translate a string like "t_xx", "<t_xx>" or "<S-Tab>" to a key number. * Translate a string like "t_xx", "<t_xx>" or "<S-Tab>" to a key number.
*/ */