[geany/geany-plugins] 7d40df: Revert "projectorganizer: Make sure project is open before trying to expand the tree"

[Frank Lanitz]

Branch:      refs/heads/revert-555-prjorg_crash_fix
Author:      Frank Lanitz <frank at frank.uvena.de>
Committer:   GitHub <noreply at github.com>
Date:        Sat, 08 Apr 2017 14:54:41 UTC
Commit:      7d40df646aeac326d341742543824f832a08a252
             https://github.com/geany/geany-plugins/commit/7d40df646aeac326d341742543824f832a08a252

Log Message:
-----------
Revert "projectorganizer: Make sure project is open before trying to expand the tree"


Modified Paths:
--------------
    projectorganizer/src/prjorg-sidebar.c

Modified: projectorganizer/src/prjorg-sidebar.c
29 lines changed, 9 insertions(+), 20 deletions(-)
===================================================================
@@ -48,13 +48,6 @@ typedef enum
 	MATCH_PATTERN
 } MatchType;
 
-typedef struct
-{
-	GeanyProject *project;
-	GPtrArray *expanded_paths;
-} ExpandData;
-
-
 static GdkColor s_external_color;
 static GtkWidget *s_toolbar = NULL;
 static gboolean s_pending_reload = FALSE;
@@ -1217,24 +1210,22 @@ static gboolean expand_path(gchar *utf8_expanded_path, gboolean select)
 }
 
 
-static gboolean expand_on_idle(ExpandData *expand_data)
+static gboolean expand_on_idle(gpointer ptr)
 {
 	GeanyDocument *doc = document_get_current();
 
-	if (prj_org && geany_data->app->project == expand_data->project &&
-		expand_data->expanded_paths)
+	if (ptr)
 	{
+		GPtrArray *expanded_paths = ptr;
 		gchar *item;
 		guint i;
 
-		foreach_ptr_array(item, i, expand_data->expanded_paths)
+		foreach_ptr_array(item, i, expanded_paths)
 			expand_path(item, FALSE);
-		g_ptr_array_free(expand_data->expanded_paths, TRUE);
+		g_ptr_array_free(expanded_paths, TRUE);
 	}
 
-	g_free(expand_data);
-
-	if (!s_follow_editor || !doc || !doc->file_name)
+	if (!s_follow_editor || !doc || !doc->file_name || !geany_data->app->project || !prj_org)
 		return FALSE;
 
 	expand_path(doc->file_name, TRUE);
@@ -1290,13 +1281,11 @@ static GPtrArray *get_expanded_paths(void)
 
 void prjorg_sidebar_update(gboolean reload)
 {
-	ExpandData *expand_data = g_new0(ExpandData, 1);
-
-	expand_data->project = geany_data->app->project;
+	GPtrArray *expanded_paths = NULL;
 
 	if (reload)
 	{
-		expand_data->expanded_paths = get_expanded_paths();
+		expanded_paths = get_expanded_paths();
 
 		load_project();
 		/* we get color information only after the sidebar is realized -
@@ -1306,7 +1295,7 @@ void prjorg_sidebar_update(gboolean reload)
 	}
 
 	/* perform on idle - avoids unnecessary jumps on project load */
-	plugin_idle_add(geany_plugin, (GSourceFunc)expand_on_idle, expand_data);
+	plugin_idle_add(geany_plugin, (GSourceFunc)expand_on_idle, expanded_paths);
 }
 
 



--------------
This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).