@techee commented on this pull request.
> else new_dir_path = g_build_filename(directory_ref, new_dir, NULL); /* Build the new file list for completion */ completion_list = build_file_list(new_dir_path, new_dir); gtk_entry_completion_set_model (completion, completion_list); g_object_unref(completion_list); + + g_free(new_dir_path); Shouldn't also `curr_dir` be freed here? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1197#pullrequestreview-1187109858 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany-plugins/pull/1197/review/1187109...@github.com>