@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>

Reply via email to