Merge pull request #2166 from ntrel/user-ft-ext

Always allow user filetype extensions to override system config file
This commit is contained in:
Nick Treleaven 2019-06-10 12:15:43 +01:00 committed by GitHub
commit 770cda4064
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 27 additions and 25 deletions

View File

@ -463,26 +463,8 @@ void filetypes_init(void)
}
/* Find a filetype that predicate returns TRUE for, otherwise return NULL. */
GeanyFiletype *filetypes_find(GCompareFunc predicate, gpointer user_data)
static gboolean match_basename(const GeanyFiletype *ft, const gchar *base_filename)
{
guint i;
for (i = 0; i < filetypes_array->len; i++)
{
GeanyFiletype *ft = filetypes[i];
if (predicate(ft, user_data))
return ft;
}
return NULL;
}
static gboolean match_basename(gconstpointer pft, gconstpointer user_data)
{
const GeanyFiletype *ft = pft;
const gchar *base_filename = user_data;
gint j;
gboolean ret = FALSE;
@ -505,7 +487,7 @@ static gboolean match_basename(gconstpointer pft, gconstpointer user_data)
}
static GeanyFiletype *check_builtin_filenames(const gchar *utf8_filename)
static GeanyFiletype *detect_filetype_conf_file(const gchar *utf8_filename)
{
gchar *lfn = NULL;
gchar *path;
@ -539,8 +521,9 @@ GeanyFiletype *filetypes_detect_from_extension(const gchar *utf8_filename)
{
gchar *base_filename;
GeanyFiletype *ft;
guint i;
ft = check_builtin_filenames(utf8_filename);
ft = detect_filetype_conf_file(utf8_filename);
if (ft)
return ft;
@ -551,7 +534,27 @@ GeanyFiletype *filetypes_detect_from_extension(const gchar *utf8_filename)
SETPTR(base_filename, g_utf8_strdown(base_filename, -1));
#endif
ft = filetypes_find(match_basename, base_filename);
for (i = 0; i < filetypes_array->len; i++)
{
if (match_basename(filetypes[i], base_filename))
{
ft = filetypes[i];
break;
}
}
// check if user config overrides found ft
if (ft && !ft->priv->user_extensions)
{
for (i++; i < filetypes_array->len; i++)
{
if (filetypes[i]->priv->user_extensions &&
match_basename(filetypes[i], base_filename))
{
ft = filetypes[i];
break;
}
}
}
if (ft == NULL)
ft = filetypes[GEANY_FILETYPES_NONE];
@ -1397,6 +1400,7 @@ static void read_extensions(GKeyFile *sysconfig, GKeyFile *userconfig)
gchar **list = g_key_file_get_string_list(
(userset) ? userconfig : sysconfig, "Extensions", filetypes[i]->name, &len, NULL);
filetypes[i]->priv->user_extensions = userset;
g_strfreev(filetypes[i]->pattern);
/* Note: we allow 'Foo=' to remove all patterns */
if (!list)

View File

@ -196,9 +196,6 @@ extern GPtrArray *filetypes_array;
extern GSList *filetypes_by_title;
GeanyFiletype *filetypes_find(GCompareFunc predicate, gpointer user_data);
void filetypes_init(void);
void filetypes_init_types(void);

View File

@ -40,6 +40,7 @@ typedef struct GeanyFiletypePrivate
gboolean xml_indent_tags; /* XML tag autoindentation, for HTML and XML filetypes */
GSList *tag_files;
gboolean warn_color_scheme;
gboolean user_extensions; // true if extensions were read from user config file
/* TODO: move to structure in build.h and only put a pointer here */
GeanyBuildCommand *filecmds;