<p>In <a href="https://github.com/geany/geany/pull/995#discussion_r59401942">src/ui_utils.c</a>:</p>
<pre style='color:#555'>> @@ -1919,7 +1919,9 @@ GtkWidget *ui_path_box_new(const gchar *title, GtkFileChooserAction action, GtkE
>    vbox = gtk_vbox_new(FALSE, 0);
>    if (gtk_widget_get_parent(path_entry))  /* entry->parent may be a GtkComboBoxEntry */
>    {
> -          GtkWidget *parent = gtk_widget_get_parent(path_entry);
> +          GtkWidget *parent = path_entry, *next_parent;
> +          while ((next_parent = gtk_widget_get_parent(parent)) != NULL)
> +                  parent = next_parent;
</pre>
<p>while at it doing something like that, we could simplify the code by always resolving the parent and using that, dropping the conditional here -- e.g. doing what this block does, but unconditionally.  If the first <code>get_parent()</code> returns NULL, it's the same as the else block anyway.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly or <a href="https://github.com/geany/geany/pull/995/files/8539388b6e51dc932fafa21f4c4395355c5d64ee#r59401942">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ_7_Zi0ybp8J9jzWzAhcPO6Ik2FUks5p28CQgaJpZM4IEg4r.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/995/files/8539388b6e51dc932fafa21f4c4395355c5d64ee#r59401942"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>