Branch: refs/heads/master Author: Jiří Techet techet@gmail.com Committer: Colomban Wendling ban@herbesfolles.org Date: Mon, 13 Mar 2023 13:40:44 UTC Commit: 6018c523948e50f645af10f5338a56cda1eccf0b https://github.com/geany/geany/commit/6018c523948e50f645af10f5338a56cda1eccf...
Log Message: ----------- Improve code commenting-out global config files
Right now, when a config file from Tools->Configuration Files is edited for the first time, the global config file is taken and all lines in it are commented-out. When modified, such a file is copied into the user's config directory. The logic behind this is that the user typically wants to modify just one or few config options while leaving the rest in the default state.
However, commenting-out all lines is problemmatic. Consider for instance the beginning of filetypes.common when all lines are commented-out by the current code:
``` #~ # For complete documentation of this file, please see Geany's main documentation #~ [styling] #~ # use foreground;background;bold;italic or named_style,bold,italic #~ #~ # used for filetype All/None #~ #default=default ```
1. Already commented-out lines are commented out once more which is ugly 2. Sections are commented out. This is problematic because users don't have to just comment-out the variable they want to modify, they must also comment-out the section in which the variable is which is easy to forget. 3. Empty lines are commented out which is unnecessary
This patch goes through the config file line by line and keeps sections, comments and empty lines unmodified so the result of the above example is this:
``` # For complete documentation of this file, please see Geany's main documentation [styling] # use foreground;background;bold;italic or named_style,bold,italic
# used for filetype All/None #default=default ```
The patch does this only for .conf filetype and uses the original code for other file types.
Modified Paths: -------------- src/ui_utils.c
Modified: src/ui_utils.c 38 lines changed, 35 insertions(+), 3 deletions(-) =================================================================== @@ -2096,6 +2096,33 @@ void ui_table_add_row(GtkTable *table, gint row, ...) }
+/* comment-out all lines that are not already commented out except sections */ +static void comment_conf_files(ScintillaObject *sci) +{ + gint line, line_count; + + line_count = sci_get_line_count(sci); + for (line = 0; line < line_count; line++) + { + gint pos_start = sci_get_position_from_line(sci, line); + gint pos_end = sci_get_line_end_position(sci, line); + gint pos; + + for (pos = pos_start; pos < pos_end; pos++) + { + gchar c = sci_get_char_at(sci, pos); + if (c == '[' || c == '#') + break; + if (!isspace(c)) + { + sci_insert_text(sci, pos_start, "#"); + break; + } + } + } +} + + static void on_config_file_clicked(GtkWidget *widget, gpointer user_data) { const gchar *file_name = user_data; @@ -2135,9 +2162,14 @@ static void on_config_file_clicked(GtkWidget *widget, gpointer user_data) doc = document_new_file(utf8_filename, ft, global_content); if (global_content) { - sci_select_all(doc->editor->sci); - keybindings_send_command(GEANY_KEY_GROUP_FORMAT, - GEANY_KEYS_FORMAT_COMMENTLINETOGGLE); + if (doc->file_type->id == GEANY_FILETYPES_CONF) + comment_conf_files(doc->editor->sci); + else + { + sci_select_all(doc->editor->sci); + keybindings_send_command(GEANY_KEY_GROUP_FORMAT, + GEANY_KEYS_FORMAT_COMMENTLINETOGGLE); + } sci_set_current_line(doc->editor->sci, 0); document_set_text_changed(doc, FALSE); sci_empty_undo_buffer(doc->editor->sci);
-------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).